Replace 'Start' with 'Resume' course after pre-test completion - #15229
Open
LianaHarris360 wants to merge 2 commits into
Open
Replace 'Start' with 'Resume' course after pre-test completion#15229LianaHarris360 wants to merge 2 commits into
LianaHarris360 wants to merge 2 commits into
Conversation
… first pre-test has been submitted
LianaHarris360
added
bug
github-actions
Bot
added
DEV: backend
Contributor
Build Artifacts
Smoke test screenshot |
AlexVelezLl
reviewed
Aug 27, 2026
AlexVelezLl
left a comment
AlexVelezLl
left a comment
Member
There was a problem hiding this comment.
Code changes look good, just a question of whether we should track the submission or the start of the test.
…n opened by learner
bjester
approved these changes
Aug 28, 2026
bjester
left a comment
bjester
left a comment
Member
There was a problem hiding this comment.
Code changes look good to me. @AlexVelezLl's suggestion was good too. I think this is ready for QA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This pull requests updates the action button label that opens a course for a learner, based on if the learner has completed the first pre-test.
Previously, if a learner had completed the 1st unit’s pre-test, the label for the open course button was “Start Course”, which was incorrect because the course had technically already been started.
Now, if the course’s first pre-test has been completed, the button label is “Resume Course”.
Because a test is closed by a coach, an active pre-test in the first unit could mean it hadn’t been taken by the learner yet, or was submitted and waiting to be closed by the coach, but the resume payload did not differentiate between the two.
get_resume_datahas been updated to include “submitted” onactive_test, using_test_submitted(), which checks the learner'sContentSummaryLog(progress >= 1) for the test's synthetic content id, the same id the progress-tracking API logs attempts against.References
Fixes #15193
Reviewer guidance
ResumeAfterPre-TestCompletion.mov
AI usage
I used Claude Code to update the backend & frontend tests, and to review the code changes.