Skip to content

fix(ui): align communications admin API paths with v0.17 module#285

Merged
kkopanidis merged 1 commit into
ui-rewritefrom
fix/communications-admin-api-paths
Jun 22, 2026
Merged

fix(ui): align communications admin API paths with v0.17 module#285
kkopanidis merged 1 commit into
ui-rewritefrom
fix/communications-admin-api-paths

Conversation

@kkopanidis

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the ui-rewrite branch

Other information:

Conduit-UI still called legacy standalone-module Admin API paths after the communications consolidation. Push token listing, test send, and several email operations returned 404 against v0.17+ backends.

  • Push: /pushNotifications/tokenGET /push/tokens; /pushNotifications/token/:idGET /push/tokens/:id (unwrap tokenDocuments); send routes → POST /push/send; multi-recipient test send fans out per user via POST /push/send
  • Email: /email/externalTemplatesGET /email/templates/external; sync PATCH /email/syncExternalTemplatesPOST /email/templates/external/sync; records GET /email/recordGET /email/emails with emailDocumentsrecords mapping; resend body { emailRecordId }{ id }; bulk template delete loops DELETE /email/templates/:id
  • Email delivery status badge shows "Unavailable" (no admin REST route for status polling)
  • Template upload to provider throws a clear error (no admin upload route)

Test plan

  • Push settings (/push-notifications/settings): load and save config
  • Push tokens (/push-notifications/tokens): list, search, pagination — requests hit /push/tokens
  • Push test send (/push-notifications/test): single and multiple recipients
  • Email settings (/email/settings): load and save
  • Email templates: list, create, edit, delete single and bulk selected
  • External templates tab loads; sync triggers POST /email/templates/external/sync
  • Email send and records list; resend from record detail
  • Upload template action shows error toast instead of 404
  • SMS settings and test send unchanged (POST /sms/send)
  • pnpm build passes locally

Update push and email Admin API clients to match the unified communications
module routes, with graceful degradation where backend endpoints are missing.
@kkopanidis kkopanidis force-pushed the fix/communications-admin-api-paths branch from 1cce664 to 2cc31b6 Compare June 22, 2026 07:50
@kkopanidis kkopanidis merged commit 372d9fc into ui-rewrite Jun 22, 2026
3 checks passed
@kkopanidis kkopanidis deleted the fix/communications-admin-api-paths branch June 22, 2026 08:31
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