feat: Add retrieve object_tags REST API - #577
Conversation
|
|
||
| router = DefaultRouter() | ||
| router.register("taxonomies", views.TaxonomyOrgView, basename="taxonomy") | ||
| router.register("object_tags", oel_tagging_views.ObjectTagView, basename="object_tag") |
There was a problem hiding this comment.
TODO before merge:
Need to merge/tag openedx/openedx-core#68 and bump the installed openedx-learning version before this PR can be merged.
There was a problem hiding this comment.
@pomegranited My branch is actually based off of Romulo's branch openedx#32871 , since he already had the rest_api file structure setup (hence why the changes in my PR are very small) and since I thought his PR might be merging before mine, I kept it that way. If we think we'd want to have them separated, I can close this PR and open a new one against upstream master?
There was a problem hiding this comment.
Oh sure @yusuf-musleh, I'm happy to merge it to @rpenido 's branch, that makes sense.
We still need openedx/openedx-core#68 to be merged and tagged first, though.
pomegranited
left a comment
pomegranited
left a comment
There was a problem hiding this comment.
👍 -- though we need to bump the openedx-learning version before merging, see note.
- I tested this as part of openedx/openedx-core#68 (review)
- I read through the code
-
I checked for accessibility issuesN/A -
Includes documentationN/A
7ee6d0b to
3591589
Compare
056955c to
8ab7acd
Compare
36f920c to
f0793d7
Compare
f0793d7 to
25a1f76
Compare
* feat: add taxonomies for org api * chore: run CI * feat: Add retrieve object_tags REST API (#577) * chore: update requirements --------- Co-authored-by: Yusuf Musleh <yusuf@opencraft.com>
2 participants
Description
This PR exposes the retrieve object tag REST API endpoint from oel_tagging.
Supporting Information
Testing Instructions
make lms-shelpip uninstall openedx-learningmake lms-shellcd /edx/srcpip install -e openedx-learninghttp://localhost:18000/api/content_tagging/v1/object_tags/Private Ref: FAL-3452