Skip to content

fix(visitor_mailer): invite each visitor on a group booking once (PPT-2375) #634

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
camreeves merged 2 commits into from
Sep 30, 2026
Merged

fix(visitor_mailer): invite each visitor on a group booking once (PPT-2375) #634

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ab96692
fix(visitor_mailer): invite each visitor on a group booking once
camreeves Sep 28, 2026
b7f14c1
test(visitor_mailer): keep the spec under the runner's six minute limit
camreeves Sep 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions drivers/place/visitor_mailer.cr
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -582,6 +582,13 @@ class Place::VisitorMailer < PlaceOS::Driver

booking_type = booking["booking_type"].as_s
template = @group_event_template if booking_type == "group-event"

# a group container lists every visitor, but each visitor is invited from
# their own child booking (its id is what the QR code and check-in use)
if booking_type == "group" && booking.dig?("extension_data", "group_members")
logger.debug { "skipping the #{template} invite to #{guest_details.attendee_email} from group container #{guest_details.booking_id}, their own booking sends it" }
return
end
in GuestNotification
# should never get here
return
Expand Down
Loading
Loading