Skip to content
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

[$250] Bank account - SSN translated placeholder overlaps with data entered in the field #49807

Closed
1 of 6 tasks
lanitochka17 opened this issue Sep 26, 2024 · 21 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Sep 26, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.0.40-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): applausetester+vd_web092624@applause.expensifail.com
Issue reported by: Applause - Internal Team

Action Performed:

Pre-requisite: user must have created a workspace and have enabled Workflows. Also, user must have set the language to Spanish

  1. Go to Ajustes > Espacios de trabajo > Created Workspace > Flujos de trabajo
  2. Click on "Conectar cuenta bancaria"
  3. Select "Conectar con Plaid", use Regions bank and testing credentials
  4. Select Saving account and continue the flow until you reach the SSN page
  5. Enter any data on the SSN field

Expected Result:

SSN placeholder should not overlap with the data entered in the field

Actual Result:

SSN placeholder overlaps with the data entered in the field

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6616465_1727374059409.bandicam_2024-09-26_13-52-07-941.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021839606882609058645
  • Upwork Job ID: 1839606882609058645
  • Last Price Increase: 2024-09-27
  • Automatic offers:
    • Nodebrute | Contributor | 104161474
Issue OwnerCurrent Issue Owner: @Ollyws
@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 26, 2024
Copy link

melvin-bot bot commented Sep 26, 2024

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@Nodebrute
Copy link
Contributor

Nodebrute commented Sep 26, 2024

Edited by proposal-police: This proposal was edited at 2024-09-26 19:28:01 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

SSN translated placeholder overlaps with data entered in the field

What is the root cause of that problem?

The Spanish translation is too long which is why it overlaps. This issue will occur in all the places where the placeholder goes to the second line. We don’t see this elsewhere because we don’t have long translations as placeholders.

last4SSN: 'Últimos 4 dígitos de tu número de la seguridad social',

What changes do you think we should make in order to solve the problem?

We can use a small translation here

        last4SSN: 'Últimos 4 dígitos de tu SSN'',

Instead of using the full form 'social security number' in the Spanish translation, we can use 'SSN.' We should make sure that we are using the correct Spanish translation. The Spanish translation can be confirmed in PR. We should also check for places where we are using long texts as placeholders.

What alternative solutions did you explore? (Optional)

Or we can use

last4SSN: Últimos 4 del SSN.

@jais-nikhil
Copy link

Looks like some padding issue with placeholder. I can handle it with padding or placeholder font size.

Copy link

melvin-bot bot commented Sep 26, 2024

📣 @jais-nikhil! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@truph01
Copy link
Contributor

truph01 commented Sep 27, 2024

Edited by proposal-police: This proposal was edited at 2024-09-27 08:43:52 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

  • SSN placeholder overlaps with the data entered in the field

What is the root cause of that problem?

What changes do you think we should make in order to solve the problem?

  • Should pass numberOfLines={1} to the above.

  • Similar can be done with native file.

  • A flag prop can be introduced to ensure the above changes are applied only for SSN input cases.

  • Result:

image

What alternative solutions did you explore? (Optional)

@twisterdotcom twisterdotcom added the External Added to denote the issue can be worked on by a contributor label Sep 27, 2024
@melvin-bot melvin-bot bot changed the title Bank account - SSN translated placeholder overlaps with data entered in the field [$250] Bank account - SSN translated placeholder overlaps with data entered in the field Sep 27, 2024
Copy link

melvin-bot bot commented Sep 27, 2024

Job added to Upwork: https://www.upwork.com/jobs/~021839606882609058645

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 27, 2024
Copy link

melvin-bot bot commented Sep 27, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws (External)

@truph01
Copy link
Contributor

truph01 commented Sep 27, 2024

@twisterdotcom Can you confirm the expected behavior here? In case the input label is too long, should we:

  1. Use the smaller label.

  2. Just display one line as we did in other logics. If the text is overflow, display "..." at the end like below:

image

@twisterdotcom
Copy link
Contributor

We should use the elipses ... to shorten it.

@Nodebrute
Copy link
Contributor

@twisterdotcom The Spanish translation here is not the same as the English translation. We don't have long placeholders anywhere else. I think we should use the same translation as we have in English here. Also, such long placeholders look weird.
Screenshot 2024-09-27 at 6 28 22 PM

My suggestion is to use the same copy as we do in English.

@Nodebrute
Copy link
Contributor

@twisterdotcom Also, we use 'SSN' in other places too, where we have the correct and shorter translation. We can use the same translation here to fix this

ssnLast4: 'Últimos 4 dígitos de tu SSN',

@twisterdotcom
Copy link
Contributor

Ah, good point. Yes, absolutely. I think the acronym is fine to keep using. SSN specifically refers to the US Social Security Number - it's a proper noun in English, so let's keep using that.

@Ollyws
Copy link
Contributor

Ollyws commented Sep 27, 2024

If we disable overflow it it will read Últimos 4 dígitos de tu número de la seguridad... which doesn't really make sense.
I like @Nodebrute's solution as that's the only peice of Spanish copy where we use seguridad social where we would usually use SSN.
🎀👀🎀 C+ reviewed

Copy link

melvin-bot bot commented Sep 27, 2024

Triggered auto assignment to @deetergp, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@Nomanahmeda789
Copy link

How to join myself as a contributor for expensify?
email: nomanahmeda789@gmail.com
Upwork account: https://www.upwork.com/freelancers/nomanahmed7

Copy link

melvin-bot bot commented Sep 27, 2024

📣 @Nomanahmeda789! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 27, 2024
Copy link

melvin-bot bot commented Sep 27, 2024

📣 @Nodebrute 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@melvin-bot melvin-bot bot added Reviewing Has a PR in review and removed Daily KSv2 labels Sep 27, 2024
@melvin-bot melvin-bot bot added the Weekly KSv2 label Sep 27, 2024
@Nodebrute
Copy link
Contributor

@twisterdotcom The pr was deployed to production 2 weeks ago #49863 (comment). It is ready for payment.

@twisterdotcom
Copy link
Contributor

Payment Summary:

@Ollyws
Copy link
Contributor

Ollyws commented Oct 15, 2024

Requested in ND.

@garrettmknight
Copy link
Contributor

$250 approved for @Ollyws

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

9 participants