Skip to content

Add API endpoint to upsert subscriptions with attribs#3134

Open
dfabulich wants to merge 1 commit into
knadh:masterfrom
dfabulich:api-upsert-subscription
Open

Add API endpoint to upsert subscriptions with attribs#3134
dfabulich wants to merge 1 commit into
knadh:masterfrom
dfabulich:api-upsert-subscription

Conversation

@dfabulich

Copy link
Copy Markdown
Contributor

Fixes #3133

@dfabulich dfabulich force-pushed the api-upsert-subscription branch from 2c1939e to 230d7aa Compare July 8, 2026 01:19
@knadh

knadh commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Hi @dfabulich. I've only been able to give this a cursory look so far, but this should be /api/subscribers/:id/subscription and not a multi-subscriber endpoint.

@knadh knadh added the enhancement New feature or request label Jul 8, 2026
@dfabulich

Copy link
Copy Markdown
Contributor Author

I think it makes sense to do some bikeshedding on how exactly to spell the endpoint URL, but I don't think it makes sense to have an :id parameter in the URL.

The idea is that the API client has an email address and N list IDs, but the client doesn't know whether the email address is already subscribed or not, and certainly doesn't already know the ID of the existing subscriber (if any).

I'm trying to mirror the shape of /api/public/subscription's endpoint, but in an authenticated API endpoint, so I can set some attribs, and I'm specifically trying to avoid making multiple API requests.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API for upserting a subscription

2 participants