-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat: record the user for library content writes #36513
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
|
Requires openedx/openedx-learning#290 |
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.
LGTM, though I do wish we'd been more consistent with the names of these various user parameters (user_id, created_by, reset_by, set_by, deleted_by)
Just update the requirements before merging.
87ec722 to
af15e55
Compare
Also: upgrade openedx-learning pin, 0.22 -> 0.23
af15e55 to
88ba4ba
Compare
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
|
2U Release Notice: This PR has been deployed to the edX production environment. |
We were previously not recording who was doing write/delete/reset operations. Prior to openedx-learning 0.23.0, we didn't have a place to write the user for a reset-to-publish. Upgrades the openedx-learning pin, 0.22 -> 0.23
We were previously not recording who was doing write/delete/reset operations. Prior to openedx-learning 0.23.0, we didn't have a place to write the user for a reset-to-publish. Upgrades the openedx-learning pin, 0.22 -> 0.23
We were previously not recording who was doing write/delete/reset operations. Prior to openedx-learning 0.23.0, we didn't have a place to write the user for a reset-to-publish. Upgrades the openedx-learning pin, 0.22 -> 0.23
We were previously not recording who was doing write/delete/reset operations.
Prior to openedx-learning 0.23.0, we didn't have a place to write the user for a
reset-to-publish.