cal-itp / benefits

Transit benefits enrollment, minus the paperwork.
https://docs.calitp.org/benefits
GNU Affero General Public License v3.0
25 stars 9 forks source link

QA Spanish copy on test-benefits.calitp.org for `Improved UX for application error states` #2077

Closed indexing closed 1 month ago

indexing commented 2 months ago

Before deploying the updates to the application error states to production, we need a person to review the Spanish translation of the copy in the Benefits app for cogency.

Typically we use the general test plan for Spanish translation documented on the manual tests section of our testing page, but for this review it will be difficult or not possible to produce the error states covered by this milestone. Thus, we will need to review the copy in Figma.

Jot down any issues or notes during the review. Please use the comment feature in Figma to share your feedback. Please mention Andy Walker in each comment.

thekaveman commented 1 month ago

@lalver1 it probably makes more sense to do this copy QA locally instead of on test-benefits, since you'll have to edit the code slightly to induce the necessary errors.

See the instructions on the associated PRs

lalver1 commented 1 month ago

Thanks for the suggestion @thekaveman, I tested locally so I could induce the errors.

@indexing the Spanish copy looks good for these 3 associated PRs:

I left 6 comments on the Cal-ITP Benefits Application Copy spreadsheet, one for each string that was checked. I guess that once you look over the spreadsheet and accept the comments we can close this ticket.

indexing commented 1 month ago

@lalver1 I believe I accepted all comments in the spreadsheet, so we can close.