GSA-TTS / usagov-benefits-eligibility

Benefits eligibility estimator tool for USAGov.
22 stars 5 forks source link

New Data Field (Date) #164

Closed r-bartlett-gsa closed 2 years ago

r-bartlett-gsa commented 2 years ago

As a BEARS site user, in order to get an accurate list of benefits which I might be eligible for based on my information, I would like to provide date information for benefits that have date as one of the eligibility requirements (example, date of birth or date of death).

Acceptance Criteria:

Date requirement calculation considerations:

Definition of Done:

More details about each item can be find in BEARS P4 DoD google doc

QA checklist

r-bartlett-gsa commented 2 years ago

@lgarceau768 Overall the date works great, so good job!! And I only have a few feedback items for you.

And here's the federalist link for testing: https://federalist-edd11e6f-8be2-4dc2-a85e-1782e0bcb08e.app.cloud.gov/preview/gsa/usagov-benefits-eligibility/date-criteria-test/death-and-burial/


r-bartlett-gsa commented 2 years ago

@lgarceau768 Just want to make sure that you aware that this feedback item does not need to be addressed at the moment, since the functionality to interact with criteria within the benefit card might go away:

This comment is just for the record, but does not need to be addressed at the moment, because we might want to remove input fields from the benefit cards in the near future. When the date is entered in the left rail, it is not copied over in the benefit card; and when the date is entered in the benefit card, it is not copied over to the left rail.

lgarceau768 commented 2 years ago

@r-bartlett-gsa I have addressed the bugs laid out above, could you please confirm that they are no longer present?

r-bartlett-gsa commented 2 years ago

@lgarceau768 This still seems to be an issue:

  • In the benefit card, when the eligibility is validated, only the label should be highlighted (red or green) but not the input field. https://www.screencast.com/t/bC0rgCF7eL
  • And let's make the label bold, so it matches the styling of other criteria types.

And I also found a new issue:

r-bartlett-gsa commented 2 years ago

@lgarceau768 In addition to the above feedback, I think there should also be some sort of date validation and error messaging for the dates that are invalid, for example the user should not be able to enter future dates (https://www.screencast.com/t/i1kjaEUoM9) or dates that don't exists (https://www.screencast.com/t/1IQDsm5ZM)

lgarceau768 commented 2 years ago

@r-bartlett-gsa All the changes have been made and the virtual criteria issue, is partially a problem on my end with something in the checking to see if the user has inputted a value before checking flow (which is now fixed). But mainly the issue was that in fema-covid-19-funeral-assistance.md the acceptableValue for the deceased_date_of_funeral was set to true and not a valid date.

r-bartlett-gsa commented 2 years ago

Thank you @lgarceau768 and apologies for the oversight with the FEMA benefit content.

And I also should have been more clear about this. The date label should be bold on the left-hand rail: https://www.screencast.com/t/kzePX7Sn

  • And let's make the label bold, so it matches the styling of other criteria types.
r-bartlett-gsa commented 2 years ago

An update on the following feedback item:

@lgarceau768 In addition to the above feedback, I think there should also be some sort of date validation and error messaging for the dates that are invalid, for example the user should not be able to enter future dates (https://www.screencast.com/t/i1kjaEUoM9) or dates that don't exists (https://www.screencast.com/t/1IQDsm5ZM)

When an invalid date is entered, an error message is shown to indicate that, and the system does not check if the eligibility is met for the benefits with that date criteria.

r-bartlett-gsa commented 2 years ago

@lgarceau768 Let's change the error message to "Please enter a valid date."

@fpigeonjr What would be your recommendation for date error message presentation based on USWDS? https://designsystem.digital.gov/components/text-input/

lgarceau768 commented 2 years ago

An update on the following feedback item:

@lgarceau768 In addition to the above feedback, I think there should also be some sort of date validation and error messaging for the dates that are invalid, for example the user should not be able to enter future dates (https://www.screencast.com/t/i1kjaEUoM9) or dates that don't exists (https://www.screencast.com/t/1IQDsm5ZM)

When an invalid date is entered, an error message is shown to indicate that, and the system does not check if the eligibility is met for the benefits with that date criteria.

Yes with the latest fix they will not check for eligibility.

r-bartlett-gsa commented 2 years ago

@lgarceau768 Just documenting the issue that still needs to be resolved. If a valid date is entered and the benefits are estimated, but then the date is changed to invalid, the previous valid date still persists in URL and benefits estimation is not updated. See: https://www.screencast.com/t/5mNlWh32bNn

r-bartlett-gsa commented 2 years ago

@lgarceau768 When an invalid date is entered, the entry remains in the input field and an error message is displayed, eligibility is not validated. This is correct and expected behavior.

When an invalid date is entered, after a valid date was entered, currently the entry is removed from the input fields and an error message is displayed. The invalid date needs to remain in the field until it's cleared or overwritten with something else.

https://www.screencast.com/t/fUqhpyOixvS

lgarceau768 commented 2 years ago

@lgarceau768 When an invalid date is entered, the entry remains in the input field and an error message is displayed, eligibility is not validated. This is correct and expected behavior.

When an invalid date is entered, after a valid date was entered, currently the entry is removed from the input fields and an error message is displayed. The invalid date needs to remain in the field until it's cleared or overwritten with something else.

https://www.screencast.com/t/fUqhpyOixvS

Fixed in PR #394