Skip to content

Auto-add users to group when registering for a group event#129

Merged
michaelchu merged 1 commit intomainfrom
auto-add-participant-to-group
Mar 9, 2026
Merged

Auto-add users to group when registering for a group event#129
michaelchu merged 1 commit intomainfrom
auto-add-participant-to-group

Conversation

@michaelchu
Copy link
Owner

Summary

  • The database trigger auto_add_participant_to_group was dropped in a prior migration and never replaced, so users registering for group events via invite links were not added as group members
  • Added application-level logic in createParticipant() to call groupService.addUserToGroup() via fireAndForget when the event has a group_id and the participant has a user_id
  • Correctly skips guest registrations and claimed spots (both have user_id = null)

Test plan

  • Added integration tests in participantService.autoGroup.test.ts covering:
    • Authenticated user registering for a group event → auto-added to group
    • Standalone event (no group_id) → no group add
    • Guest registration (no user_id) → no group add
    • Claimed spot (user_id null, claimed_by_user_id set) → no group add
    • Batch registration → auto-add works via delegation to createParticipant
  • All 23 participantService tests pass
  • Full test suite passes (pre-push hook)
  • Manual: register for a group event via invite link, verify user appears in group members

🤖 Generated with Claude Code

The database trigger `auto_add_participant_to_group` was dropped in a
prior migration and never replaced. This adds application-level logic
to automatically add users to the group when they register for a group
event via the invite link, using fire-and-forget so failures don't
block registration.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@michaelchu michaelchu temporarily deployed to auto-add-participant-to-group - Roster PR #129 March 9, 2026 19:12 — with Render Destroyed
@michaelchu michaelchu merged commit a761e27 into main Mar 9, 2026
2 checks passed
@michaelchu michaelchu deleted the auto-add-participant-to-group branch March 9, 2026 19:15
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