Skip to content

Copy and CTA updates for home page - #6421

Open
tmccoy-cfa wants to merge 4 commits into
mainfrom
GYR1-1192-update-homepage-copy-in-english-and-spanish
Open

Copy and CTA updates for home page#6421
tmccoy-cfa wants to merge 4 commits into
mainfrom
GYR1-1192-update-homepage-copy-in-english-and-spanish

Conversation

@tmccoy-cfa

Copy link
Copy Markdown
Contributor

Link to Jira issue

Is PM acceptance required?

  • Yes - don't merge until Jira issue is accepted!

Reminder: merge main into this branch and get green tests before merging to main

What was done?

  • Updated service names
  • Changed CTA for in-person Assist
  • Updated footer copy
  • Removed Get Started CTA in the top right corner and replaced with “Sign In” CTA
  • Removed translation box for Spanish version

How to test?

  • Compare deployed demo site to current production site to review changes. Alternatively, screenshots are attached for review.

Screenshots (visual changes)

  • Before
image image image image
  • After
image image image image

@github-actions

Copy link
Copy Markdown

Heroku app: https://gyr-review-app-6421-505bd5606114.herokuapp.com/
View logs: heroku logs --app gyr-review-app-6421 (optionally add --tail)

Comment thread config/locales/es.yml
off_season_filing: No podemos aceptar nuevos clientes después de %{end_of_intake_date}. Si su declaración está en progreso, inicie sesión y envíe sus documentos antes de %{end_of_docs_date} para presentarla antes de la fecha límite.
footer:
service_description_html: GetYourRefund.org es un servicio sin ánimo de lucro creado por %{cfa_link} en colaboración con los sitios de Asistencia Voluntaria al Contribuyente (%{vita_link}) certificados por el IRS en todo el país.
service_description_html: GetYourRefund.org es un servicio sin ánimo de lucro creado por %{cfa_link}.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The part that says: "sin ánimo de lucro " should say "sin fines de lucro". I believe its a bad translation. We should talk to the person that submitted it 😅

<% unless locale == :es %>
<span><%= link_to_locale('es', 'Traducir esta página al español', { "data-track-click": "intake-language-changed-habla-espanol-section" }) %></span>
<% end %>
<% unless locale == :es %>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like this +1

Comment thread config/locales/es.yml
reveal_title: Información adicional
subtitle: Responda algunas preguntas y deje que un profesional de impuestos presente su declaración a nombre suyo
title: Declarar con ayuda
subtitle: Responde unas preguntas y un experto en impuestos presenta tu declaración

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here, it should say: "Contesta unas preguntas y un experto en impuestos presentará tu declaración"

responder and contestar both mean same "to answer", but Responder is more for actions and Contestar is more verbal, like: Answer the phone, a question, a message, etc

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I'll flag both of these to Ryan to see if we'd like to have updates here

@Noaj Noaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Confirmed by Ryan Hatch that we will stick with what's in the Figma file.
+1
🚢 it!

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