-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update consent language on the calculator #190
base: main
Are you sure you want to change the base?
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -110,16 +110,22 @@ const renderEmailField = ( | |
/> | ||
<div className="mt-1 mx-3 text-color-text-secondary text-xsm leading-normal"> | ||
<slot name="email-helper"> | ||
{msg( | ||
'Get updates on incentives, rebates, and more from Rewiring America.', | ||
)}{' '} | ||
{msg('View our')}{' '} | ||
{msg('By sharing your email, you agree to our')}{' '} | ||
<a | ||
className="text-color-action-primary font-medium" | ||
href="https://rewiringamerica.org/terms-of-use" | ||
href="https://www.rewiringamerica.org/terms" | ||
target="_blank" | ||
> | ||
{msg('terms')} | ||
</a>{' '} | ||
{msg('and to receive updates from Rewiring America.')}{' '} | ||
{msg('We’ll store and protect your data in accordance with our')}{' '} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same, add string description here |
||
<a | ||
className="text-color-action-primary font-medium" | ||
href="https://content.rewiringamerica.org/view/privacy-policy.pdf" | ||
target="_blank" | ||
> | ||
{msg('privacy policy')} | ||
</a> | ||
. | ||
</slot> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -255,10 +255,6 @@ | |
<source>Reset</source> | ||
<target>Reiniciar</target> | ||
</trans-unit> | ||
<trans-unit id="s45d344440b59c309"> | ||
<source>Get updates on incentives, rebates, and more from Rewiring America.</source> | ||
<target>Manténgase al día sobre incentivos, reembolsos, y más de Rewiring America.</target> | ||
</trans-unit> | ||
<trans-unit id="s8fd029fdcc452602"> | ||
<source>Enter your ZIP code to select a utility.</source> | ||
<target>Ingrese su código postal para seleccionar una empresa de servicios eléctricos.</target> | ||
|
@@ -275,20 +271,11 @@ | |
<source>Email address</source> | ||
<target>Dirección de correo electrónico</target> | ||
</trans-unit> | ||
<trans-unit id="sb21ad862204e6186"> | ||
<source>View our</source> | ||
<note from="lit-localize">as in view our terms</note> | ||
<target>Ver nuestros</target> | ||
</trans-unit> | ||
<trans-unit id="s180a95484a16608c"> | ||
<source>terms</source> | ||
<note from="lit-localize">as in terms of service</note> | ||
<target>términos</target> | ||
</trans-unit> | ||
<trans-unit id="s64fbe9b97e397b12"> | ||
<source>We’re dedicated to safeguarding your privacy.</source> | ||
<target>Estamos dedicados a proteger su privacidad.</target> | ||
</trans-unit> | ||
<trans-unit id="s4455d8f660ceaa3b"> | ||
<source>10,000 Btuh</source> | ||
<target>10,000 Btuh</target> | ||
|
@@ -781,6 +768,22 @@ | |
<trans-unit id="s4cc06ab146dba194"> | ||
<source>Electric transportation</source> | ||
<target>Transporte eléctrico</target></trans-unit> | ||
<trans-unit id="sbb254e3763405cb1"> | ||
<source>By sharing your email, you agree to our</source> | ||
<target>Al compartir tu correo electrónico, aceptas nuestros</target> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is one risk of machine translation: we've decided in this UI to use the Spanish formal "you", but this is using the informal "you" --- "tu" should be "su", and possibly "aceptas" needs to be conjugated differently? (I don't actually know enough Spanish to do the translations from scratch myself, but I can recognize issues like this) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oof thank you for catching this. |
||
</trans-unit> | ||
<trans-unit id="s52ca6c8b262e0183"> | ||
<source>and to receive updates from Rewiring America.</source> | ||
<target>y recibir actualizaciones de Rewiring America.</target> | ||
</trans-unit> | ||
<trans-unit id="s0571211bbc7d4cce"> | ||
<source>We’ll store and protect your data in accordance with our</source> | ||
<target>Almacenaremos y protegeremos sus datos de acuerdo con nuestra</target> | ||
</trans-unit> | ||
<trans-unit id="sbc6953a36ced943f"> | ||
<source>privacy policy</source> | ||
<target>política de privacidad</target> | ||
</trans-unit> | ||
</body> | ||
</file> | ||
</xliff> |
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 one should have a description to tell translators what word is going to be next: