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

feat(18003): add localStorage for the configs of the metric panels #223

Merged

Conversation

vanch3d
Copy link
Contributor

@vanch3d vanch3d commented Dec 1, 2023

See https://hivemq.kanbanize.com/ctrl_board/57/cards/18003/details/

This PR is a follow-up to the main metrics initiative, see #215. It adds persistence to the obervability panels created by the users, ensuring consistency across session.

The persistency is done through localStorage, under individual keys (i.e. edge.reports-#####).

Out-of-scope

  • There is no UI-based mechanism to clear all the metric panels, especially the orphans.

@vanch3d vanch3d requested a review from sfrehse December 1, 2023 09:24
@vanch3d vanch3d self-assigned this Dec 1, 2023
@cla-bot cla-bot bot added the cla-signed label Dec 1, 2023
@vanch3d vanch3d changed the base branch from master to refactor/17951/metric-editor-ux December 1, 2023 09:25
Copy link
Collaborator

@simon622 simon622 left a comment

Choose a reason for hiding this comment

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

Added graphs and stats, refresh pages moved about and noted my configuration was persisted as expected - great work.

@vanch3d vanch3d merged commit 96e7078 into refactor/17951/metric-editor-ux Dec 5, 2023
7 checks passed
@vanch3d vanch3d deleted the feat/18003/metrics-persistence branch December 5, 2023 11:46
vanch3d added a commit that referenced this pull request Dec 5, 2023
reactor(17951): Refactor the metrics editor to a better collapsible UX

* refactor(17951): clean metric flags

* fix(17951): change portal for the select component in order to be int…

* refactor(17951): refactor the collapsible component of the editor

* refactor(17951): fix translations

* fix(17951): fix import & testid

* test(17951): add test

* Merge pull request #223
vanch3d added a commit that referenced this pull request Dec 5, 2023
reactor(17951): Refactor the metrics editor to a better collapsible UX

* refactor(17951): clean metric flags

* fix(17951): change portal for the select component in order to be int…

* refactor(17951): refactor the collapsible component of the editor

* refactor(17951): fix translations

* fix(17951): fix import & testid

* test(17951): add test

* Merge pull request #223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants