Open tecnetico opened 6 years ago
Hey @tecnetico,
We love this idea! All the text for the widget is in this file. If you can do the translation, we'll gladly publish a Spanish version.
Done!
@tecnetico Just deployed this. The config options are documented in the README, but the gist is:
<script>
var RED_ALERT_OPTIONS = { language: 'es' };
</script>
<script src="https://redalert.battleforthenet.com/widget.js"></script>
will always show the Spanish widget.
And if no value is set for RED_ALERT_OPTIONS.language
and the viewer's browser is set to Spanish, the Spanish widget will show.
The code has been deployed at Tecnetico.com
Thank you for the opportunity to join the fight!
@tecnetico Awesome. Thanks for all your help translating this!
We tweeted out the setup instructions earlier: https://twitter.com/fightfortheftr/status/996042559091703808
Hi. We won in the Senate!
Do you need us to revise the copy in Spanish to reflect the new copy in English?
@tecnetico That would be fantastic! These two commits have the English changes:
https://github.com/fightforthefuture/redalert-widget/commit/6f71269c101a5de6cf03f0274bc894ec1a4e8c9c https://github.com/fightforthefuture/redalert-widget/commit/fd5fac49602ca9657fcbc4f19417b6db6ed3dcc3
I will proceed with the translation but I have a question.
On GitHub, the description of redalert-widget/src/strings/es.yml has "Add non-US CTA" as its description.
Why "non-US"?
On Wed, May 16, 2018 at 6:02 PM, Ken Mickles notifications@github.com wrote:
@tecnetico https://github.com/tecnetico That would be fantastic! These two commits have the English changes:
6f71269 https://github.com/fightforthefuture/redalert-widget/commit/6f71269c101a5de6cf03f0274bc894ec1a4e8c9c fd5fac4 https://github.com/fightforthefuture/redalert-widget/commit/fd5fac49602ca9657fcbc4f19417b6db6ed3dcc3
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/fightforthefuture/redalert-widget/issues/3#issuecomment-389680074, or mute the thread https://github.com/notifications/unsubscribe-auth/AMA2jZ0vF8f61qJR52deyzbF2_HeEYL-ks5tzKHtgaJpZM4T8JxD .
@tecnetico The description is the last Git commit that touched that file. "Add non-US CTA" was this commit: 317673984d1e050936ef29da56d87bbddfafa910. It adds code to show the "Not in the US? Do this instead." line to users who appear (based on IP address) to be outside of the US:
That line links to this petition, which is a way for people who don't live in the United States to show their support.
Also, I translated that line into Spanish via Google Translate, so it might be wrong: https://github.com/fightforthefuture/redalert-widget/blob/master/src/strings/es.yml#L10
We'd love to run this but we need it to be in Spanish.
We know this could be difficult to do, so we volunteer to translate the text to Spanish. It's just to pique the interest of our visitors. Once they click through, they'll be OK with proceeding in English.