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

Vaccine SMS reminder recipient's Phone number not validated #110

Open
tychus6 opened this issue Jan 27, 2021 · 6 comments
Open

Vaccine SMS reminder recipient's Phone number not validated #110

tychus6 opened this issue Jan 27, 2021 · 6 comments
Assignees
Labels

Comments

@tychus6
Copy link

tychus6 commented Jan 27, 2021

On the child registration page, upon selecting the person to receive the vaccine reminder SMS it should validate whether the person's(mother/father) phone number has been captured , currently there is no validation at all

This implementation should be inlign with data dictionary line 54
Implementation steps

  1. When the father has been selected to receive SMS reminders and the form is submitted, a validation should kick in to check whether the father's phone number has been entered, it should show an error message if the validation fails.
  2. When the mother has been selected to receive SMS reminders and the form is submitted, a validation should kick in to check whether the mother's phone number has been entered, it should show an error message if the validation fails.
  3. When both the father and mother have been selected to receive SMS reminders and the form is submitted, a validation should kick in to check whether the both the mother and father phone numbers have been entered, it should show an error message if the validation fails.
  4. When none has been selected to receive SMS reminders and the form is submitted, there should not be a validation rule applied
@tychus6 tychus6 added Affects functionality Issue that affects functionality of the app bug Something isn't working and removed Affects functionality Issue that affects functionality of the app labels Jan 27, 2021
@AnnieMungai

This comment has been minimized.

@AnnieMungai AnnieMungai added enhancement New feature or request and removed bug Something isn't working labels Jun 7, 2021
@tychus6

This comment has been minimized.

@AnnieMungai

This comment has been minimized.

@AnnieMungai
Copy link
Contributor

@hamza-vd kindly hold onto to this issue as @tychus6 looks at it further.

@tychus6
Copy link
Author

tychus6 commented Jun 28, 2021

@AnnieMungai i have updated the issue with a link to the data dictionary

@AnnieMungai
Copy link
Contributor

@tychus6 @hamza-vd looks good to go!

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

No branches or pull requests

3 participants