-
Notifications
You must be signed in to change notification settings - Fork 3
Add Canvas platform for learning resources #2345
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
Conversation
OpenAPI ChangesShow/hide 3 changes: 0 error, 0 warning, 3 info
Unexpected changes? Ensure your branch is up-to-date with |
90863d6
to
6ae9e11
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.
In heroku we had a release-phase script that ran the data migrations and a few other things. Not sure if this has carried over to EKS but we should make sure its there otherwise this wont get applied on deploy
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.
changes look good. deploying this will be a good test to see if the heroku deploy scripts in k8 works
6ae9e11
to
4d9e444
Compare
What are the relevant tickets?
Related to https://github.com/mitodl/hq/issues/7779
Description (What does it do?)
LearningResourcePlatform
entry for canvasHow can this be tested?
./manage.py backpopulate_canvas_courses
on the main branch../manage.py backpopulate_canvas_courses --overwrite
on this branch.platform=canvas
: http://ai.open.odl.local:8065/admin/learning_resources/learningresource/?etl_source=canvasplatform=canvas
: http://ai.open.odl.local:8065/api/v1/content_file_search/?platform=canvas./manage.py generate_embeddings --all --overwrite
and then check http://ai.open.odl.local:8065/api/v0/vector_content_files_search/?platform=canvas to make sure some files are returned