fix: ensure run-based assignments are taken into account on course page redirect #1196
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.
When a learner only has an assignment available, there is logic on the course about page that checks whether or not the learner only has an allocated assignment (i.e., no other subsidies available). If the learner is assignment-only and the course is not assigned, we redirect back to the dashboard.
In the case of a run-based assignment, we were missing a check for assignment-only learners against both the
assignment.contentKey
vs.assignment.parentContentKey
when comparing to the top-level course key in the course page route URL.With fix
Screen.Recording.2024-09-18.at.2.17.51.PM.mov
For all changes
Only if submitting a visual change