-
Notifications
You must be signed in to change notification settings - Fork 13.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: Add locale for DatePicker component #20063
fix: Add locale for DatePicker component #20063
Conversation
superset-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.tsx
Outdated
Show resolved
Hide resolved
Hello @zhaoyongjie, just to know if you got time to check on my changes |
@aehanno Sorry for the waiting. I will review PR tomorrow. Thanks for your patience. |
Hi @zhaoyongjie any news on this PR ? |
Hi @zhaoyongjie is it possible to review this PR ? Thank you ! |
triggering CI |
Hi @nytai we pushed a correction, is it possible to trigger CI ? |
Hi @mathieudruart, Thank you for the patient. I have pushed a branch to the repo base on your current branch, you should just 1) rebase the master branch 2) cheery-pick the latest commit to this for unblocking the CI. |
b5588f5
to
0404d90
Compare
Codecov Report
@@ Coverage Diff @@
## master #20063 +/- ##
=======================================
Coverage 66.58% 66.58%
=======================================
Files 1790 1790
Lines 68416 68419 +3
Branches 7287 7287
=======================================
+ Hits 45554 45556 +2
Misses 20986 20986
- Partials 1876 1877 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
lgtm, thanks for the new feature.
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr> Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr> Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr> Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr> Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
SUMMARY
Adding french translation for DatePicker component
ADDITIONAL INFORMATION
fix #20062