Skip to content

PIR: Add email fallback#8656

Open
landomen wants to merge 1 commit into
feature/domen/pir-email-flexibility-pixelsfrom
feature/domen/pir-email-flexibility-store-email-record
Open

PIR: Add email fallback#8656
landomen wants to merge 1 commit into
feature/domen/pir-email-flexibility-pixelsfrom
feature/domen/pir-email-flexibility-store-email-record

Conversation

@landomen
Copy link
Copy Markdown
Contributor

@landomen landomen commented May 21, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/task/1214973541222181?focus=true

Description

Adds email fallback for the email confirmation job record.

Steps to test this PR

See https://app.asana.com/1/137249556945/task/1214973541222183?focus=true

UI changes

No UI changes


Note

Medium Risk
Changes the email address persisted for email-confirmation job records, which can affect downstream polling/confirmation flows if the fallback is incorrect or empty.

Overview
Ensures email-confirmation job records are populated even when an ExtractedProfile has no email by adding an optional generatedEmail to BrokerRecordEmailConfirmationNeeded and falling back to it when marking an opt-out as waiting for confirmation.

Updates BrokerStepCompletedEventHandler to forward State.generatedEmailData.emailAddress into that state, and adds tests covering both the forwarding behavior and the fallback selection logic.

Reviewed by Cursor Bugbot for commit 438c3f9. Bugbot is set up for automated code reviews on this repo. Configure here.

@landomen landomen force-pushed the feature/domen/pir-email-flexibility-store-email-record branch from 8a7cd21 to a454d2b Compare May 21, 2026 17:48
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-store-email-record branch from a454d2b to 438c3f9 Compare May 22, 2026 06:46
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-pixels branch from 59f04e1 to 0a2a076 Compare May 22, 2026 06:46
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