-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat: entity link view and api #36190
base: master
Are you sure you want to change the base?
feat: entity link view and api #36190
Conversation
Thanks for the pull request, @navinkarkera! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.
|
c748d56
to
1423fe2
Compare
1423fe2
to
55e68a9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Working beautifully.
- I tested this using the test instructions from feat: course libraries page frontend-app-authoring#1641
Also had to run migrations,recreate_upstream_links
andreindex_studio
to get the data ready to test. - I read through the code that's changed since feat: upstream-downstream entity linking #36111 (ref compare)
-
I checked for accessibility issuesN/A - Includes documentation - code comments
-
User-facing strings are extracted for translationN/A
Description:
Adds api for listing upstream entity links for a given course.
Test instructions: See openedx/frontend-app-authoring#1641
Part of: openedx/frontend-app-authoring#1565
Private-ref: FAL-4006