-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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] Fix Dashboard OnAppLeave #86193
[Time to Visualize] Fix Dashboard OnAppLeave #86193
Conversation
…etermine whether or not to show onAppLeave confirm
Pinging @elastic/kibana-presentation (Team:Presentation) |
725f147
to
cc3aded
Compare
cc3aded
to
85f2d43
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
App Services code changes LGTM.
…ConfirmOnAppLeave
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected for me.
Just want to confirm, it's not possible to get into a state where there is a isTransgerInProgress
where it's not immediately redirecting you to another app?
@crob611, This is a good point. I think the only way that could happen is if the transfer was cancelled, but there isn't yet code in place to cancel the state transfer once it's started. If we add any confirm modals or anything for the redirect, we will have to be aware of this. |
💚 Build SucceededMetrics [docs]Async chunks
Distributable file count
Page load bundle
History
To update your PR or re-run it, just comment with: |
Added isTransferInProgress to embeddable_state_transfer for apps to determine whether or not to show onAppLeave confirm
Summary
A small fix that adds shows a confirm modal on leaving the dashboard application with unsaved changes.
This warning will not show when the user is redirecting to an editor via the create new button or via the edit link on a dashboard panel. This is accomplished with a new property in the
embeddableStateTransfer
service, used to determine whether or not a transfer is in progress.Note Ignore the blue header, it's just a browser script to help tell kibana instances apart.
Checklist
Delete any items that are not applicable to this PR.
For maintainers