Skip to content

[Uptime] Update Moment Locale to keep it localized to the component#55177

Merged
shahzad31 merged 1 commit intoelastic:masterfrom
shahzad31:fix/location-tags-locale-effecting-whole-uptime-app
Jan 17, 2020
Merged

[Uptime] Update Moment Locale to keep it localized to the component#55177
shahzad31 merged 1 commit intoelastic:masterfrom
shahzad31:fix/location-tags-locale-effecting-whole-uptime-app

Conversation

@shahzad31
Copy link
Copy Markdown
Contributor

@shahzad31 shahzad31 commented Jan 17, 2020

Summary

Fix #55176

Reseting Moment locale to default after usage in the Location Status Tag component.

I am not able to find another way to keep it localized.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@shahzad31 shahzad31 added release_note:skip Skip the PR/issue when compiling release notes v7.6.0 v8.0.0 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Jan 17, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/uptime (Team:uptime)

@shahzad31 shahzad31 self-assigned this Jan 17, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@justinkambic
Copy link
Copy Markdown
Contributor

I verified this works by running a moment.js elsewhere on the page in master and this patch.

master

image

this patch

image

Copy link
Copy Markdown
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

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

LGTM

@shahzad31 shahzad31 merged commit 81bcaed into elastic:master Jan 17, 2020
@shahzad31 shahzad31 deleted the fix/location-tags-locale-effecting-whole-uptime-app branch January 17, 2020 23:24
shahzad31 added a commit to shahzad31/kibana that referenced this pull request Jan 20, 2020
shahzad31 added a commit to shahzad31/kibana that referenced this pull request Jan 20, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jan 20, 2020
* upstream/master: (24 commits)
  Show error page when accessing unavailable app (elastic#54656)
  [ML] Improving job wizards with datafeed aggregations (elastic#55180)
  remove flaly assetion. a license presence tested anyway (elastic#55289)
  fix commonly used ranges uptime (elastic#54930)
  [SIEM] Use proper icons on Detections view (elastic#55215)
  Fix: invalid translation referenced (elastic#54901)
  [State Management] Remove AppState from edit_index_pattern page (elastic#54104)
  Implements `getStartServices` on server-side (elastic#55156)
  Move vis_vega_type/data_model tests to jest (elastic#55186)
  [SIEM] [Detection Engine] Update status on rule details page (elastic#55201)
  Fix KQL value suggestions for nested fields (elastic#54820)
  Enforce camelCase format for a plugin id (elastic#53759)
  [SIEM] Detection engine cleanup for rule details/creation/edit page (elastic#55069)
  Remove nested root from index pattern (elastic#54978)
  [Reporting/Migration] ReportingSetup, LegacySetup (elastic#54198)
  [SIEM] [Detection Engine] Fixes duplicate rule action (elastic#55252)
  [SIEM] Detections add alert & signal tab (elastic#55127)
  Management API - redirect on disabled app path (elastic#55136)
  [SIEM][Detection Engine] Fixes critical regression on the backend with immutable and tags
  update local (elastic#55177)
  ...
shahzad31 added a commit that referenced this pull request Jan 20, 2020
shahzad31 added a commit that referenced this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.6.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Uptime] Fix Moment Local change effecting other parts of the app

4 participants