Skip to content

fix(date-picker): modify weekStart value for ar locale #6154

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

Merged
merged 2 commits into from
Dec 27, 2022

Conversation

annierm18
Copy link
Contributor

Related Issue: n/a

Summary

Previously, date-picker had the weekStart value as "7" for the locale "ar", when it's actual weekStart as returned by intl object is "6". JSAPI team asked for the weekStart to be changed to "6" so that the week start day for the "ar" locale can be consistent across Esri web products.

@annierm18 annierm18 requested a review from a team as a code owner December 21, 2022 22:46
@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label Dec 21, 2022
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@geospatialem geospatialem added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Dec 21, 2022
Copy link
Contributor

@eriklharper eriklharper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Aine!

@geospatialem geospatialem added pr ready for visual snapshots Adding this label will run visual snapshot testing. and removed pr ready for visual snapshots Adding this label will run visual snapshot testing. labels Dec 27, 2022
@annierm18 annierm18 merged commit f9fe230 into master Dec 27, 2022
@annierm18 annierm18 deleted the annierm18/i18n-ar-datepicker branch December 27, 2022 19:23
@github-actions github-actions bot added this to the 2023 January Priorities milestone Dec 27, 2022
benelan added a commit that referenced this pull request Dec 28, 2022
* master:
  chore(notice): Remove invalid kind values (#6186)
  1.0.0-next.703
  build(deps): update dependencies. (#6189)
  refactor(pagination)!: Remove css variable (#6183)
  docs: consistent slot messaging across components (#6190)
  docs(flow-item): make showBackButton internal (#6179)
  ci(chromatic): set higher diffThreshold and delay for unstable tests (#6180)
  build(deps): bump storybook from 6.5.14 to 6.5.15 (#6176)
  build(deps): bump @typescript-eslint/eslint-plugin from 5.47.0 to 5.47.1 (#6175)
  build(deps): bump @storybook/addon-docs from 6.5.14 to 6.5.15 (#6174)
  docs: update component READMEs (#6184)
  build(deps): bump calcite-ui-icons from 3.20.5 to 3.20.7 (#6170)
  build(deps): bump @typescript-eslint/parser from 5.45.1 to 5.47.1 (#6177)
  1.0.0-next.702
  fix(date-picker): modify weekStart value for ar locale (#6154)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports for broken functionality. Issues should include a reproduction of the bug. pr ready for visual snapshots Adding this label will run visual snapshot testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants