-
Notifications
You must be signed in to change notification settings - Fork 9
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
Refactor: eligibility forms #1657
Conversation
80723e3
to
3544819
Compare
Reviewing now |
name_max_length=None, | ||
sub_input_mode=None, | ||
sub_max_length=None, | ||
sub_pattern=None, |
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.
These deleted comments from the data file could become docstring or comments on this file to describe these:
# A regular expression used to validate the 'sub' API field before sending to this verifier
form_sub_pattern = models.TextField(null=True)
# Input mode can be "numeric", "tel", "search", etc. to override default "text" keyboard on mobile devices
form_input_mode = models.TextField(null=True)
# The maximum length accepted for the 'sub' API field before sending to this verifier
form_max_length = models.PositiveSmallIntegerField(null=True)
# The maximum length accepted for the 'name' API field before sending to this verifier
form_name_max_length = models.PositiveSmallIntegerField(null=True)
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.
Thanks for pasting here 🙏 I'll add those.
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.
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.
🎉🎉🎉
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.
Ahh, typo... one more rebase incoming
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.
👀 |
verifier instance can create an instance of its form class
3544819
to
c9a2dfe
Compare
moves Courtesy Card config into a form class specific to this agency card
c9a2dfe
to
0b6ea04
Compare
Merging based on @machikoyasuda's prior approval. Thanks both! |
Closes #1564
Gets us closer to #1525
What this PR does
EligibilityVerifier
model:form_blurb
,form_sub_label
, etc.form_class
that holds the name of a Python form class, adds a method to create an instance of this classMSTCourtesyCard
verification, moves all config there (out of migration)Testing
bin/init.sh