Skip to content

insights: refactor- update insight view resolver to preload series data for the 1 case it didn't already

Warren Gifford requested to merge cw/series-data-load-lift into main

Created by: chwarwick

Previously the timeseries was loaded in the InsightViewResolver for JustInTime and Capture Group recorded series. Move the loading of the standard search timeseries data into InsightViewResolver for consistency.

Removed unused arguments to dataPoints field.

Marked legacy insights query as deprecated and a todo to remove a resolver that should be removed when insights is removed.

resolves #33804 (closed) closes #27089 (closed)

Test plan

No functionality change - existing automated tests pass

Manual testing: Created & verified 4 insights:

  1. Just in time
  2. Just in time capture group
  3. backfilled
  4. backfilled capture group

Merge request reports

Loading