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 ability accessibility and photo questions #493

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

ericli18
Copy link
Collaborator

This should add the requirements to the form, admin dashboard, and internal database models

2024-11-19T19:47:46,103669274-06:00
2024-11-19T20:09:11,363062136-06:00

@ericli18 ericli18 requested a review from NitroGuy10 November 20, 2024 02:14
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just let the autoformatter handle this file, sorry. I'll highlight all relevant lines

@@ -242,6 +242,13 @@ def __init__(self, *args, **kwargs):
}

super().__init__(*args, **kwargs)

photo_agreement = "Do you grant permission for TAMUhack to use your name, likeness, voice, and any photographs, video recordings, or audio recordings taken during the event 'TAMUhack 2025' for promotional and media purposes, including but not limited to publications, websites, social media, and press releases?"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

relevant

@@ -312,8 +316,10 @@ class Meta:
"agree_to_coc": forms.CheckboxInput,
"agree_to_mlh_stuff": forms.CheckboxInput,
"signup_to_mlh_newsletter": forms.CheckboxInput,
"agree_to_photos": forms.CheckboxInput,
"accessibility_requirements": forms.CheckboxInput,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

relevant

@@ -358,6 +364,8 @@ class Meta:
"emergency_contact_phone",
"emergency_contact_email",
"notes",
"agree_to_photos",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

relevant

@ericli18
Copy link
Collaborator Author

Sorry for comment spam, let me know if you want me to change the order of things

@NitroGuy10
Copy link
Contributor

Good stuff! Thanks, Eric!

@NitroGuy10 NitroGuy10 merged commit c82382e into main Nov 22, 2024
2 of 6 checks passed
@NitroGuy10 NitroGuy10 deleted the photos-and-accessibility branch November 22, 2024 14:13
@NitroGuy10
Copy link
Contributor

Last thing, could you migrate the prod database to have these new changes?

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.

2 participants