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

Updating children-ccap-start-date by updating the tests #63

Closed
wants to merge 3 commits into from

Conversation

enyia21
Copy link
Contributor

@enyia21 enyia21 commented Feb 21, 2024

#187090179

πŸ”— Pivotal Tracker ticket

✍️ Description

πŸ“· Design reference

βœ… Completion tasks

  • Added relevant tests
  • Meets acceptance criteria
  • Added testing instructions to ticket

@enyia21 enyia21 marked this pull request as draft February 22, 2024 23:19
@enyia21 enyia21 marked this pull request as ready for review February 24, 2024 00:40
@enyia21 enyia21 requested a review from sree-cfa February 24, 2024 00:50
@enyia21 enyia21 closed this Feb 24, 2024
@enyia21 enyia21 deleted the update-children-ccap-start-date branch February 24, 2024 00:56
@enyia21 enyia21 restored the update-children-ccap-start-date branch February 24, 2024 01:01
@enyia21 enyia21 deleted the update-children-ccap-start-date branch February 24, 2024 01:03
@enyia21 enyia21 restored the update-children-ccap-start-date branch February 24, 2024 01:11
@enyia21 enyia21 reopened this Feb 24, 2024
@sree-cfa sree-cfa force-pushed the update-children-ccap-start-date branch from df72bb1 to b49a1f4 Compare February 26, 2024 15:03

@Slf4j
@Component
public class ValidateChildrenCCAPStartDateWhenChildNotInChildCareWithProvider extends VerifyDate {
Copy link
Contributor

Choose a reason for hiding this comment

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

These two validate classes are pretty similar - I think we could just have 1 action that also checks for that "in care with provider" variable as well

Copy link
Contributor

Choose a reason for hiding this comment

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

actually if we combine this validation, we could combine the html pages as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I tried that. I had trouble pulling the childInCare variable in so I moved to separating them

@enyia21 enyia21 closed this Feb 26, 2024
@enyia21 enyia21 deleted the update-children-ccap-start-date branch February 26, 2024 21:07
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