Skip to content

Move registration to async Producer pattern#143

Merged
steeevin88 merged 1 commit into
mainfrom
steven/fix-consumer-write
May 13, 2026
Merged

Move registration to async Producer pattern#143
steeevin88 merged 1 commit into
mainfrom
steven/fix-consumer-write

Conversation

@steeevin88
Copy link
Copy Markdown
Contributor

CONTEXT

  • RegisterForEvent handler was doing synchronous seat reservation via Redis and immediate DB acceptance inline
  • this meant the HTTP handler was directly coupled to capacity-checking and acceptance logic
  • moving to an async producer pattern decouples the handler from business logic and lets a consumer handle seat reservation + acceptance

CHANGES

  • replace inline TryTakeEventSeat / MarkRegistrationAccepted / ReleaseEventSeat flow with producer.PublishRegistration
  • status code changed from 201 Created to 202 Accepted to reflect async processing
  • error messages updated to match the new enqueue-based flow

@steeevin88 steeevin88 force-pushed the steven/fix-consumer-write branch from bcd19f3 to 31f11d9 Compare May 12, 2026 22:23
@steeevin88 steeevin88 marked this pull request as ready for review May 13, 2026 08:15
Copy link
Copy Markdown
Collaborator

@c1ndytat c1ndytat left a comment

Choose a reason for hiding this comment

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

scevents stamp time

@steeevin88 steeevin88 merged commit bd22c3b into main May 13, 2026
3 checks passed
@steeevin88 steeevin88 deleted the steven/fix-consumer-write branch May 13, 2026 08:19
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.

2 participants