Skip to content

Add Feature Settings, update GenAI settings#6122

Merged
lcawl merged 6 commits into
elastic:mainfrom
lcawl:feature-settings
Apr 30, 2026
Merged

Add Feature Settings, update GenAI settings#6122
lcawl merged 6 commits into
elastic:mainfrom
lcawl:feature-settings

Conversation

@lcawl
Copy link
Copy Markdown
Contributor

@lcawl lcawl commented Apr 24, 2026

Summary

Update https://www.elastic.co/docs/explore-analyze/ai-features/manage-access-to-ai-assistant to describe the differences between the existing GenAI Settings page and the new Feature Settings page. Refresh screenshots and all pages that refer to GenAI Settings for default model/connector configuration (since that's now moved to the other app).

Reviewer notes

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@lcawl lcawl requested review from a team as code owners April 24, 2026 22:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

Vale Linting Results

Summary: 1 suggestion found

💡 Suggestions (1)
File Line Rule Message
solutions/security/ai/ai-assistant.md 48 Elastic.WordChoice Consider using 'deactivate, deselect, hide, turn off' instead of 'disable', unless the term is in the UI.

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.

Copy link
Copy Markdown
Member

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@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.

LGTM2!

Comment thread explore-analyze/ai-features/manage-access-to-ai-assistant.md Outdated
Comment thread explore-analyze/ai-features/manage-access-to-ai-assistant.md Outdated
Comment thread explore-analyze/ai-features/manage-access-to-ai-assistant.md Outdated
Comment thread explore-analyze/ai-features/manage-access-to-ai-assistant.md
@saikatsarkar056
Copy link
Copy Markdown
Contributor

@lcawl Will this new Feature Settings section serve as the main documentation page for the feature settings going forward, or is a dedicated page planned? I want to make sure we're linking to the right place from the Kibana UI.

Co-authored-by: Saikat Sarkar <132922331+saikatsarkar056@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@lcawl
Copy link
Copy Markdown
Contributor Author

lcawl commented Apr 27, 2026

@lcawl Will this new Feature Settings section serve as the main documentation page for the feature settings going forward, or is a dedicated page planned? I want to make sure we're linking to the right place from the Kibana UI.

Yes, unless there are plans to change the Feature Settings significantly in the near future, I think it make sense to keep it here for now. I have a PR open to use this URL from the Feature Settings page here: elastic/kibana#265633

Copy link
Copy Markdown
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

Looks great. Page level changes make sense individually, and together the IA is clear.

Comment thread explore-analyze/ai-features/manage-access-to-ai-assistant.md
@lcawl lcawl enabled auto-merge (squash) April 30, 2026 16:33
@lcawl lcawl merged commit 20d58f5 into elastic:main Apr 30, 2026
12 checks passed
@lcawl lcawl deleted the feature-settings branch April 30, 2026 16:33
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.

5 participants