Skip to content

chore(appeals): add broadcast routes (no-ticket)#3593

Merged
bryn-pins merged 1 commit intomainfrom
task/broadcast-endpoints
Apr 8, 2026
Merged

chore(appeals): add broadcast routes (no-ticket)#3593
bryn-pins merged 1 commit intomainfrom
task/broadcast-endpoints

Conversation

@bryn-pins
Copy link
Copy Markdown
Contributor

Describe your changes

Endpoints to broadcast entities

  • events not done in time
  • success tests don't work as all broadcast calls are mocked...

Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Comment thread appeals/api/src/server/endpoints/broadcast/broadcast.controller.js Dismissed
Copy link
Copy Markdown
Contributor

@DavidJWhiteDev DavidJWhiteDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a lovely piece of work.

Not for now, but maybe for later: I am thinking we might want to be able to take an array of values - eg appeal Ids, or doc ids, to allow batch processing?

also for later consideration: can we add one for appeal events please?

@bryn-pins bryn-pins marked this pull request as ready for review April 7, 2026 13:39
@bryn-pins bryn-pins enabled auto-merge April 7, 2026 13:39
Copy link
Copy Markdown
Contributor

@DavidJWhiteDev DavidJWhiteDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all good!

@bryn-pins bryn-pins added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit d50debb Apr 8, 2026
8 checks passed
@bryn-pins bryn-pins deleted the task/broadcast-endpoints branch April 8, 2026 10:00
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.

3 participants