Skip to content
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

Merged

Conversation

aehanno
Copy link
Contributor

@aehanno aehanno commented May 13, 2022

SUMMARY

Adding french translation for DatePicker component

ADDITIONAL INFORMATION

fix #20062

@aehanno aehanno changed the title fix : add locale to activate french translation for DatePicker component fix: Add locale to activate french translation for DatePicker component May 16, 2022
@aehanno
Copy link
Contributor Author

aehanno commented Jun 6, 2022

Hello @zhaoyongjie, just to know if you got time to check on my changes

@zhaoyongjie
Copy link
Member

@aehanno Sorry for the waiting. I will review PR tomorrow. Thanks for your patience.

@mathieudruart
Copy link
Contributor

Hi @zhaoyongjie any news on this PR ?

@pull-request-size pull-request-size bot added size/M and removed size/S labels Jul 7, 2022
@mathieudruart
Copy link
Contributor

Hi @zhaoyongjie is it possible to review this PR ? Thank you !

@nytai
Copy link
Member

nytai commented Aug 30, 2022

triggering CI

@nytai nytai closed this Aug 30, 2022
@nytai nytai reopened this Aug 30, 2022
@pull-request-size pull-request-size bot added size/S and removed size/M labels Sep 2, 2022
@mathieudruart
Copy link
Contributor

mathieudruart commented Sep 2, 2022

Hi @nytai we pushed a correction, is it possible to trigger CI ?

@zhaoyongjie
Copy link
Member

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.

@pull-request-size pull-request-size bot added size/L and removed size/S labels Sep 6, 2022
@pull-request-size pull-request-size bot added size/L and removed size/S labels Sep 7, 2022
@codecov
Copy link

codecov bot commented Sep 7, 2022

Codecov Report

Merging #20063 (ca89c4a) into master (286474c) will not change coverage.
The diff coverage is 66.66%.

@@           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     
Flag Coverage Δ
javascript 52.71% <66.66%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...trols/DateFilterControl/components/CustomFrame.tsx 70.96% <50.00%> (-1.45%) ⬇️
...ents/controls/DateFilterControl/utils/constants.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@zhaoyongjie zhaoyongjie left a 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.

@zhaoyongjie zhaoyongjie changed the title fix: Add locale to activate french translation for DatePicker component fix: Add locale for DatePicker component Sep 7, 2022
@zhaoyongjie zhaoyongjie merged commit c382d53 into apache:master Sep 7, 2022
eschutho pushed a commit that referenced this pull request Sep 27, 2022
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr>
Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
AAfghahi pushed a commit that referenced this pull request Oct 5, 2022
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr>
Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
AAfghahi pushed a commit that referenced this pull request Oct 6, 2022
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr>
Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
Fahrenheit35 pushed a commit to Fahrenheit35/superset that referenced this pull request Nov 11, 2022
Co-authored-by: Kevin Dethelot <kevin.dethelot@kosmos.fr>
Co-authored-by: Yongjie Zhao <yongjie.zhao@gmail.com>
@mistercrunch mistercrunch added 🍒 2.0.1 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 2.1.0 and removed 🚢 2.1.3 labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/L v2.0 v2.0.1 🍒 2.0.1 🚢 2.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impossible to have translation for DatePicker
7 participants