Skip to content

Specifying allowed alert types#1063

Open
lovesprung wants to merge 2 commits into
kptdev:mainfrom
nokia:contributors_note
Open

Specifying allowed alert types#1063
lovesprung wants to merge 2 commits into
kptdev:mainfrom
nokia:contributors_note

Conversation

@lovesprung

Copy link
Copy Markdown
Contributor

Title

Specifying allowed alert types


Description

Adding a short specification to the contributors guide on allowed alert types. Based on the corrections done in #1042.


Type of Change

  • Bug fix
  • New feature
  • Enhancement
  • Refactor
  • Documentation
  • Tests
  • Other: ________

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>
@netlify

netlify Bot commented Jun 19, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit 0b99965
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a35080f1ffae4000852cd25
😎 Deploy Preview https://deploy-preview-1063--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 docs contribution guide to explicitly specify which Docsy alert types (and corresponding color values) are allowed, aligning alert usage guidance with the earlier documentation harmonization work.

Changes:

  • Replaces the generic “use Docsy alert shortcodes” guidance with a concrete list of allowed alert types.
  • Adds a small reference table mapping allowed alert titles to Docsy color values.
  • Adds a rule clarifying that alert titles should remain standardized (Note/Warning/Critical).

Comment thread docs/content/en/docs/12_contributing/docs-contribution/_index.md Outdated
Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>
@dosubot dosubot Bot added the lgtm #ededed label Jun 19, 2026
@efiacor efiacor requested review from a team June 19, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm #ededed size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants