-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Tagging] Clean up Tagging models #85
Milestone
Comments
Done here: |
This was referenced Sep 29, 2023
@pomegranited @bradenmacdonald I think we can close this issue |
github-project-automation
bot
moved this from Refined - To do
to In Progress
in Modular Learning - Tagging and Libraries
Nov 23, 2023
bradenmacdonald
moved this from In Progress
to Done
in Modular Learning - Tagging and Libraries
Nov 23, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Story
"As an Open edX developer, I want the Open edX Tagging and Content Tagging models to be as clearly delineated and simple as possible, so that developing new tagging use cases is straightforward."
Description
While implementing #63 and #77, we ended up using custom Taxonomy and custom ObjectTag classes to achieve the goals of the tagging MVP. There was a lot of back-and-forth on these PRs and changes in direction, and while the result works fine, it could be cleaner.
See comments:
Completion criteria
get
andfilter
/exclude
clauses used in oel_tagging and content_tagging, and ensure that the appropriate indexes and unique_together constraints are in place.Documentation improvements
Relevant repositories
The text was updated successfully, but these errors were encountered: