Skip to content

notification-channels: events (resource.action) subscription filter#33

Merged
acoshift merged 1 commit into
mainfrom
notification-events
Jun 20, 2026
Merged

notification-channels: events (resource.action) subscription filter#33
acoshift merged 1 commit into
mainfrom
notification-events

Conversation

@acoshift

Copy link
Copy Markdown
Member

What

Follows deploys-app/api#90: the notification subscription's separate resource-type / action axes become a single events list of resource.action patterns.

  • Rewrites the Subscription filters section: two axes (events + outcomes), a grammar table (*, deployment.*, *.delete, deployment.deploy), and a note that listing patterns is an OR (so one channel can watch deployment.* and domain.create without matching domain.delete).
  • Switches every CLI example (create, update, the pull example) from --resource-type/--action to --event, and updates the "What you get" blurb.

Verification

hugo builds clean. Depends on api#90 (contract) + the CLI/apiserver PRs.

Follow api#90: the subscription's separate resource-type/action axes become a
single events list of resource.action patterns. Rewrite the Subscription
filters section (grammar table *, resource.*, *.delete, resource.action; OR
semantics) and switch every CLI example from --resource-type/--action to
--event. hugo builds clean.
@deploys-app deploys-app Bot temporarily deployed to pr-33 June 20, 2026 09:33 Destroyed
@deploys-app

deploys-app Bot commented Jun 20, 2026

Copy link
Copy Markdown

Preview deleted (PR closed).

@acoshift acoshift merged commit cffb0b8 into main Jun 20, 2026
1 check passed
@acoshift acoshift deleted the notification-events branch June 20, 2026 09:52
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.

1 participant