Skip to content

feat(invitation): let a guest who is coming invite someone from the card - #238

Merged
mrvin100 merged 1 commit into
developfrom
jiku-285-invite-someone
Nov 3, 2026
Merged

mrvin100 merged 1 commit into
developfrom
jiku-285-invite-someone

Conversation

@mrvin100

@mrvin100 mrvin100 commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

JIKU-285, server side. Guests who say yes can pass the card on to a relative the host forgot, which turns every guest into a relay. Following decision D2, this is on by default when places are unlimited and off when the host set a capacity. The host can change it. The screen comes in the matching web PR.

Changes

  • invitation: adds OpenInvitation.relay, with migration V106__open_invitation_relay.sql.
    • A new open invitation starts with relay = remainingAttendance == null.
    • Existing invitations are set by the same rule, from event.max_capacity.
  • invitation: OpenInvitationSettingsRequest.relay sets the choice; null keeps it. The organizer view returns it.
  • invitation: PublicOpenInvitationView.relay is true only when relay is on and the card takes answers. A closed or full card never offers it.
  • invitation: CardSource gains RELAY("relay"), so relayed links count under their own line in the funnel (JIKU-282).
  • openapi/openapi.json is regenerated.

Behaviour a client will notice

Additive fields only.

Tests

  • OpenInvitationFlowTest:
    • on without a capacity, off with one;
    • the organizer turns it off;
    • a closed card never offers it, even when relay is on.
  • CardFunnelTest, ModularityTests and ktlintCheck pass.

Refs: JIKU-285

An open invitation gains relay. Following decision D2, it starts on
when the event has no capacity and off when the host set one; the
settings change it, and null keeps the choice. Existing invitations
follow the same rule. The public view says whether to offer "Invite
someone", and never on a card that takes no answers. A relayed link
counts under its own network, relay.

Refs: JIKU-285
@mrvin100
mrvin100 merged commit 8978148 into develop Nov 3, 2026
3 checks passed
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