Clarify that saving a Discover session always saves to the library#6317
Merged
Conversation
Follow-up to #6037. Per Davis's review comment in the merged PR: saving a Discover session from the Discover app menu (Save or Save as) always saves the session to the library. The Add to dashboard option (9.5+) optionally adds it as a panel on a dashboard - it doesn't replace the library save. Only the Save table button and the Add panel flow from a dashboard create panels that bypass the library. The previous wording presented Add to dashboard options as a mutually exclusive choice between dashboard and library, which is incorrect. This commit reframes Add to dashboard as "also add to a dashboard" and clarifies that the library save always happens. Affected: - save-open-search.md: main Save step, Duplicate Save as step, Save panel edits step, Save-session H4 title and content, list bullet pointing to it - discover-get-started.md: Save your Discover session step - kibana-data-exploration-learning-tutorial.md: Discover-to-dashboard navigation paragraph (also fixes a conflation of the table toolbar flow with the full session save flow) Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
Contributor
Contributor
✅ Vale Linting ResultsNo issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
florent-leborgne
commented
May 6, 2026
florent-leborgne
commented
May 6, 2026
florent-leborgne
commented
May 6, 2026
florent-leborgne
commented
May 6, 2026
Co-authored-by: Florent LB <florent.leborgne@elastic.co>
davismcphee
approved these changes
May 6, 2026
Contributor
davismcphee
left a comment
davismcphee
left a comment
There was a problem hiding this comment.
Thanks for the quick fix, looks great now! And in general I think the new structure is much easier to follow 👌
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #6037 addressing @davismcphee's review comment on the merged PR.
Per Davis's correction:
The previous wording in #6037 framed the Add to dashboard options (
None,New,Existing) as a mutually exclusive choice between dashboard and library. This is incorrect: the library save always happens for the Save flow; only the dashboard placement is controlled by Add to dashboard.Changes
In
explore-analyze/discover/save-open-search.md:In
explore-analyze/discover/discover-get-started.md:In
explore-analyze/kibana-data-exploration-learning-tutorial.md:Test plan
save-open-search.md(main Save, Duplicate, Save panel edits) and the cross-link indiscover-get-started.md.Made with Cursor