-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
fix: visual and progress score display enhancement #34860
fix: visual and progress score display enhancement #34860
Conversation
Thanks for the pull request, @ihor-romaniuk! This repository is currently maintained by @openedx/wg-maintenance-edx-platform. 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.
|
Hi @ihor-romaniuk - it looks like some checks didn't run on this. Are you able to try again? |
96140d8
to
a11b065
Compare
Hello @mphilbrick211. This Pull Request has been rebased on the |
Thanks, @ihor-romaniuk! |
a11b065
to
8562dd5
Compare
Hi @openedx/wg-build-test-release-security-patchers! Are you able to help with this and its 2 backports? |
@mphilbrick211 I can help with this, also do we really need to backport this? |
@farhaanbukhsh - thanks! At this point I'm not sure about the backports, but can check. |
8562dd5
to
3e388a7
Compare
Hey @ihor-romaniuk I am trying to test this and I am still not getting any UI changes that are suggested in the PR, can you have a look at these please? :) |
@ihor-romaniuk any updates here? @mphilbrick211 if @ihor-romaniuk is not ready to pursue these changes then we should close this and other backport PRs. |
Hello @farhaanbukhsh, |
@farhaanbukhsh I think it's OK to close this and the backports for the moments. @ihor-romaniuk - if you want to pursue in the future, we can reopen. Thank you both! |
@mphilbrick211 Thank you @ihor-romaniuk Feel free to open a new PR, we can start with making a PR to master and then once it gets merged we can backport those changes if required, this will save a lot of effort from your side and from maintainers side to review them. |
Description
This pull request contains styling fixes and display problem score for Problem xblock.
Related Pull Requests
PR to the quince branch: #34861
PR to the redwood branch: #34862
Screenshots before:
Screenshots after:
Steps to Reproduce:
/admin/waffle/flag/