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.

Sign up for GitHub

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

Jump to bottom

[search-session] on-board search-session on content-management #157079

Closed
thomasneirynck opened this issue May 8, 2023 · 3 comments
Closed

[search-session] on-board search-session on content-management #157079

thomasneirynck opened this issue May 8, 2023 · 3 comments
Assignees
@mattkime
Labels
chore Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph)

Comments

@thomasneirynck
Copy link
Contributor

Remove dependency on saved-object client in the browser for the search-session SO.

Part of #157043
@botelastic botelastic bot added the needs-team Issues missing a team label label May 8, 2023
@thomasneirynck thomasneirynck added Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph) and removed needs-team Issues missing a team label labels May 8, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@mattkime mattkime self-assigned this May 10, 2023
@kertal
Copy link
Member

kertal commented Jun 5, 2023

@mattkime given we didn't use the saved object client and you're already working on BWCA this can be closed, right?

@mattkime
Copy link
Contributor

Replaced with BWCA-ing the routes as it doesn't use the browser side saved object client - #158981

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattkime mattkime

Labels
chore Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mattkime @kertal @thomasneirynck @elasticmachine