-
Notifications
You must be signed in to change notification settings - Fork 200
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
feat: [FC-0056] courseware sidebar enhancement #1386
feat: [FC-0056] courseware sidebar enhancement #1386
Conversation
Thanks for the pull request, @ihor-romaniuk! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1386 +/- ##
==========================================
- Coverage 88.74% 88.66% -0.09%
==========================================
Files 303 307 +4
Lines 5224 5266 +42
Branches 1330 1338 +8
==========================================
+ Hits 4636 4669 +33
- Misses 572 581 +9
Partials 16 16 ☔ View full report in Codecov by Sentry. |
Sandbox deployment successful 🚀 |
bb73f23
to
b13de55
Compare
Sandbox deployment successful 🚀 |
Adding |
@ihor-romaniuk Thanks for these changes. It looks like codecov is failing right now. Once that's sorted out we should be good to start engineering review. EDIT: Actually, we might need to wait for openedx/edx-platform#34816 to get merged? |
Sandbox deployment successful 🚀 |
- Display section and sequence progress - Add tracking event to the unit button - Hide the horizontal unit navigation with enabled sidebar navigation
b13de55
to
d28f1bb
Compare
Sandbox deployment successful 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, works great! Thanks!
I'm rerunning tests before merging. |
@ihor-romaniuk 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Running into token issues for coverage. Let's deal with that separately. |
Settings
Description
This pull request adds several improvements to the courseware sidebar navigation.
Related PRs
Depends on BE PRs
Design
https://www.figma.com/file/gew5tORDX4Q7wxOS8vjqZu/side-nav-OEX?type=design&node-id=318-3234&mode=design&t=rBe1ToNYP8RY6QOp-0
Testing instructions