Skip to content

[Uptime] Change default status filter in ping list to all on monitor page #47108

Merged
shahzad31 merged 3 commits intoelastic:masterfrom
shahzad31:fix/issue-52-monitor-page-status-dropdown-default-to-all
Oct 3, 2019
Merged

[Uptime] Change default status filter in ping list to all on monitor page #47108
shahzad31 merged 3 commits intoelastic:masterfrom
shahzad31:fix/issue-52-monitor-page-status-dropdown-default-to-all

Conversation

@shahzad31
Copy link
Copy Markdown
Contributor

@shahzad31 shahzad31 commented Oct 2, 2019

Summary

Fixes elastic/uptime#52

This PR will change default status drop down value to "all" from "down" in ping list table on monitor page.

Testing:

Open Uptime app, and try navigation to monitor page from overview page, while landing on monitor page, default value in dropdown should be "all" instead "down".

image

@shahzad31 shahzad31 added release_note:fix v8.0.0 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.5.0 labels Oct 2, 2019
@shahzad31 shahzad31 requested a review from justinkambic October 2, 2019 11:36
@shahzad31 shahzad31 self-assigned this Oct 2, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/uptime (Team:uptime)

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

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

WFG

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@shahzad31 shahzad31 merged commit e322acc into elastic:master Oct 3, 2019
@shahzad31 shahzad31 deleted the fix/issue-52-monitor-page-status-dropdown-default-to-all branch October 3, 2019 08:57
shahzad31 added a commit to shahzad31/kibana that referenced this pull request Oct 3, 2019
…page (elastic#47108)

* change default status filter in ping list to all

* update snaps
jloleysens added a commit to jloleysens/kibana that referenced this pull request Oct 3, 2019
…ysens/kibana into console-fix-menu-actions-alignment

* 'console-fix-menu-actions-alignment' of github.com:jloleysens/kibana: (59 commits)
  [Canvas] i18n for page manager, page preview, and shape preview (elastic#46865)
  [SIEM] hide siem data on empty string or _all (elastic#47166)
  [Uptime] Shim UI exports for new platform (elastic#44722)
  [Monitoring] Metricbeat Migration Wizard Tests (elastic#47139)
  [Logs UI] Support partitioned log rate results in API (elastic#46751)
  [APM] Remove beta badge from transaction breakdown (elastic#47163)
  test: 💍 convert Interpreter font function tests to Jest (elastic#47193)
  visualizations 👉 NP-ready (elastic#47142)
  [Uptime] Update pings chart colors (elastic#46780)
  Add scroll into view to avoid tooltip overlap (elastic#47106)
  Variety of quick a11y fixes (elastic#46569)
  [Graph] Empty workspace overlay (elastic#45547)
  [ML] Converts index and saved search selection to React (elastic#47117)
  [Uptime] Change default status filter in ping list to all on monitor page  (elastic#47108)
  refactor: 💡 convert Interpreter .js -> .ts (elastic#44545)
  Add TypeScript rules to STYLEGUIDE [skip ci] (elastic#47125)
  chore(NA): fix logic behind cleaning x-pack node modules on build (elastic#47091)
  [SIEM] Update Settings Text (elastic#47147)
  Add KQL functionality in the find function of the saved objects (elastic#41136)
  [Maps] Add 'InjectedData' class and revise so File Upload Features are assigned to new InjectedData instances (elastic#46381)
  ...
shahzad31 added a commit to shahzad31/kibana that referenced this pull request Oct 7, 2019
…page (elastic#47108)

* change default status filter in ping list to all

* update snaps
shahzad31 added a commit that referenced this pull request Oct 8, 2019
…page (#47108) (#47546)

* change default status filter in ping list to all

* update snaps
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Oct 9, 2019
…page (elastic#47108) (elastic#47546)

* change default status filter in ping list to all

* update snaps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:fix Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Monitor page status dropdown for Monitor History component should default to All

3 participants