-
Notifications
You must be signed in to change notification settings - Fork 22
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
CRVS bug fixes #2067
CRVS bug fixes #2067
Conversation
…, Remove Family Member, Out of Area Death Registration completed.
…tion, Birth Certification, Out Area Child registration implemented and tested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See review comments
opensrp-chw/src/crvs/assets/json.form/birth_certification_changed.json
Outdated
Show resolved
Hide resolved
"type": "edit_text", | ||
"v_regex": { | ||
"value": "([0-9]{1,4}\\/[0-9]{1,4})", | ||
"err": "Number must take the format of ##/####" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the error text/string to the relevant properties files to support translation
"type": "edit_text", | ||
"v_max_length": { | ||
"value": "9", | ||
"err": "Characters must be less than or equal to 9" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the error texts to the relevant properties files
"type": "edit_text", | ||
"v_regex": { | ||
"value": "([0-9]{1,4}\\/[0-9]{1,4})", | ||
"err": "Number must take the format of ##/####" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the text to the relevant properties files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relevance is added
family_register.step2.highest_edu_level.values[2]=Primary | ||
family_register.step2.dob.v_required.err=Please enter the date of birth | ||
family_register.step1.fam_source_income.values[1]=Agriculture, hunting and fishing | ||
family_register.step1.fam_add_province.values[0]=Bas Uele Province |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since these are added to the en properties file, they should also be added to the fr properties file as well
@@ -164,7 +164,6 @@ | |||
"key": "birth_cert_num", | |||
"openmrs_entity_parent": "", | |||
"openmrs_entity": "concept", | |||
"openmrs_entity_id": "162052AAAAAAAAAAAAAAAAAAAAAAAAAAAAAA", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why ws this entity_id removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relevance is added
# Conflicts: # opensrp-chw/build.gradle
Fixed french translations for out_of_area_child_enrollment
#2047
#2048