Billing Address field in AddDebitCardPage doesn't accept text

[hayata-suenaga]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

Break down in numbered steps

1. Go to Settings > Payments
2. Tap Add payment method > Debit card
3. Try to type in Billing Address field

Expected Result:

Inputted text should be displayed.

Actual Result:

Text doesn't appear when text is inputted.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

I tried to paste a text in the field instead of typing, but it doesn't work either. No workaround found.

Platforms:

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

• Android / native
• Android / Chrome
• iOS / native
• iOS / Safari
• MacOS / Chrome / Safari
• MacOS / Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation: https://expensify.slack.com/archives/C03TQ48KC/p1675793090919109

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~011c6f32902f1a74c9
• Upwork Job ID: 1623035041406087168
• Last Price Increase: 2023-02-07

[MelvinBot]

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[MelvinBot]

Bug0 Triage Checklist (Main S/O)

• This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[MelvinBot]

Triggered auto assignment to @grgia (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[aldo-expensify]

Removing @grgia because @hayata-suenaga is working on it and I think he already found the cause

[hayata-suenaga]

maxInputLength prop that is passed to TextInput inside AddressSearch component seems to the the root cause.

[mountiny]

Can you confirm what versions this is happening on, we just had a deploy so I want to know if this is in production now or the fresh new staging. In which case it would not be a deploy blocker.

[MelvinBot]

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

[MelvinBot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @parasharrajat (Internal)

[aldo-expensify]

Can you confirm what versions this is happening on, we just had a deploy so I want to know if this is in production now or the fresh new staging. In which case it would not be a deploy blocker.

@mountiny I tested in production and it is not there oh, you just deployed.. testing again

[aldo-expensify]

@mountiny I tested with version v1.2.65-2

[mountiny]

Ok yeah so that is production now so this does not have to be a deploy blocker! But I think we should still CP to Staging the fix so this gets out sooner

[hayata-suenaga]

This issue is being held until the QA can have a test debit card to implement the above testing steps.

[aldo-expensify]

Issue about testing debit card credential for QA: https://github.com/Expensify/Expensify/issues/226985

[hayata-suenaga]

I thought if the issue title has [HOLD], melvin-bot will be chill. does anyone know if we created a test card for QA team to use?

[mountiny]

No, HOLD does not stop melvin, only Reviewing does, but that should only be used for actual PR being in a review.

You can also change the priority of the issue if appropriate

[aldo-expensify]

I think Applause has the ability to test with cards now: https://github.com/Expensify/Expensify/issues/226985#issuecomment-1502170174

@hayata-suenaga we should be able to proceed with updating the test: #14918 (comment)

[mallenexpensify]

Dropped in #qa to ask about getting this tested
https://expensify.slack.com/archives/C9YU7BX5M/p1681317318570329

[hayata-suenaga]

@hayata-suenaga he should be able to proceed with updating the test: #14918 (comment)

Has the comment already been updated?

[trjExpensify]

👋 Noticed this in the +4 weeks old WAQ dashboard. There isn't a hold in the title anymore, dropping back to Daily.

[aldo-expensify]

@hayata-suenaga he should be able to proceed with updating the test: #14918 (comment)

Has the comment already been updated?

Sorry, which comment?

[hayata-suenaga]

@aldo-expensify nvm I was confused between updating testing steps in the original comment and actual Applause tests 😅

[mallenexpensify]

@mvtglobally can you test this?

Here's Maria's reply in #QA

we can add Debit card now. Only issue tester faced when set the newly added debit card as the default payment method - Default label is NOT present ( Step 10 - Add debit card section). Team logged two issue related default payment #16911 and #16914
Let me know if I need to log a separate issue for Debit Card.

[mvtglobally]

I will update in the morning with details

[mvtglobally]

@Beamanator @mallenexpensify
Doesn't look like this issue is repro any more https://user-images.githubusercontent.com/43995119/232060261-e274c8e9-3622-4dbc-a8be-6eb656a33bf6.MOV
https://user-images.githubusercontent.com/43995119/232060295-21b051c4-55c2-476d-845a-9179e5ce50d6.MOV
https://user-images.githubusercontent.com/43995119/232060336-21a3c7ea-f6b8-4029-a67e-ee1457daa6fa.MOV

[mallenexpensify]

Thanks @mvtglobally , I'm going to close then. Comment/reopen if needed.

[MelvinBot]

@mallenexpensify @aldo-expensify @hayata-suenaga Be sure to fill out the Contact List!