fix: Include metadata for blocked sequence units to enable proper navigation #36891
Conversation
…igation (openedx#36485) The original issue was that when a sequence was locked due to prerequisites, the API returned an empty items array ([]). This prevented the frontend from knowing what units were inside the locked sequence, meaning it couldn't construct the URLs correctly for navigation so the next/previous buttons stop working.
|
Thanks for the pull request, @efortish! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
|
@mariajgrimaldi FYI! |
b4f4e3a to
6753de7
Compare
Backport from #36485
Description
The original issue was that when a sequence was locked due to prerequisites, the API returned an empty items array ([]). This prevented the frontend from knowing what units were inside the locked sequence, meaning it couldn't construct the URLs correctly for navigation so the next/previous buttons stop working.
This modification ensured that even when the sequence is locked, basic metadata for each unit is included. This allows the frontend to:
Know how many units exist and their IDs
Correctly construct URLs for "Previous" and "Next" buttons
Still enforce access restrictions (thanks to the is_gated: True flag)
With this change, the Mfe can continue functioning normally without requiring modifications, as it now consistently receives the necessary information, regardless of whether the sequence is locked or not.
This fix can be backported.
Test cases and test instructions
2025-04-04.12-55-52.mp4
Achievements
Related issues
#36826
openedx/frontend-app-learning#1546