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] Android - App crashed when try to split bill for group. #10826

Closed
kbecciv opened this issue Sep 5, 2022 · 20 comments
Closed

[$250] Android - App crashed when try to split bill for group. #10826

kbecciv opened this issue Sep 5, 2022 · 20 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Needs Reproduction Reproducible steps needed Weekly KSv2

Comments

@kbecciv
Copy link

kbecciv commented Sep 5, 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. Open App and login with HT account
  2. Go to any group
  3. Tap on [+] and select split bill
  4. Tap on select and deselect the participants of the IOU (maybe 2 times)

Expected Result:

App should be stay on page without crash.

Actual Result:

App crashed.

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Android

Version Number: 1.1.97.2

Reproducible in staging?: Yes

Reproducible in production?: n/a

Email or phone of affected tester (no customers): applausetester+fish@applause.expensifail.com/Qwerty123

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

Notes/Photos/Videos: Any additional supporting documentation

Bug5719026_20220905_151325.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added the DeployBlockerCash This issue or pull request should block deployment label Sep 5, 2022
@OSBotify
Copy link
Contributor

OSBotify commented Sep 5, 2022

👋 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.

@melvin-bot
Copy link

melvin-bot bot commented Sep 5, 2022

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

@luacmartins
Copy link
Contributor

I can't reproduce this. @kbecciv can you check that the issue is consistently reproducible?

@Julesssss Julesssss self-assigned this Sep 6, 2022
@Julesssss
Copy link
Contributor

Also unable to reproduce this with a few different groups of varing sizes.

@Julesssss
Copy link
Contributor

Open App and login with HT account

@kbecciv would you mind explaining what you mean by a HT account?

@luacmartins
Copy link
Contributor

@Julesssss I think that means high traffic

@kbecciv
Copy link
Author

kbecciv commented Sep 6, 2022

@Julesssss Yes, it means high traffic account.

@Julesssss
Copy link
Contributor

Thanks. Will ask for someone with a HT account to verify on staging.

@kbecciv
Copy link
Author

kbecciv commented Sep 6, 2022

@Julesssss I checked with team this morning, and issue is still reproduceable in tester's high traffic account after clear cookies and cache.
Screenshot_20220906-082825 (1)

@Julesssss
Copy link
Contributor

Asked in Slack for someone to verify. Will return to this once I resolve the Apple cert issue

@Julesssss
Copy link
Contributor

This is inconsistently reproducible and only for Android users with high traffic accounts.

So I'm removing the deploy blocker label.

@Julesssss Julesssss added Daily KSv2 Hourly KSv2 External Added to denote the issue can be worked on by a contributor and removed Hourly KSv2 DeployBlockerCash This issue or pull request should block deployment Daily KSv2 labels Sep 6, 2022
@Julesssss Julesssss removed their assignment Sep 6, 2022
@melvin-bot
Copy link

melvin-bot bot commented Sep 6, 2022

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

@luacmartins luacmartins removed their assignment Sep 7, 2022
@kadiealexander
Copy link
Contributor

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Sep 7, 2022
@melvin-bot
Copy link

melvin-bot bot commented Sep 7, 2022

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

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

melvin-bot bot commented Sep 7, 2022

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

@melvin-bot melvin-bot bot changed the title Android - App crashed when try to split bill for group. [$250] Android - App crashed when try to split bill for group. Sep 7, 2022
@kadiealexander kadiealexander removed their assignment Sep 12, 2022
@kadiealexander kadiealexander 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 Sep 12, 2022
@kadiealexander
Copy link
Contributor

Reassigning as I'm ooo for the next 9 days.

@melvin-bot
Copy link

melvin-bot bot commented Sep 12, 2022

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Sep 12, 2022
@kadiealexander kadiealexander added Weekly KSv2 and removed Daily KSv2 labels Sep 12, 2022
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@mvtglobally
Copy link

Issue not reproducible during KI retests. (Second week)

@laurenreidexpensify laurenreidexpensify added the Needs Reproduction Reproducible steps needed label Sep 20, 2022
@melvin-bot melvin-bot bot added the Overdue label Sep 20, 2022
@laurenreidexpensify
Copy link
Contributor

As this issue hasn't been reproducible for 2 weeks now, am going to close out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Needs Reproduction Reproducible steps needed Weekly KSv2
Projects
None yet
Development

No branches or pull requests

10 participants