-
Notifications
You must be signed in to change notification settings - Fork 2
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
mod 2: age range checks #422
Comments
Thanks! |
@joshid-ims oh yes! there should be a range check for calendar years in shiftwork. can you please copy and paste the english range check into the spanish? good catch. |
Will do. Thanks! |
Are you going to make the change in the document? I do not think I can edit the document on Box. I will add it in code. |
I added the range check to the Spanish M2 SHIFTWORK10 year text box. I also added the change log at the top. The formatting of the change log seems weird, but when I open it for editing, it is as expected. |
thanks @woodruffr Spanish M2 SHIFTWORK10 age check looks good. I know what you mean about the log, I think it's fine for now! |
Thanks for catching these! I made the updates in the English and Spanish qx docs |
Thanks for the quick turn-around. I have reviewed the changes and they are as expected in the English and Spanish M2 Questionnaire. |
@joshid-ims , I see you made the changes as expected already for these. This was just a documentation issue. |
I updated the year range checks in the module 2 English and Spanish questionnaire documents to account for the change in #424 |
@joshid-ims @huip-ims
Module 2 English age range checks are updated: https://app.box.com/s/cqoq9qexq3dstzdr5p7v1vbe0ekel9ct
Module 2 Spanish age range checks are updated: https://app.box.com/s/3dcpco42ne5nzf0pqeg3qimhzrcl54dr
The text was updated successfully, but these errors were encountered: