-
Notifications
You must be signed in to change notification settings - Fork 56
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
Adding Python to OData modal and changing text #743
Conversation
@@ -213,272 +200,3 @@ export default { | |||
} | |||
</i18n> | |||
|
|||
<!-- Autogenerated by destructure.js --> |
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.
Probably no need to delete this section even if things are broken for non-English locales, unless tests are failing. I'll update this section when I update translations before regression testing.
// The text of {python} is "Python". The text of {excel} is "Excel". | ||
// {powerBi}, {r}, {excel} and {python} are all links. | ||
"OData is a standard for transferring data between tools and services. Free and powerful analysis tools like {powerBi}, {excel}, {python}, and {r} can fetch data via OData for analysis.", | ||
"To connect to this form’s OData feed, select your tool and copy the link into it." |
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.
form → Form
"here": "here" | ||
} | ||
], | ||
"python": "To connect to Central from Python, we recommend {pyODK}. pyODK is the official Python client for Central and it makes common data analysis and workflow automation tasks simple.", |
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.
How about a translator comment here:
{pyODK} is a link. Its text is "pyODK".
} | ||
], | ||
"python": "To connect to Central from Python, we recommend {pyODK}. pyODK is the official Python client for Central and it makes common data analysis and workflow automation tasks simple.", | ||
"r": "To connect to Central from R, we recommend {ruODK}. ruODK is developed and supported by ODK community members.", |
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.
Probably a translator comment here as well
Closes #740. This will show up on the submissions page and the entities page.
What has been done to verify that this works as intended?
Tests and looking at it.
Why is this the best possible solution? Were any other approaches considered?
Pretty straightforward template and text changes following guidance in issue.
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
Adds python! Less text.
Does this change require updates to user documentation? If so, please file an issue here and include the link below.
No.
Before submitting this PR, please make sure you have:
npm run test
andnpm run lint
and confirmed all checks still pass OR confirm CircleCI build passes