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

[HOLD for payment 2021-12-28] Payment - Keyboard type is wrong for CVV at debit card page - Reported by: @thesahindia #6748

Closed
mvtglobally opened this issue Dec 14, 2021 · 19 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@mvtglobally
Copy link

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


Action Performed:

  1. Navigate to Profile > Settings
  2. Select Payments
  3. Click Add payment method
  4. Select Debit card
  5. Click on CVV CTA

Expected Result:

Numeric keyboard should open

Actual Result:

Letters keyboard opens

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS
  • Android
  • Mobile Web

Version Number: 1.1.20-1
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1639348013154400

View all open jobs on GitHub

@mvtglobally mvtglobally added AutoAssignerTriage Auto assign issues for triage to an available triage team member Engineering Daily KSv2 labels Dec 14, 2021
@MelvinBot
Copy link

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

@MelvinBot
Copy link

Triggered auto assignment to @puneetlath (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Dec 14, 2021
@MariaHCD
Copy link
Contributor

I'll let @puneetlath do the initial triage here!

@puneetlath
Copy link
Contributor

Confirmed this is indeed an issue. Over to you @MariaHCD!

@puneetlath puneetlath removed their assignment Dec 14, 2021
@mananjadhav
Copy link
Collaborator

mananjadhav commented Dec 14, 2021

This is a small fix:

Proposed solution: is to add keyboardType to the text input in AddDebitCardPage

<ExpensiTextInput
      label={this.props.translate('addDebitCardPage.cvv')}
      onChangeText={securityCode => this.clearErrorAndSetValue('securityCode', securityCode)}
      value={this.state.securityCode}
      errorText={this.getErrorText('securityCode')}
      keyboardType={CONST.KEYBOARD_TYPE.NUMBER_PAD} // Add this line
  />

@MariaHCD MariaHCD added the External Added to denote the issue can be worked on by a contributor label Dec 15, 2021
@MelvinBot
Copy link

Triggered auto assignment to @michaelhaxhiu (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@MariaHCD MariaHCD removed their assignment Dec 15, 2021
@michaelhaxhiu
Copy link
Contributor

michaelhaxhiu commented Dec 16, 2021

Upwork job post - https://www.upwork.com/jobs/~016c765e8ac277bc66

@thesahindia can you apply for the job so we can hire you for the reporting bonus?

First solution proposal for review will be @mananjadhav

@MelvinBot
Copy link

Triggered auto assignment to @parasharrajat (Exported)

@MelvinBot MelvinBot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 16, 2021
@MelvinBot
Copy link

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

@MelvinBot MelvinBot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 16, 2021
@MelvinBot
Copy link

📣 @mananjadhav You have been assigned to this job by @Julesssss!
Please apply to this job in Upwork 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 📖

@parasharrajat
Copy link
Member

All good here.
🎀 👀 🎀

@michaelhaxhiu
Copy link
Contributor

@thesahindia is hired for reporting bonus ✅, but still waiting for @mananjadhav to to apply for the job in Upwork.

@michaelhaxhiu
Copy link
Contributor

I'm re-applying the External label on this GH to trigger auto-assignment to a fellow CM, as I'm going OOO for the next 2 weeks. 🙏

@michaelhaxhiu michaelhaxhiu removed their assignment Dec 16, 2021
@michaelhaxhiu michaelhaxhiu added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Dec 16, 2021
@MelvinBot
Copy link

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@MelvinBot MelvinBot added Daily KSv2 and removed Weekly KSv2 labels Dec 16, 2021
@bfitzexpensify bfitzexpensify added Weekly KSv2 and removed Daily KSv2 labels Dec 16, 2021
@mananjadhav
Copy link
Collaborator

but still waiting for @mananjadhav to apply for the job in Upwork

@michaelhaxhiu @bfitzexpensify Applied and also PR submitted.

@Julesssss
Copy link
Contributor

Julesssss commented Dec 17, 2021

Merged, awaiting deployment.

@mananjadhav
Copy link
Collaborator

@michaelhaxhiu @bfitzexpensify Applied and also PR submitted.

@bfitzexpensify Upwork hires pending.

@botify botify added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Dec 21, 2021
@botify
Copy link

botify commented Dec 21, 2021

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.1.22-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2021-12-28. 🎊

@botify botify changed the title Payment - Keyboard type is wrong for CVV at debit card page - Reported by: @thesahindia [HOLD for payment 2021-12-28] Payment - Keyboard type is wrong for CVV at debit card page - Reported by: @thesahindia Dec 21, 2021
@laurenreidexpensify
Copy link
Contributor

@mananjadhav @thesahindia I've issued payment 👍🏽 Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests