-
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
temp: logs for course export issue #31481
Conversation
e6f7ee8
to
6f57521
Compare
xmodule/modulestore/xml_exporter.py
Outdated
@@ -355,8 +355,13 @@ def adapt_references(subtree, destination_course_key, export_fs): | |||
Map every reference in the subtree into destination_course_key and set it back into the xblock fields | |||
""" | |||
subtree.runtime.export_fs = export_fs # ensure everything knows where it's going! | |||
# TODO: Remove logging statements after export issue is resolved |
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.
Also add the ticket number here. It is good for reference
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.
Ticket number addded
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.
Just one comment. Rest looks good
6f57521
to
fa27253
Compare
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
EdX Release Notice: This PR has been deployed to the production environment. |
1 similar comment
EdX Release Notice: This PR has been deployed to the production environment. |
Logs for course export issue.
Export file was missing some content
Ticket:
https://2u-internal.atlassian.net/browse/INF-667