-
Notifications
You must be signed in to change notification settings - Fork 9
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
Aug 10th Feedback #29
Conversation
we got back a few changes from the appendix document today, and these are the ones I have been able to implement -- some are translations for non-translated sections
This is a custom survey for deployment in the usaid-laos-ev project -- it adapts a few of the questions to be more contextually appropriate
I tested this using the files uploaded to my repo, so once the new files come into main this path should work
we're waiting now on a decision about what language (since we don't currently support 2) the contact info should be in |
I tested the survey in the config locally, and accidentally committed the change
switching the admin_contact to Lao, had to choose just 1 language this time - hope to internationalize in the future
After changing the contact info to Lao, we've addressed the last of the issue identified in issue#943 and should now have an app that can work in Lao! |
@@ -67,7 +67,7 @@ <h3>Join your NREL OpenPATH-powered study!</h3> | |||
{name: "Testing Trip confirm + Time use surveys", joinurl: "nrelopenpath://join_study?label=stage-timeuse&source=github" }, | |||
{name: "Testing Program", joinurl: "nrelopenpath://join_study?label=stage-program&source=github" }, | |||
{name: "Testing Study", joinurl: "nrelopenpath://join_study?label=stage-study&source=github" }, | |||
{name: "UPRM NICR devapp", joinurl: "emission://join_study?label=uprm-nicr&source=github" }, | |||
{name: "UPRM NICR devapp", joinurl: "emission://join_study?label=uprm-nicr&source=github" } |
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.
nit: extraneous whitespace
There were some more translations provided for the appendix document today, so those updates will be included in this PR.
A translation was added for the contact section, but we don't currently have things set up to read the contact info from the translated part of the dynamic config file.