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

IOS - App is freezing after tap on "Make default payment method" #7869

Closed
kbecciv opened this issue Feb 22, 2022 · 11 comments
Closed

IOS - App is freezing after tap on "Make default payment method" #7869

kbecciv opened this issue Feb 22, 2022 · 11 comments
Assignees
Labels
Engineering Monthly KSv2 Reviewing Has a PR in review

Comments

@kbecciv
Copy link

kbecciv commented Feb 22, 2022

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. Launch App and login with expensifail account
  2. Go to the Payments page.
  3. Click any existing payment method to set it to default
  4. Click set default

Expected Result:

Payment should be set as default payment method

Actual Result:

App is freezing after tap on "Make default payment method"

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS

Version Number: 1.1.39.2

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5465487_RPReplay_Final1645571874.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause

Slack conversation:

Issue was found when execute the PR: #7782

View all open jobs on GitHub

@MelvinBot
Copy link

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

@TomatoToaster
Copy link
Contributor

So this looks like it had to do with the password input not rendering correctly after selecting the payment method
image

@parasharrajat your PR fixed this perfectly so I'm going to add this to the list of fixed issues in your PR and close this issue.

@mountiny
Copy link
Contributor

@TomatoToaster The QA seems to reproduce this issue on production and staging still. Relevant comments here and here.

@mountiny mountiny reopened this Mar 14, 2022
@mountiny mountiny self-assigned this Mar 14, 2022
@mountiny
Copy link
Contributor

not overdue

@MelvinBot MelvinBot removed the Overdue label Mar 14, 2022
@mountiny mountiny removed their assignment Mar 14, 2022
@parasharrajat
Copy link
Member

parasharrajat commented Mar 14, 2022

Wrong issue. Sorry.

@TomatoToaster
Copy link
Contributor

As far as I can tell, this seems to be an issue where the passover prompt doesn't end up showing up after being clicked here:

<TouchableOpacity
onPress={() => {
this.setState({
shouldShowPasswordPrompt: true,
shouldShowDefaultDeleteMenu: false,
passwordButtonText: this.props.translate('paymentsPage.setDefaultConfirmation'),
});
}}
style={[styles.button, styles.alignSelfCenter, styles.w100]}

There must be another issue causing this if this is only happening on iOS but I'm not sure why this is happening. I think this can be externally solved.

@TomatoToaster TomatoToaster removed their assignment Mar 14, 2022
@TomatoToaster TomatoToaster added the External Added to denote the issue can be worked on by a contributor label Mar 14, 2022
@MelvinBot
Copy link

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

@parasharrajat
Copy link
Member

parasharrajat commented Mar 14, 2022

This is a duplicate of #7768. The same solution will fix both.

@mountiny
Copy link
Contributor

mountiny commented Mar 14, 2022

I thought they are very similar, I will assign this to myself then, but I wont close it yet and wait to see if this wont be reproducible on iOS.

@jboniface sorry for the ping.

@mountiny mountiny assigned mountiny and unassigned jboniface Mar 14, 2022
@mountiny mountiny added Weekly KSv2 Reviewing Has a PR in review and removed External Added to denote the issue can be worked on by a contributor Daily KSv2 labels Mar 14, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 6, 2022

This issue has not been updated in over 15 days. @mountiny eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@melvin-bot melvin-bot bot added the Monthly KSv2 label Apr 6, 2022
@mountiny
Copy link
Contributor

mountiny commented Apr 6, 2022

Closing as the linked issue seems to be solved.

@mountiny mountiny closed this as completed Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Monthly KSv2 Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

6 participants