Skip to content

Clean up AutoOps notification settings page#6944

Merged
wajihaparvez merged 4 commits into
elastic:mainfrom
wajihaparvez:autoops-notifications-fix
Jun 26, 2026
Merged

Clean up AutoOps notification settings page#6944
wajihaparvez merged 4 commits into
elastic:mainfrom
wajihaparvez:autoops-notifications-fix

Conversation

@wajihaparvez

@wajihaparvez wajihaparvez commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

When trying to fix broken anchor links on the Notification settings page, I realized some of the info was inconsistent/confusing. So I did some clean up and polished the page.

Closes #6940

Generative AI disclosure

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

@wajihaparvez wajihaparvez self-assigned this Jun 16, 2026
@wajihaparvez wajihaparvez requested a review from a team as a code owner June 16, 2026 21:40
@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.

@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 3 warnings found

⚠️ Warnings (3): Fix when the suggestion improves clarity or correctness.
File Line Rule Message
deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md 82 Elastic.MenuArrowsBold Use '→' to separate menu items, not '' or '='. Example: Select Manage index → Add lifecycle policy.
deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md 112 Elastic.MenuArrowsBold Use '→' to separate menu items, not '' or '='. Example: Select Manage index → Add lifecycle policy.
deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md 131 Elastic.MenuArrowsBold Use '→' to separate menu items, not '' or '='. Example: Select Manage index → Add lifecycle policy.

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.

@wajihaparvez wajihaparvez requested a review from arnonst June 16, 2026 22:40

@arnonst arnonst left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@shainaraskas shainaraskas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cool, very thorough. I like the page structure.

couple of comments on depth and then some random style nits to consider, but lgtm :)

Comment thread deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md Outdated
Comment on lines +81 to +89
1. Go to [Slack API](https://api.slack.com/apps)
2. Click **Create an App** > **From scratch**.
3. Enter an app name and select your workspace, and then click **Create App**.
4. From the left menu, select **Incoming Webhooks**.
5. Turn on **Activate Incoming Webhooks**.
6. Click **Request to Add New Webhook** and submit a request to your organization's admins.
7. When your request is approved, select a Slack channel from the list of **Webhook URLs for Your Workspace** and click **Allow**.
8. Copy the webhook URL.
9. Store the URL securely. You will need it when configuring the connector in AutoOps.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

there are lots of steps here that could age. is there a way to link off to official docs for the other product and then just indicate the key settings? this applies to all external procedures on this page.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I considered this, but unfortunately not all the external products have clear documentation about this. I've replaced some sections with external links, but the rest of them will have to stay

:::{dropdown} VictorOps

$$$ec-autoops-victorops$$$
:::{dropdown} VictorOps

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should this dropdown also have splunk on-call in it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I opted not to add it because the UI just says "VictorOps"

Comment thread deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-notifications-settings.md Outdated
@wajihaparvez wajihaparvez merged commit 954807a into elastic:main Jun 26, 2026
8 checks passed
@wajihaparvez wajihaparvez deleted the autoops-notifications-fix branch June 26, 2026 15:57
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.

Rework notification settings page

3 participants