-
Notifications
You must be signed in to change notification settings - Fork 75
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: add xpert summaries configuration by default for units #567
Conversation
germanolleunlp
commented
Aug 15, 2023
- Update modals to support reset and delete course xpert summary settings when is disabled
- Add radio buttons to specify the default value for the xpert course summary settings
603ea35
to
a015322
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #567 +/- ##
==========================================
+ Coverage 84.38% 84.42% +0.03%
==========================================
Files 259 259
Lines 4247 4307 +60
Branches 959 974 +15
==========================================
+ Hits 3584 3636 +52
- Misses 639 647 +8
Partials 24 24
☔ View full report in Codecov by Sentry. |
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.
Can you write more unit tests to cover the cases discovered by codecov that are not yet covered by unit tests?
@schenedx all the lines added have tests for them, I will add some more to increase the coverage for them 🆗 |
Done ✅ |
src/pages-and-resources/xpert-unit-summary/settings-modal/SettingsModal.jsx
Show resolved
Hide resolved
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.
👍
a015322
to
3b3009c
Compare
3b3009c
to
bbf3674
Compare
Updated UI: https://jira.2u.com/browse/ACADEMIC-16422 |
src/pages-and-resources/xpert-unit-summary/settings-modal/SettingsModal.jsx
Outdated
Show resolved
Hide resolved
…#567) * feat: add xpert summaries configuration by default for units