Skip to content

openapi: update Subscription schemas for Intent-Based RSS#2504

Open
neilv-g wants to merge 1 commit into
neilv-rss-final-1from
neilv-rss-final-2
Open

openapi: update Subscription schemas for Intent-Based RSS#2504
neilv-g wants to merge 1 commit into
neilv-rss-final-1from
neilv-rss-final-2

Conversation

@neilv-g
Copy link
Copy Markdown
Collaborator

@neilv-g neilv-g commented May 28, 2026

Modify Subscription and SubscriptionResponse to support polymorphic channel identification, meaning two new "modes":

  • Explicit Mode: Provide a 'channel_id' (for manual channels like Webhook/Email).
  • Implicit Mode: Provide a 'channel_type' (e.g., 'rss'), allowing the system to automatically resolve or create the channel.

Additionally:

  • Flattened SubscriptionResponse to avoid intersection types in generated TypeScript.
  • Added 'channel_type' to response so the UI can identify the delivery mode without fetching the channel.

@neilv-g neilv-g force-pushed the neilv-rss-final-1 branch from a3931fb to f7c0d7e Compare May 29, 2026 16:18
@neilv-g neilv-g force-pushed the neilv-rss-final-2 branch from e316ca5 to 0c81fcd Compare May 29, 2026 16:18
@neilv-g neilv-g force-pushed the neilv-rss-final-1 branch from f7c0d7e to 1090487 Compare May 29, 2026 16:29
@neilv-g neilv-g force-pushed the neilv-rss-final-2 branch 6 times, most recently from 10d5cb6 to 9ba9412 Compare May 29, 2026 19:09
Modify Subscription and SubscriptionResponse to support polymorphic channel identification.

- Explicit Mode: Provide a 'channel_id' (for manual channels like Webhook/Email).

- Implicit Mode: Provide a 'channel_type' (e.g., 'rss'), allowing the system to automatically resolve or create the channel.

- Flattened SubscriptionResponse to avoid intersection types in generated TypeScript, improving DX.

- Added 'channel_type' to response so the UI can identify the delivery mode without fetching the channel.
@neilv-g neilv-g force-pushed the neilv-rss-final-2 branch from 9ba9412 to 5fb0564 Compare May 29, 2026 19:39
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