Skip to content

Conversation

@mgiota
Copy link
Contributor

@mgiota mgiota commented Dec 1, 2025

Fixes #244242

This PR:

  • brings back storing the selected history range as part of the URL
  • removes auto refresh on history tab
  • brings back locator refactoring
  • removes unused TAB_ID_URL_PARAM

@mgiota mgiota self-assigned this Dec 1, 2025
@github-actions github-actions bot added the author:actionable-obs PRs authored by the actionable obs team label Dec 1, 2025
@mgiota mgiota added Team:obs-ux-management release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Dec 2, 2025
@mgiota mgiota marked this pull request as ready for review December 2, 2025 09:10
@mgiota mgiota requested review from a team as code owners December 2, 2025 09:10
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

? `?instanceId=${encodeURIComponent(instanceId)}`
: '';
const path = !!sloId ? `/${encodeURIComponent(sloId)}${queryParams}` : '/';
public readonly getLocation = async ({ sloId, instanceId, tabId }: SloDetailsLocatorParams) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kdelemme I saw you had refactored the locator here, so I decided to bring that part as well

import { SloRemoteCallout } from './slo_remote_callout';
import { ActionModalProvider } from '../../../context/action_modal';

export const TAB_ID_URL_PARAM = 'tabId';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TAB_ID_URL_PARAM is not used anywhere, so I removed it. Rest consts were moved to deeplinks package

Copy link
Contributor

@kdelemme kdelemme left a comment

Choose a reason for hiding this comment

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

Works as expected 👍🏻

@mgiota mgiota enabled auto-merge (squash) December 3, 2025 16:33
@mgiota mgiota merged commit f93bb7c into elastic:main Dec 3, 2025
13 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
slo 1322 1323 +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/deeplinks-observability 86 92 +6

Async chunks

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

id before after diff
slo 991.7KB 992.2KB +569.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
slo 33.2KB 33.5KB +324.0B
Unknown metric groups

API count

id before after diff
@kbn/deeplinks-observability 87 93 +6

History

cc @mgiota

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

Labels

author:actionable-obs PRs authored by the actionable obs team backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-management v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SLO] Bring back url state for history tab

4 participants