Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: initialize app for vertical tagging in course-discovery and add an ADR for it #4542

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

AfaqShuaib09
Copy link
Contributor

@AfaqShuaib09 AfaqShuaib09 commented Jan 13, 2025

This PR adds the ADR for Vertical Tagging feature and initialize a separate app for it in discovery

@bradenmacdonald
Copy link

This ADR is a bit confusing because in a course context, "Vertical" usually refers to a unit, but I take it that here it's referring to something like "subject area" ?

Are you familiar with the Taxonomy and Tagging functionality from openedx_tagging and the taxonomy features in Studio? I'm wondering if you can just create a business-oriented "Vertical Taxonomy" and then use openedx_tagging to tag the courses as needed.

Example taxonomy view in Studio:

Screenshot

@DawoudSheraz
Copy link
Contributor

Hey @bradenmacdonald, thanks for your input. Adding my responses below.

This ADR is a bit confusing because in a course context, "Vertical" usually refers to a unit, but I take it that here it's referring to something like "subject area" ?

Yes, this vertical refers to subject area but from a business point of view. While I agree that Vertical refers to a unit, that is primarily true in studio/cms terms. course-discovery does not follow all the studio conventions (such as course vs course run concept). However, we can think of a different name to avoid any potential conflicts for future (at least until the DEPR is not in place).

Are you familiar with the Taxonomy and Tagging functionality from openedx_tagging and the taxonomy features in Studio? I'm wondering if you can just create a business-oriented "Vertical Taxonomy" and then use openedx_tagging to tag the courses as needed.

This is something we are looking into. At the moment, it is not certain if we go for openedx_tagging as it is, given that it seems to be designed with platform in mind. Once we have that, this vertical tagging will be a candidate to be "mapped" as a taxonomy.

@AfaqShuaib09 AfaqShuaib09 marked this pull request as ready for review January 14, 2025 16:37
@DawoudSheraz
Copy link
Contributor

please add the app in INSTALLED_APPS before the merge.

docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
docs/decisions/0031-vertical-content-tagging.rst Outdated Show resolved Hide resolved
by granular tagging using vertical filters.
The Vertical Tagging functionality introduces a structured approach to categorizing courses by
aligning them with specific business objectives. It enables more effective management of courses based on verticals,
ensuring that courses can be easily discovered and grouped according to strategic priorities, thereby improving overall discoverability.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[suggest] Bullet points and some rephrasing
1. Introduce a structured approach to categorizing courses by aligning them with specific business objectives
2. Enable more effective classification of courses based on verticals, ensuring that courses can be easily discovered and grouped according to strategic priorities, thereby improving overall discoverability.

@AfaqShuaib09 AfaqShuaib09 merged commit 01a8aaa into master Jan 16, 2025
14 checks passed
@AfaqShuaib09 AfaqShuaib09 deleted the afaq/prod-4279 branch January 16, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants