feat: Add RSVP functionality with email integration#1
Open
Aryan-mor wants to merge 6 commits intoomid:masterfrom
Open
feat: Add RSVP functionality with email integration#1Aryan-mor wants to merge 6 commits intoomid:masterfrom
Aryan-mor wants to merge 6 commits intoomid:masterfrom
Conversation
Aryan-mor
commented
Sep 21, 2025
Aryan-mor
commented
- Added optional organizer email field to event creation form
- Implemented RSVP component with Yes/No/Maybe buttons
- Created modal for collecting user name before sending RSVP email
- Integrated mailto functionality to send RSVP responses to organizer
- Added email validation for organizer email field
- Separated RSVP functionality into dedicated component for better code organization
- Updated event form types and URL parameter handling to support creator email
- Added optional organizer email field to event creation form - Implemented RSVP component with Yes/No/Maybe buttons - Created modal for collecting user name before sending RSVP email - Integrated mailto functionality to send RSVP responses to organizer - Added email validation for organizer email field - Separated RSVP functionality into dedicated component for better code organization - Updated event form types and URL parameter handling to support creator email
Owner
|
@Aryan-mor Could you please fix the conflict? |
# Conflicts: # src/App.tsx
omid
requested changes
Sep 26, 2025
src/RSVP.tsx
Outdated
| </div> | ||
|
|
||
| <div className="flex gap-3"> | ||
| <Button |
Owner
There was a problem hiding this comment.
The form is very small, no need for cancel button, I'd say.
And click outside should also cancel!
|
|
||
| Response: ${responseText} | ||
|
|
||
| Event Details: |
|
|
||
| Event Details: | ||
| ${event.description ? `Description: ${event.description}` : ''} | ||
| ${event.location ? `Location: ${event.location}` : ''} |
Owner
There was a problem hiding this comment.
Here, each line prints an empty line even if they are not there!
| ${event.description ? `Description: ${event.description}` : ''} | ||
| ${event.location ? `Location: ${event.location}` : ''} | ||
| ${event.sDate ? `Date: ${event.sDate}` : ''} | ||
|
|
Owner
There was a problem hiding this comment.
It can be good to add the link to the event somewhere in the email.
| ${event.sDate ? `Date: ${event.sDate}` : ''} | ||
|
|
||
| Best regards, | ||
| ${name}`; |
Owner
There was a problem hiding this comment.
Put the poor calf somewhere at the bottom:
Something like this is enough:
❤️ Calf 🐮📅
omid
reviewed
Sep 26, 2025
| initiallyOpen={!!form.creatorEmail} | ||
| title={ | ||
| <span className="text-sm"> | ||
| Organizer email (optional) |
Owner
There was a problem hiding this comment.
Or... we can keep this CollapsibleSection and rename it to RSVP.
And put all RSVP related stuff here.
- An email address
- OR a link to an external page and open it in a new tab
- OR an iframe in a modal
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.