Skip to content

Conversation

@nreese
Copy link
Contributor

@nreese nreese commented Nov 9, 2020

PR adds icon when layer is filtered by time

Screen Shot 2020-11-05 at 3 06 49 PM

PR allows for turning off global time range for layer and joins

Screen Shot 2020-11-09 at 12 28 14 PM

Screen Shot 2020-11-09 at 12 28 22 PM

@nreese nreese added release_note:enhancement Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.11.0 labels Nov 9, 2020
@nreese nreese requested a review from thomasneirynck November 9, 2020 19:32
@nreese nreese requested review from a team as code owners November 9, 2020 19:32
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@botelastic botelastic bot added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Nov 9, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@nreese nreese changed the title @nreese [Maps] show icon when layer is filtered by time and allow layers to ignore global time range [Maps] show icon when layer is filtered by time and allow layers to ignore global time range Nov 9, 2020
Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

thx a lot, this is really useful imho. it will remove a lot of ambivalence, and help user troubleshoot their map. e.g. it's very common to hear "why aren't my results showing" because the timefilter is set by default at "-15m". It will also help with matching layers to geo-threshold jobs in the alerting framework. These geo-threshold jobs do not take into account the time-field for the boundary-layer. In those cases, it would not be possible to build maps that correspond to such a job-configuration.

}),
});
}
if (this.state.isFilteredByGlobalTime) {
Copy link
Contributor

Choose a reason for hiding this comment

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

as discussed, this can toggle on/off directly when the flag turns on/off.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in a97a4cd

@nreese nreese requested a review from thomasneirynck November 9, 2020 21:35
Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

thx!

lgtm when green

@nreese
Copy link
Contributor Author

nreese commented Nov 10, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
maps 620 623 +3

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 3.2MB 3.2MB +42.0B
maps 2.8MB 2.8MB +7.3KB
total +7.3KB

History

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

nreese added a commit to nreese/kibana that referenced this pull request Nov 10, 2020
…gnore global time range (elastic#83006)

* [Maps] show icon when layer is filtered by time and allow layers to ignore global time range

* show icon if layer is narrowed by time fitler

* tslint

* apply global time to source check box

* apply global time to join check box

* tslint and jest expect updates

* one more tslint fix

* tslint, fix apm jest test, update time filter icon when disabling applyGlobalTime

Co-authored-by: Kibana Machine <[email protected]>
nreese added a commit that referenced this pull request Nov 10, 2020
…gnore global time range (#83006) (#83067)

* [Maps] show icon when layer is filtered by time and allow layers to ignore global time range

* show icon if layer is narrowed by time fitler

* tslint

* apply global time to source check box

* apply global time to join check box

* tslint and jest expect updates

* one more tslint fix

* tslint, fix apm jest test, update time filter icon when disabling applyGlobalTime

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
phillipb added a commit to phillipb/kibana that referenced this pull request Nov 10, 2020
…kibana into bootstrap-node-details-overlay

* 'bootstrap-node-details-overlay' of github.com:phillipb/kibana: (49 commits)
  [Security Solution] Fix DNS Network table query (elastic#82778)
  [Workplace Search] Consolidate groups routes (elastic#83015)
  Adds cloud links to user menu (elastic#82803)
  [Security Solution][Detections] - follow up cleanup on auto refresh rules (elastic#83023)
  [App Search] Added the log retention panel to the Settings page (elastic#82982)
  [Maps] show icon when layer is filtered by time and allow layers to ignore global time range (elastic#83006)
  [DOCS] Consolidates drilldown pages (elastic#82081)
  [Maps] add on-prem EMS config (elastic#82525)
  migrate i18n mixin to KP (elastic#81799)
  [bundle optimization] fix imports of react-use lib (elastic#82847)
  [Discover] Add metric on adding filter (elastic#82961)
  [Lens] Performance refactoring for indexpattern fast lookup and Operation support matrix computation (elastic#82829)
  skip flaky suite (elastic#82804)
  Fix SO query for searching across spaces (elastic#83025)
  renaming built-in alerts to Stack Alerts (elastic#82873)
  [TSVB] Disable using top_hits in pipeline aggregations (elastic#82278)
  [Visualizations] Remove kui usage (elastic#82810)
  [Visualizations] Make the icon buttons labels more descriptive (elastic#82585)
  [Lens] Do not reset formatting when switching between custom ranges and auto histogram (elastic#82694)
  Fix ilm navigation (elastic#81664)
  ...
gmmorris added a commit to gmmorris/kibana that referenced this pull request Nov 10, 2020
…na into alerts/stack-alerts-public

* 'alerts/stack-alerts-public' of github.com:gmmorris/kibana:
  [Security Solution] Fix DNS Network table query (elastic#82778)
  [Workplace Search] Consolidate groups routes (elastic#83015)
  Adds cloud links to user menu (elastic#82803)
  [Security Solution][Detections] - follow up cleanup on auto refresh rules (elastic#83023)
  [App Search] Added the log retention panel to the Settings page (elastic#82982)
  [Maps] show icon when layer is filtered by time and allow layers to ignore global time range (elastic#83006)
  [DOCS] Consolidates drilldown pages (elastic#82081)
  [Maps] add on-prem EMS config (elastic#82525)
  migrate i18n mixin to KP (elastic#81799)
  [bundle optimization] fix imports of react-use lib (elastic#82847)
  [Discover] Add metric on adding filter (elastic#82961)
  [Lens] Performance refactoring for indexpattern fast lookup and Operation support matrix computation (elastic#82829)
  skip flaky suite (elastic#82804)
  Fix SO query for searching across spaces (elastic#83025)
  renaming built-in alerts to Stack Alerts (elastic#82873)
  [TSVB] Disable using top_hits in pipeline aggregations (elastic#82278)
  [Visualizations] Remove kui usage (elastic#82810)
  [Visualizations] Make the icon buttons labels more descriptive (elastic#82585)
  [Lens] Do not reset formatting when switching between custom ranges and auto histogram (elastic#82694)
:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:enhancement Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Geo Former Team Label for Geo Team. Now use Team:Presentation Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants