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

Add Graph Insights and Watchdog Explains docs #24141

Merged
merged 14 commits into from
Aug 9, 2024
Merged

Add Graph Insights and Watchdog Explains docs #24141

merged 14 commits into from
Aug 9, 2024

Conversation

estherk15
Copy link
Contributor

@estherk15 estherk15 commented Jul 15, 2024 •
edited
Loading

What does this PR do? What is the motivation?

  • Add new Graph Insights category
  • Move Metric Correlations under Graph Insights
  • New Watchdog Explains documentation

Merge instructions

[x] Merge after review

Additional notes

@estherk15 estherk15 added the WORK IN PROGRESS No review needed, it's a wip ;) label Jul 15, 2024
@estherk15 estherk15 requested a review from a team as a code owner July 15, 2024 16:04
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Jul 15, 2024
@github-actions GitHub Actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

New or renamed files

Renamed files

Modified Files

@github-actions github-actions bot added the Images Images are added/removed with this PR label Aug 6, 2024
@estherk15 estherk15 removed the WORK IN PROGRESS No review needed, it's a wip ;) label Aug 9, 2024
@estherk15 estherk15 force-pushed the esther/docs-7934-watchdog-explains branch from f3f971c to 8dc6238 Compare August 9, 2024 13:57
jhgilbert
jhgilbert approved these changes Aug 9, 2024
View reviewed changes
Copy link
Contributor

@jhgilbert jhgilbert left a comment

Choose a reason for hiding this comment

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

Approved with some minor suggestions, thank you!

content/en/dashboards/graph_insights/_index.md Outdated Show resolved Hide resolved
content/en/dashboards/graph_insights/_index.md Outdated Show resolved Hide resolved
content/en/dashboards/graph_insights/correlations.md Outdated Show resolved Hide resolved
content/en/dashboards/graph_insights/correlations.md Outdated Show resolved Hide resolved
content/en/dashboards/graph_insights/watchdog_explains.md Outdated Show resolved Hide resolved
content/en/dashboards/graph_insights/watchdog_explains.md Outdated Show resolved Hide resolved
@estherk15 estherk15 merged commit 39eefce into master Aug 9, 2024
14 checks passed
@estherk15 estherk15 deleted the esther/docs-7934-watchdog-explains branch August 9, 2024 18:07
rtrieu pushed a commit that referenced this pull request Sep 3, 2024
@estherk15 @jhgilbert @rtrieu
Add Graph Insights and Watchdog Explains docs (#24141)
ba99c11 
* Add Graph Insights and Watchdog Explains docs

* Fix title

* Fix reference link

* Add info div to graph insight for individual feature

* Add filter features and images

* Add suggestions from PM review

* Update images

* Fix links

* Apply suggestions from code review

* Fix broken link

* Apply suggestions from code review

Co-authored-by: Jen Gilbert <[email protected]>

---------

Co-authored-by: Jen Gilbert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhgilbert jhgilbert jhgilbert approved these changes

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend Images Images are added/removed with this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@estherk15 @jhgilbert