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

[Dashboards] Vega visualization is reporting unsaved changes #196954

Open
nickpeihl opened this issue Oct 18, 2024 · 4 comments
Open

[Dashboards] Vega visualization is reporting unsaved changes #196954

nickpeihl opened this issue Oct 18, 2024 · 4 comments
Labels
bug Fixes for quality problems that affect the customer experience Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@nickpeihl
Copy link
Member

A dashboard containing a Vega visualization is erroneously reporting unsaved changes.

To see the error

  1. Add the web logs sample data
  2. Open the [Logs] Web Traffic dashboard
  3. Change a control or size of a panel and Save changes.
  4. Go to the Dashboard listing page and re-open the [Logs] Web Traffic dashboard
  5. Notice that an Unsaved changes badge appears despite not having changed anything.

To debug this, log the unsavedPanelState by adding a console log statement here.

e.g.

]).subscribe(([dashboardChanges, unsavedPanelState, controlGroupChanges]) => {
      // calculate unsaved changes
      console.log(`unsavedPanelState`, unsavedPanelState);
      const hasUnsavedChanges =
        Object.keys(omit(dashboardChanges, keysNotConsideredUnsavedChanges)).length > 0 ||
        unsavedPanelState !== undefined ||
        controlGroupChanges !== undefined;

When opening the dashboard the following unsaved panel state is logged to the console.

{
    "8e59c7cf-6e42-4343-a113-c4a255fcf2ce": {
        "savedObjectId": "cb099a20-ea66-11eb-9425-113343a037e3"
    }
}
@nickpeihl nickpeihl added bug Fixes for quality problems that affect the customer experience Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas labels Oct 18, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@Heenawter Heenawter added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Oct 18, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

@nickofthyme
Copy link
Contributor

Possibly related to changes in #194330

@nreese nreese removed the Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas label Oct 23, 2024
@nreese
Copy link
Contributor

nreese commented Oct 23, 2024

Removing label Team:Presentation. Issue is in visualize embeddable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

5 participants