Skip to content

Add RSVP Email Confirmation#879

Open
noahk004 wants to merge 5 commits into
mainfrom
feat/rsvp-conf-and-updated-portal-statuses
Open

Add RSVP Email Confirmation#879
noahk004 wants to merge 5 commits into
mainfrom
feat/rsvp-conf-and-updated-portal-statuses

Conversation

@noahk004
Copy link
Copy Markdown
Member

Changes

  • Create new function email_handler.send_rsvp_confirmation_email that sends an RSVP confirmation email on /rsvp
    • Added RSVP template

Important

RSVP Confirmation Email needs to be configured on SendGrid, then the appropriate template ID must be added.

@noahk004 noahk004 requested a review from NoNathan17 April 22, 2026 22:15
@noahk004 noahk004 linked an issue Apr 22, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Deploy preview for irvinehacks-site ready!

Name IrvineHacks Site
Preview Visit Preview
Commit ca3db95

@noahk004 noahk004 removed the request for review from NoNathan17 April 22, 2026 22:33
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[flake8] <821> reported by reviewdog 🐶
undefined name 'applicant'

discriminated unions don't work with FastAPI's Annotated Form()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[flake8] <821> reported by reviewdog 🐶
undefined name 'application_type'

discriminated unions don't work with FastAPI's Annotated Form()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [flake8] <303> reported by reviewdog 🐶
too many blank lines (2)

"tech_experienced_technologies",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [mypy] reported by reviewdog 🐶
Name "applicant" is not defined [name-defined]

discriminated unions don't work with FastAPI's Annotated Form()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [mypy] reported by reviewdog 🐶
Name "application_type" is not defined [name-defined]

discriminated unions don't work with FastAPI's Annotated Form()

"roles": [Role.VOLUNTEER],
"decision": None,
}
} No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[black-format] reported by reviewdog 🐶

Suggested change
}
}

"roles": [Role.VOLUNTEER],
"decision": None,
}
} No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ [flake8] <292> reported by reviewdog 🐶
no newline at end of file

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.

RSVP Confirmation and Updated Portal Statuses

2 participants