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

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

Conversation

shahzad31
Copy link
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 self-assigned this Oct 2, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
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
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