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

PD Type Translations #1404

Merged
merged 20 commits into from
Sep 15, 2023
Merged

PD Type Translations #1404

merged 20 commits into from
Sep 15, 2023

Conversation

JVickery-TBS
Copy link
Contributor

feat(i18n): translations for PD table yaml files;

  • Included strings and translations from the recombinant table yaml files.

- Included strings and translations from the recombinant table yaml files.
@wardi
Copy link
Member

wardi commented Sep 6, 2023

If you want to get fancy and reduce some duplication you could extract these strings from the yaml files with a custom babel message extractor

- Moved sql error messages to schema yaml files.
- Added shortname for the PD types that need it.
- Added pd shortnames to use instead of string concatenation.
- Added custom message extractor to get strings from the Recombinant schemas and into the translation interfece.
- Removed PD strings from `string.py`.
- Added missed trigger string for ati.
- Added line numbers for custom message extractor for pd yamls.
- Updated catalogs with new message extractions.
@JVickery-TBS
Copy link
Contributor Author

depends on open-data/ckanext-recombinant#116

@JVickery-TBS
Copy link
Contributor Author

@wardi custom message extractor is done now.

ckanext/canada/extract.py Outdated Show resolved Hide resolved
- Added custom safe loader class to record line numbers for string tags.
ckanext/canada/extract.py Outdated Show resolved Hide resolved
- Simple loop of recombinant recortds in PD message extractor.
- Config file for CircleCI.
- Updated readme with new translation stuffs.
- Added circleci badge.
- Added Slack notifications.
@JVickery-TBS
Copy link
Contributor Author

Trying to get our test moved to circleci. (https://github.com/open-data/ckanext-canada/tree/circleci)

Ran into a roadblock and had to open up a support ticket with circleci. But hopefully that can get solved and we can figure out why these string triggers are failing.

- Removed circleci changes as that is now in a different branch.
- Added back normal tests.
- Readme update.
- Reverted debugging.
- Debugging for some of the pd contracts tests.
- YAML pip strings not supported for trigger strings.
- Removed debug logging from contracts tests.
- Added comments to pd schema yamls.
@JVickery-TBS
Copy link
Contributor Author

@wardi okay cool, so with that fix to Recombinant and some minor fixes in the pd tests and yaml files here the tests are all working now, yay!

Are you able to take a final look at this to confirm it is all good to go?

Copy link
Member

@wardi wardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are the # yaml | not supported for a linter or something?

@JVickery-TBS
Copy link
Contributor Author

@wardi haha no, they are just for some inline comments for us :D

@JVickery-TBS JVickery-TBS merged commit 27ef292 into canada-v2.9 Sep 15, 2023
2 checks passed
@JVickery-TBS JVickery-TBS deleted the i18n/pd-types branch September 15, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants