Skip to content

Conversation

@petar-i-todorov
Copy link
Contributor

@petar-i-todorov petar-i-todorov marked this pull request as draft October 17, 2025 15:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the AI-powered insights documentation by splitting a single comprehensive article into four focused, modular articles that better organize the content for different user needs and workflows.

Key Changes:

  • Separated overview, setup, configuration, and extensibility content into distinct articles
  • Created dedicated article for REST service integration
  • Added standalone guide for custom AI client implementations
  • Improved content structure with clearer navigation between related topics

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
interactivity/ai-powered-insights-overview.md New overview article introducing AI-powered insights feature and key capabilities
interactivity/using-ai-powered-insights-rest-service.md New article explaining REST service setup and LLM provider configuration
interactivity/configuring-ai-powered-insights.md New article detailing declarative and programmatic customization options
interactivity/custom-iclient.md New article providing implementation guide for custom AI clients
interactivity/AI-powered-insights.md Original comprehensive article removed (content redistributed)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@telerik telerik deleted a comment from Copilot AI Oct 18, 2025
docs: make the ai-related snippets use tabs and add introductions for each of them
docs: add custom aiclient article
docs: add demo links
Update interactivity/configuring-ai-powered-insights.md
Update interactivity/custom-iclient.md

Co-Authored-By: Copilot <[email protected]>
@ighristov ighristov force-pushed the petodoro/ai-insights-docs branch from 4a050a1 to b8b423d Compare October 18, 2025 07:25
@petar-i-todorov petar-i-todorov marked this pull request as ready for review October 20, 2025 16:17
@petar-i-todorov
Copy link
Contributor Author

All four articles are currently located in the Interactivity section. They could be moved to a separate folder, but I am unsure if they will be easily discoverable. Open to suggestions; cc @elkinmilen @ighristov @yordan-mitev

image

@yordan-mitev
Copy link
Contributor

yordan-mitev commented Oct 23, 2025

The root structure of the left-hand navigation looks pretty crammed, so I don't think adding an additional node there would be user-friendly. With that said, I suggest renaming the "Interactivity" section to "Interactivity and AI".

yordan-mitev
yordan-mitev previously approved these changes Oct 23, 2025
Copy link
Contributor

@yordan-mitev yordan-mitev left a comment

Choose a reason for hiding this comment

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

The improvements are significant. Reviewing the changes was a pleasure, and I feel confident that I can configure and customize the AI insights by following the guidance here.

Copy link
Contributor

@elkinmilen elkinmilen left a comment

Choose a reason for hiding this comment

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

You have done a great improvement! Thank you so much! I feel confident I understand and can implement the feature now!
Please consider my comments. I choose Request changes to take a final look. Thank you!

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.

3 participants