Skip to content

update: OS destination creation moved to notifications #1190

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
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

update: OS destination creation moved to notifications #1190

wants to merge 4 commits into from

Conversation

@wojcik-dorota
Copy link
Contributor

@wojcik-dorota wojcik-dorota commented Nov 21, 2025 •
edited
Loading

https://aiven.atlassian.net/browse/MA-3631

DOC PREVIEW

Checklist

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

@wojcik-dorota wojcik-dorota requested a review from a team as a code owner November 21, 2025 08:53
@wojcik-dorota wojcik-dorota self-assigned this Nov 21, 2025
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2025 •
edited
Loading

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a7a5300
Status: ✅  Deploy successful!
Preview URL: https://00d90017.aiven-docs.pages.dev
Branch Preview URL: https://ma-3631-review-alerts-with-o.aiven-docs.pages.dev

View logs

Copilot finished reviewing on behalf of wojcik-dorota November 21, 2025 08:56
Copilot AI reviewed Nov 21, 2025
View reviewed changes
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 updates the OpenSearch alerting documentation to reflect that destination creation has been moved to the notifications plugin. The documentation now uses "notification channel" terminology instead of "destination" throughout.

  • Terminology updated from "Destination" to "Notification channel" throughout the document
  • Navigation instructions updated to use the Notifications panel instead of the Destination tab
  • Various grammar and style improvements for clarity

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wojcik-dorota wojcik-dorota force-pushed the MA-3631-review-alerts-with-osd-how-to-as-destinations-are-deprecated branch from fb76147 to 2fb0efe Compare November 21, 2025 09:02
Copilot finished reviewing on behalf of wojcik-dorota November 21, 2025 09:07
Copilot AI reviewed Nov 21, 2025
View reviewed changes
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

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

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wojcik-dorota wojcik-dorota force-pushed the MA-3631-review-alerts-with-osd-how-to-as-destinations-are-deprecated branch from 2fb0efe to 315d4fd Compare November 21, 2025 09:15
ArthurAnsquer
ArthurAnsquer previously requested changes Nov 21, 2025
View reviewed changes
Copilot finished reviewing on behalf of wojcik-dorota November 21, 2025 13:17
Copilot AI reviewed Nov 21, 2025
View reviewed changes
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

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

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot finished reviewing on behalf of wojcik-dorota November 21, 2025 13:53
Copilot AI reviewed Nov 21, 2025
View reviewed changes
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

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

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot finished reviewing on behalf of wojcik-dorota November 21, 2025 14:00
Copilot AI reviewed Nov 21, 2025
View reviewed changes
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@ArthurAnsquer ArthurAnsquer Awaiting requested review from ArthurAnsquer

At least 1 approving review is required to merge this pull request.

Assignees

@wojcik-dorota wojcik-dorota

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wojcik-dorota @ArthurAnsquer