Skip to content

Language of timestamp should update on-the-fly#2414

Merged
compulim merged 4 commits intomicrosoft:masterfrom
compulim:fix-2360
Sep 25, 2019
Merged

Language of timestamp should update on-the-fly#2414
compulim merged 4 commits intomicrosoft:masterfrom
compulim:fix-2360

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Sep 22, 2019

Fixes #2360.

Changelog Entry

Fixed

Description

During our last React function component push, a bug was introduced to the RelativeTime.js. If language change, the display didn't adopt.

Specific Changes

  • Fix the bug in RelativeText.js
  • Add tests
  • Update test harness with a new updateProps() function, to introduce changes to the props, so we can modify language after the component is rendered

  • Testing Added

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 22, 2019

Coverage Status

Coverage increased (+0.4%) to 67.999% when pulling e963ba2 on compulim:fix-2360 into 850890f on microsoft:master.

Comment thread __tests__/setup/web/index.html Outdated
@compulim compulim force-pushed the fix-2360 branch 2 times, most recently from 09f02b3 to cb81cf9 Compare September 25, 2019 03:30
@compulim compulim merged commit 7de0a84 into microsoft:master Sep 25, 2019
@compulim compulim deleted the fix-2360 branch September 25, 2019 07:06
@compulim compulim mentioned this pull request Oct 25, 2019
55 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Timestamp locale does not change when updating language on-the-fly

3 participants