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

Implement Applicants dob with gov_uk_date_fields #23

Merged
merged 5 commits into from
Nov 6, 2017

Conversation

zheileman
Copy link
Member

Individual commits to make it easier to understand.

Still some polishing is needed as the markup the gem produce is not updated to what it should produce and thus the hint is a bit our of place, and the errors are not shown in the individual fields, only in the summary.

Also, no partial validation support with this gem (entering only month and year, and leaving blank the day, for example).

In order to have the ability of calling callbacks, which is needed by
the `gov_uk_date_fields` gem.
Also controller and view. As we want this field to behave as a
`gov_uk_date`, we mark it as such with `acts_as_gov_uk_date :dob`
For the fields generated by the gem to be retrieved in the controller,
those need to be whitelisted. These fields are (in the case of `dob`):
dob_dd, dob_mm and dob_yyyy.

As there could be more than a gov_uk_date field per form, the method
will iterate through all the dates and build all the attribute names.
Copy link
Contributor

@digitalronin digitalronin left a comment

Choose a reason for hiding this comment

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

Please raise a ticket in Trello for fixing the markup, later.

@zheileman zheileman merged commit 142a1e7 into master Nov 6, 2017
@zheileman zheileman deleted the date-form-element branch November 6, 2017 09:48
WillTaylor22 added a commit that referenced this pull request Oct 20, 2022
Update dependency mimemagic to ~> 0.4.0
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