-
Notifications
You must be signed in to change notification settings - Fork 124
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: Fixed the display of the selection of available time zones (for Palm) #895
fix: Fixed the display of the selection of available time zones (for Palm) #895
Conversation
Thanks for the pull request, @DmytroAlipov! 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. |
ea3efe4
to
c009d8a
Compare
c009d8a
to
3fcddc9
Compare
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## open-release/palm.master #895 +/- ##
===========================================================
Coverage ? 38.76%
===========================================================
Files ? 110
Lines ? 2291
Branches ? 625
===========================================================
Hits ? 888
Misses ? 1318
Partials ? 85 ☔ View full report in Codecov by Sentry. |
Hi @abdullahwaheed and @Mashal-m! Would one of you be able to merge this for us? The master has been merged. Thanks! |
Hi @openedx/2u-aperture! Would someone be able to please merge this for us? Thanks! |
c72411e
into
openedx:open-release/palm.master
@DmytroAlipov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Backport from the master.
Description
An error was found when selecting a timezone in the user account.
In this PR, a time zone selection was added.
Then the select field was rewritten in PR. (Paragon form component deprecations)
However, these changes have broken the time zone selection field.
Before:
After: