docs(alerting): add unified alerts view page to starlight docs - #431
docs(alerting): add unified alerts view page to starlight docs#431jackiehanyang wants to merge 2 commits into
Conversation
Signed-off-by: Jackie Han <hnyng@amazon.com>
63079c1 to
a223d8e
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #431 +/- ##
=======================================
Coverage 55.62% 55.62%
=======================================
Files 4 4
Lines 169 169
Branches 47 47
=======================================
Hits 94 94
Misses 74 74
Partials 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Verified the anomaly-detection and forecasting content against the implementation in 1. Step namesThe wizard omits "the". The
2. "Advanced settings" is mislabeledThe shingle-size section is titled Advanced model parameters in the UI, not Advanced settings: Suggestion:
3. "Initially unselected" is slightly overstated
The data source field defaults to the data source selected in the view, or is left unselected if none applies. Also — not blocking, and outside this repo — this PR renames the anchors |
Thanks for verifying these details against the implementation. I addressed all three wording issues:
I also searched the Starlight documentation tree for links to #rule-types and #inspect-and-manage-a-rule and found no remaining references to the old anchors. The Starlight build and internal link validation pass. |
Signed-off-by: Jackie Han <hnyng@amazon.com>
Summary
Testing