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

[Time to Visualize] Breadcrumb doesn't behave correctly when switching the dashboard.allowByValueEmbeddables flag from true to false #87671

Closed
stratoula opened this issue Jan 7, 2021 · 4 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas

Comments

@stratoula
Copy link
Contributor

Kibana version:
7.11

Describe the bug:
I have the dashboard.allowByValueEmbeddables flag set to true and I create a Lens chart from the dashboard. When I set the flag to false and edit the Lens viz I see the Visualize link in breadcrumb which works ok as it redirects me to the list. But when I go back to history then it redirects me to the Lens Create page.

Screenshot 2021-01-07 at 2 34 47 PM

Screenshot 2021-01-07 at 2 35 02 PM

Steps to reproduce:

  1. Set the dashboard.allowByValueEmbeddables to true
  2. Go to Dashboard and create a new Lens Visualization
  3. Set the flag to false
  4. Refresh the dashboard
  5. Choose the edit Lens
  6. The breadcrumb has the Visualize link
  7. Click the Visualize link
  8. Go back to history. I can't see my chart, I see the create a new Lens viz instead

Expected behavior:
As it is suggested here we should throw an error toast and redirect to the visualize listing page any time there is value input but by value embeddables are not allowed.

@stratoula stratoula added bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas labels Jan 7, 2021
@elasticmachine
Copy link
Contributor

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

@ThomThomson ThomThomson self-assigned this Jan 7, 2021
@kmartastic kmartastic added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated labels Mar 2, 2021
@kmartastic
Copy link
Contributor

@ThomThomson -- can you estimate a loe on this?

@kmartastic kmartastic added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. and removed impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Mar 2, 2021
@ThomThomson ThomThomson added loe:small Small Level of Effort and removed loe:needs-research This issue requires some research before it can be worked on or estimated labels Mar 2, 2021
@ThomThomson
Copy link
Contributor

As Time to Visualize has been quite stable, it's seeming less and less likely that users would want to disable it. And if they did, it's even less likely that they would attempt to edit a lens which was saved by value after disabling the flag.

I think that it might be the right call to remove the feature flag entirely rather than fixing bugs based around users turning it off.

I don't think removing the flag would constitute a breaking change either, but we'll have to check telemetry to see if anyone has it turned off.

@ThomThomson
Copy link
Contributor

Closing this one as the plan is to remove the flag entirely.

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 Feature:Dashboard Dashboard related features impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas
Projects
None yet
Development

No branches or pull requests

4 participants