-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add decouple language packs #209
base: main
Are you sure you want to change the base?
Conversation
#178) Moved translations into separate repo and using git updater code to load translations off github repo translations. --------- Signed-off-by: Andy Fragen <[email protected]>
* update mixed case textdomain to lowercase of aspireupdate references * rename `AspireUpdate` repository to `aspireupdate` * change text domain header * add composer script to more easily create POT file * requires WP-CLI installed locally Signed-off-by: Andy Fragen <[email protected]>
* update afragen/translations-updater with error caching/logging * fix return of false due to validation failure * return WP_Error instead of false * better error handling * updates to afragen/translations-updater * composer update * update error log messaging * update library
You should be able to use Git Updater to switch branches for testing. When you're done testing simply switch branches back to Translation updates should occur just as they do for a normal WP installation for translations from .org. You will see a Translations button in Please leave testing comments here or in #208 |
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.
Looks good to me! Would be good to get at least one other review and test on this.
Pull Request
Merge branch
translations
intomain
.Changes to allow for decoupled language pack updates allowing for plugin release to be separated from translation releases.
Why did it change?
Allows for different release workflows for the plugin and the translations.
Did you fix any specific issues?
Fixes #208
CERTIFICATION
By opening this pull request, I do agree to abide by the Code of Conduct and be bound by the terms of the Contribution Guidelines in effect on the date and time of my contribution as proven by the revision information in GitHub.