-
Notifications
You must be signed in to change notification settings - Fork 4
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
Handles AL custom birthdate field #217
Conversation
See tests at https://github.com/SuffolkLITLab/docassemble-MA209AProtectiveOrder/runs/2492735082?check_suite_focus=true. They will fail, but they should get to at least the question id 'children-user-is-parent-of'. Don't worry about the messages there about multiple items found on the page (for now).
// Handle setting values for da custom datatypes | ||
// E.g. `da-field-container-datatype-BirthDate` | ||
// TODO: Make this more easily extensible | ||
setCustomDatatype: { |
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.
How can we make setCustomDatatype
more easily extensible?
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.
(at some point in the future)
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.
I think this is a good start. This seems extensible, although the exact particulars will depend on what other custom fields we come across.
Tangentially: That test actually got through way more question ids than I can get locally. Wat? I need it to pass locally too to keep developing. Any ideas are welcome... |
See tests at https://github.com/SuffolkLITLab/docassemble-MA209AProtectiveOrder/runs/2492735082?check_suite_focus=true. They will fail, but they should get to at least the question id
children-user-is-parent-of
. Don't worry about the messages there about multiple items found on the page (for now).Custom fields will sometimes have to be handled separately. This one replaces one input date field with three other fields to make it easier to input birthdates.
Example html of the fields in their question: