Skip to content
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.

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

Add Participant Box #2147

Merged
merged 7 commits into from
Nov 30, 2023
Merged

Add Participant Box #2147

merged 7 commits into from
Nov 30, 2023

Conversation

ahwnsals1234
Copy link
Contributor

@ahwnsals1234 ahwnsals1234 commented Nov 27, 2023

First draft of the Add Participant box. Has a text field for the name of the participants to add and a field to type the Gordon email of the participant.

@ahwnsals1234 ahwnsals1234 self-assigned this Nov 27, 2023
@ahwnsals1234
Copy link
Contributor Author

Not yet ready for a review, need a couple more updates to be made before being eligible for a code review.

@ahwnsals1234 ahwnsals1234 changed the title First draft of textField needed to add participant Add Participant Box Nov 27, 2023
Copy link
Contributor

@ArabellaJi ArabellaJi left a comment

Choose a reason for hiding this comment

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

Thank you for working on this! Just a few simple things and you should be good to go.

@andrew-wzj
Copy link
Contributor

Once you clean the code and conflicts mentioned in the HousingLottery.module.scss and index.jsx, your branch would be ready to merge. Thank you!

@ahwnsals1234 ahwnsals1234 marked this pull request as ready for review November 30, 2023 00:06
@ahwnsals1234 ahwnsals1234 dismissed ArabellaJi’s stale review November 30, 2023 00:32

Fixed all the changes requested.

@ahwnsals1234 ahwnsals1234 merged commit fdd5a97 into senior-project Nov 30, 2023
@ahwnsals1234 ahwnsals1234 deleted the hl-add-participant branch December 2, 2023 20:35
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.

3 participants