-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: application pydantic models #76
Conversation
Deploy preview for irvinehacks-site-2024-sanity ready!
|
Deploy preview for irvinehacks-site-2024 ready!
|
3b0fc57
to
426d26b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can go with these for now and then figure out if we need to adjust None
vs empty string when actually connecting the application form submission.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pull requests adds the Pydantic models related to application data. A few changes were made to the code:
max_length
argument of theField
instances.utils.py
, two parameters/attributes were deprecated/no longer existing, so I updated them to their respective current ones.