Skip to content

"Unsaved dashboard changes" dialog is not shown when navigating directly to a different dashboard #1641

@daniellevinson

Description

@daniellevinson

Describe the bug
It's probably not a likely scenario, but if you navigate from an unsaved dashboard directly to a different dashboard, the dialog is not shown.
The fix might be easier than writing this ticket (copying/moving the dashboard id comparison from cacheQuery to willTransition https://github.com/yavin-dev/framework/blob/master/packages/dashboards/addon/routes/dashboards/dashboard.ts#L232)
but probably requires some more thorough testing to see that nothing else breaks.

To Reproduce

  1. Go to https://yavin-dev.github.io/framework/#/dashboards/2/view
  2. Click on Home (breadcrumb link)
  3. Go to Coin Health Check dashboard
  4. Add "Coin counts" to dashboard
  5. Don't save ☠️
  6. Navigate directly 2 steps back to dashboard 2

Screen Shot 2022-05-25 at 1 24 11 PM

Expected behavior
Show dialog unless same dashboard id.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions