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

[Data Explorer][Discover][BUG] Fail to re-route to discover from single/surrounding doc due to lost history #4842

Closed
ananzh opened this issue Aug 28, 2023 · 1 comment
Labels
bug Something isn't working data explorer Issues related to the Data Explorer project discover for discover reinvent

Comments

@ananzh
Copy link
Member

ananzh commented Aug 28, 2023

Describe the bug

Currently, we are using two state managements for Discover (Redux from data explorer) and View single/surrounding doc (legacy). When click breadcrumb Discover or side navigation Discover, the url won't be able to pick the state (ex http://localhost:5603/app/discover#/). Ideally, we should see http://localhost:5603app/discover#?_a=... & _g=..., then route to data explorer.Therefore we see issue when re-route to discover from single/surrounding doc due to lost history.

Screenshot 2023-08-28 at 10 22 50
@ananzh ananzh added bug Something isn't working discover for discover reinvent data explorer Issues related to the Data Explorer project labels Aug 28, 2023
@ananzh ananzh removed the untriaged label Aug 29, 2023
@ananzh
Copy link
Member Author

ananzh commented Aug 31, 2023

PR is out. Resolve this issue.

@ananzh ananzh closed this as completed Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working data explorer Issues related to the Data Explorer project discover for discover reinvent
Projects
None yet
Development

No branches or pull requests

1 participant