Skip to content

feat: record the user for library content writes - #36513

Merged
kdmccormick merged 1 commit into
openedx:masterfrom
ormsbee:draft-log-4
Apr 16, 2025
Merged

feat: record the user for library content writes#36513
kdmccormick merged 1 commit into
openedx:masterfrom
ormsbee:draft-log-4

Conversation

@ormsbee

@ormsbee ormsbee commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

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.

@ormsbee

ormsbee commented Apr 11, 2025

Copy link
Copy Markdown
Contributor Author

Requires openedx/openedx-core#290

@bradenmacdonald bradenmacdonald left a comment

Copy link
Copy Markdown
Contributor

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.

Also: upgrade openedx-learning pin, 0.22 -> 0.23
@kdmccormick
kdmccormick enabled auto-merge (squash) April 16, 2025 19:58
@kdmccormick
kdmccormick merged commit fd7ce0e into openedx:master Apr 16, 2025
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

tonybusa pushed a commit to tonybusa/edx-platform that referenced this pull request Apr 23, 2025
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
UsamaSadiq pushed a commit that referenced this pull request May 14, 2025
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
marlonkeating pushed a commit that referenced this pull request Jul 15, 2025
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants