Skip to content

Conversation

florent-leborgne
Copy link
Contributor

@florent-leborgne florent-leborgne commented Oct 9, 2025

This PR:

  • Adds a section to indicate how to view active profiles in Discover
  • Brings missing information about how to inspect queries. Similar information existed in the dashboards docs so I turned it into a snippet and included mentions of Discover in it. Now the same snippet is used in both places.

Closes: #3090

@florent-leborgne florent-leborgne requested review from a team as code owners October 9, 2025 10:11
Copy link

github-actions bot commented Oct 9, 2025

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Words and their arrangement LGTM 👍

Copy link
Contributor

@mdbirnstiehl mdbirnstiehl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines +46 to +53
### View active context-aware experience

You can check which experience is currently active for your current Discover session. This can help you confirm whether the type of data you're currently exploring is properly detected or if Discover is currently using its default experience.

1. Select **Inspect** from Discover's toolbar.
1. Open the **View** dropdown, then select **Profiles**.

The various profiles listed show details such as the active solution and data source contexts, which determine Discover's context-aware experiences.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I overlooked this one in the checklist you shared. While it's probably useful for some users, this was mainly added as a troubleshooting tool for profile issues in prod deployments. I don't personally have an issue with public docs for it, but we should check with @ninoslavmiskovic first to see what he thinks.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is fine, since we have a section that explains we are rendering something for the user that is contextual, to also show how they can check it. There is no harm in that. I doubt it will be used much, but no harm in documentation it. ++ LGTM

@florent-leborgne florent-leborgne enabled auto-merge (squash) October 10, 2025 09:32
@florent-leborgne florent-leborgne merged commit 8495820 into elastic:main Oct 10, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Discover] Ability to view active profiles in Discover

5 participants