-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
refactor(date picker): Migrate Date Picker to Ant Design 5 #31019
Merged
geido
merged 47 commits into
apache:master
from
msyavuz:msyavuz/refactor/migrate-datepicker
Jan 13, 2025
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
2b1950a
refactor(date picker): initial commit
msyavuz ba7a325
refactor(moment): Replace Moment.js with DayJs
msyavuz ded1285
fix(package): sync package.json and tests
msyavuz 63d7c02
refactor(moment): refactor moment out of components
msyavuz 17d0041
refactor(dayjs): re-add moment and migrate components
msyavuz 8eb99d3
refactor(dayjs): migrate some components
msyavuz f556d9c
chore(moment): add comment and change moment to dayjs
msyavuz 8572277
refactor(dayjs): centralize dayjs extensions
msyavuz fd2f451
chore(dayjs): add test for extensions and change package.jsons
msyavuz 4a55d06
fix(type): type from dayjs instead
msyavuz 9c8eca1
Merge branch 'master' into msyavuz/momentjs-to-dayjs
msyavuz 6cbac28
fix(test): only wait for load if its loading
msyavuz e9fcacd
test(AlertReportModal): use events directly instead?
msyavuz a035c87
test(AlertReportModal): fix the test
msyavuz af97ee0
refactor(pluigin-echarts): move to dayjs
msyavuz 5a0862a
chore: remove unused waitfor
msyavuz a3e10bf
feat(moment): remove timezone-support, migrate handlebars to dayjs
msyavuz 638599f
chore(timezone-support): remove usages
msyavuz 854a66a
refactor(legacy-plugins): migrate to dayjs
msyavuz 972bf08
fix(timezoneselector): use map instead of array and remove unneccessa…
msyavuz 58a7355
refactor(alert report modal): remove loading check
msyavuz f60a221
chore(alert report modal): remove unused import
msyavuz 568b4f5
Merge branch 'msyavuz/refactor/migrate-datepicker' of https://github.…
msyavuz 962ad20
chore(package): remove package manager
msyavuz 4fee444
feat(DatePicker): migrate to dayjs and fix some usages
msyavuz 554b4f2
Merge branch 'master' into msyavuz/refactor/migrate-datepicker
msyavuz 0a49f2f
chore(package): remove moment related stuff
msyavuz 97f88be
chore(webpack): remove unused functions
msyavuz 5aa5581
chore(moment): readd moment-plugin for peer deps
msyavuz c1c5cb5
Update superset-frontend/packages/superset-ui-chart-controls/src/cons…
msyavuz 85f4664
fix(days): import and include locales dynamically
msyavuz 9c0cb0c
Merge branch 'master' into msyavuz/refactor/migrate-datepicker
msyavuz 056dc14
chore(datepicker): cleanup
msyavuz 8710a60
fix(webpack): change plugin for build errors
msyavuz 79925f3
chore(timezones): remove deduping logic
msyavuz b3d0a24
chore(moment): remove moment locales plugin
msyavuz 4939091
Merge branch 'master' into msyavuz/refactor/migrate-datepicker
msyavuz 3de0a5d
refactor(invalid date): turn to sentence case
msyavuz 5835167
test(TimezoneSelector): change test for all timezones
msyavuz ace0177
fix(TimezoneSelector): change tests for all timezones
msyavuz 5b22577
feat(useLocale): add useLocale hook to unify locale
msyavuz f5a699d
test(ListView): provide store
msyavuz 32f2d46
chore(webpack): remove unused function
msyavuz 246ad4d
feat(webpack): add ignore plugin
msyavuz 407a819
fix(webpack): import dayjs locales directly instead of webpack plugin
msyavuz c6b6aab
Update superset-frontend/src/components/TimezoneSelector/index.tsx
geido ddb7af8
chore(format): remove line
msyavuz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Incompatible Date-Time Format![category Functionality](https://camo.githubusercontent.com/be66cdb454480bb0cbb9d568c5947d38543421fabac2bcbd8b17a41babf9d7f2/68747470733a2f2f696d672e736869656c64732e696f2f62616467652f46756e6374696f6e616c6974792d303238346337)
Tell me more
What is the issue?
The date-time format 'YYYY-MM-DD hh:mm a' is a moment.js format. Dayjs requires 'YYYY-MM-DD HH:mm A'.
Why this matters
Using moment.js date-time format with dayjs will result in incorrect date and time formatting in the UI.
Suggested change
Change the format to 'YYYY-MM-DD HH:mm A' to match dayjs formatting requirements.
Chat with Korbit by mentioning @korbit-ai, and give a 👍 or 👎 to help Korbit improve your reviews.
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.
"YYYY-MM-DD-hh:mm a" is a valid dayjs format.
https://day.js.org/docs/en/display/format