Skip to content
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

Updating library version of a pasted library_content component loses learner state #33910

Closed
kdmccormick opened this issue Dec 11, 2023 · 2 comments
Assignees

Comments

@kdmccormick
Copy link
Member

kdmccormick commented Dec 11, 2023

Reproduction

  • In Studio:
    • Create a library ("libX") and add a problem to it.
    • Within a course unit, create a Library Content component ("LC1") and point it at libX.
    • Go to libX and make an edit. For example, you could change the question prompt slightly.
    • Go back to the unit with LC1. It should prompt you to "Update Now". Don't update.
    • Copy LC1 and paste it into the same unit (creating "LC2"). It should also prompt you to "Update Now". Don't update.
    • Publish the unit.
  • In LMS:
    • Go to LC1. Submit the correct answer.
    • Go to LC2. Submit the correct answer.
  • In Studio:
    • Go to the unit with LC1 and LC2.
    • Hit "Update Now" on both LC1 and LC2.
    • Publish the unit.
  • Back in LMS:
    • Go to LC1. Your correct answer should still be submitted.
    • Go to LC2.
      • Expected: Your correct answer should still be submitted.
      • Actual: The problem is unanswered. Your learner state has been lost.

Explanation

Slack: https://openedx.slack.com/archives/C05NT2YN820/p1702058122859519?thread_ts=1702047279.641349&cid=C05NT2YN820

Fix

We've discussed a fix in the #content-libraries-v2 channel in Slack: https://openedx.slack.com/archives/C05NT2YN820/p1702072194157149?thread_ts=1702047279.641349&cid=C05NT2YN820

Affects

master (confirmed)

Quince (assumed but not confirmed)

@kdmccormick kdmccormick changed the title Pasted library_content components lose learner state upon update-from-library Updating library version of a pasted library_content component loses learner state Dec 11, 2023
@robrap robrap removed this from Arch-BOM Dec 12, 2023
@connorhaugh
Copy link
Contributor

#33926

@kdmccormick
Copy link
Member Author

Note: Braden's PR will fix this going forward, but any copy/pasted library_content components that have been made in the meantime will lose learner state if ever updated.

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

No branches or pull requests

3 participants