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

CCAP-466 #960

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

enyia21
Copy link
Contributor

@enyia21 enyia21 commented Dec 20, 2024

🔗 Jira ticket

CCAP-466

✍️ Description

  • Create registration-home-provider-dob-screen and the fields listed below:
    • providerIdentityCheckDateOfBirthDay
    • providerIdentityCheckDateOfBirthMonth
    • providerIdentityCheckDateOfBirthYear
  • Map inputs to the PROVIDER_DOB

📷 Design reference

Figma

✅ Completion tasks

  • Added relevant tests
  • Meets acceptance criteria

@enyia21 enyia21 temporarily deployed to il-gcc-ccap-466-create--xzklaz December 20, 2024 22:18 Inactive
<tr>
<td>calendar</td>
<td>
<svg th:fragment="calendar" width="101" height="75" viewBox="0 0 101 75" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link
Contributor

Choose a reason for hiding this comment

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

friendly reminder to always add: aria-hidden="true"

@enyia21 enyia21 temporarily deployed to il-gcc-ccap-466-create--xzklaz December 20, 2024 23:41 Inactive
@enyia21 enyia21 force-pushed the CCAP-466-Create-providerresponse-registration-home-provider-dob-screen branch from 1949f7d to 0c6cefd Compare December 23, 2024 19:16
@enyia21 enyia21 temporarily deployed to il-gcc-ccap-466-create--xzklaz December 23, 2024 19:16 Inactive
@enyia21 enyia21 temporarily deployed to il-gcc-ccap-466-create--xzklaz December 23, 2024 19:33 Inactive
@enyia21 enyia21 marked this pull request as ready for review December 23, 2024 19:37
@github-actions github-actions bot requested a review from spokenbird December 23, 2024 19:38
@enyia21 enyia21 requested a review from analoo December 23, 2024 20:51
@enyia21 enyia21 temporarily deployed to il-gcc-ccap-466-create--xzklaz December 23, 2024 20:52 Inactive
@@ -674,6 +674,8 @@ flow:
nextScreens:
- name: registration-home-provider-dob
registration-home-provider-dob:
crossFieldValidationAction: ValidateHomeProviderDateOfBirth
onPostAction: CreateProviderIdentityCheckDOB
condition: EnableProviderRegistration
Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2024-12-23 at 12 51 19 PM

I don't know if this is in the ticket but the design shows that only a subset of provider types should be able to see this screen

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll make the change

@enyia21 enyia21 force-pushed the CCAP-466-Create-providerresponse-registration-home-provider-dob-screen branch from 4bfc979 to 4d2b176 Compare December 24, 2024 01:29
@enyia21 enyia21 requested a review from analoo December 24, 2024 01:35
Copy link
Contributor

@spokenbird spokenbird left a comment

Choose a reason for hiding this comment

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

Looks good, works as expected in the flow!

@@ -178,6 +178,11 @@ void LicenseExemptInProviderHomeFlow() {

testPage.clickContinue();
// registration-home-provider-dob
assertThat(testPage.getTitle()).isEqualTo(getEnMessage("registration-home-provider-dob.title"));
Copy link
Contributor

Choose a reason for hiding this comment

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

This is great! It should skip in the other flows (Licensed home, Licensed group home, Licensed day care, licensed exempt day care)

Copy link
Contributor

@analoo analoo left a comment

Choose a reason for hiding this comment

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

non blocking comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants