-
Notifications
You must be signed in to change notification settings - Fork 134
Remove mention to Google Translator Toolkit #1026
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
Conversation
Reference a section in the polyglots handbook instead of an old Goole link.
ocean90
left a comment
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.
This should be moved to the GlotPress handbook instead. I just sent an invite for the site. I guess https://glotpress.blog/the-manual/glossaries/ is a good place for this.
|
I've created the appropriate page over at https://glotpress.blog/the-manual/glossaries/ |
|
@naokomc Could you update your PR with the new URL as well as address the CI failure? Thanks. |
|
@toolstack Thank you for the page on the manual. I've updated the URL. |
toolstack
left a comment
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.
+1
| <h2><?php _e( 'Import Glossary Entries', 'glotpress' ); ?></h2> | ||
| <p> | ||
| <?php printf( __( 'Use this form to bulk upload glossary entries. The entries should be stored in a CSV file, matching the custom glossary format from <a href="%s">Google Translator Toolkit</a>.', 'glotpress' ), 'https://support.google.com/translate/toolkit/answer/147854' ); ?><br/> | ||
| <?php printf( __( 'Use this form to bulk upload glossary entries. The entries should be stored in a CSV file with a custom glossary format. Read more on <a href="%s">Polyglots Handbook</a>.', 'glotpress' ), 'https://glotpress.blog/the-manual/glossaries/' ); ?><br/> |
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.
"Polyglots Handbook" should have been replaced with "GlotPress handbook"
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.
@ocean90 good point. This PR was closed - I just checked "Allow edits from maintainers." so that'll make it easier to update the text?
Sorry to take your time with this small change... 🙏
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.
I've created a new PR for this and will merge it.
Reference a section in the polyglots handbook instead of an old Goole link to fix #1021