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 - Tapping settings button from room details crashes the app #7182

Closed
kavimuru opened this issue Jan 12, 2022 · 16 comments
Closed

iOS - Tapping settings button from room details crashes the app #7182

kavimuru opened this issue Jan 12, 2022 · 16 comments
Assignees
Labels
Daily KSv2 Engineering Reviewing Has a PR in review

Comments

@kavimuru
Copy link

kavimuru commented Jan 12, 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!


Issue was found when executing the PR #7028

Action Performed:

  1. Open app and login applausetester+0704sveta@applause.expensifail.com
  2. Select any room
  3. Tap on name of the room to view the details
  4. Tap on Settings

Expected Result:

Settings page should open

Actual Result:

App crashes all the time

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS
  • Web - Mac safari - Blank page appear
  • mweb - Blank page appears

Version Number: 1.1.29-2
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Bug5404870_log_Settings_12.01.txt

https://user-images.githubusercontent.com/43996225/149233948-d5cb55a4-efaa-4598-9bb6-879c925b4a84.mp4
Screen Shot 2022-01-12 at 5 45 22 PM

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

@kavimuru kavimuru added the DeployBlockerCash This issue or pull request should block deployment label Jan 12, 2022
@OSBotify
Copy link
Contributor

👋 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
Copy link

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

@Jag96
Copy link
Contributor

Jag96 commented Jan 13, 2022

cc @TomatoToaster looks like this was introduced in #7028, can you confirm this should be a blocker?

@TomatoToaster
Copy link
Contributor

Yep thanks for the tag, I can work on this

@stitesExpensify
Copy link
Contributor

Should anything related to rooms be a blocker? It's all behind a beta right?

@stitesExpensify
Copy link
Contributor

Also while testing just now I found another bug for a new room because I didn't have an icon
2022-01-13_16-04-48

Besides that though, I cannot reproduce this in safari

@Jag96
Copy link
Contributor

Jag96 commented Jan 13, 2022

Should anything related to rooms be a blocker? It's all behind a beta right?

If its behind a beta I agree it shouldn't be a blocker, removing the label

@Jag96 Jag96 added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Jan 13, 2022
@TomatoToaster
Copy link
Contributor

@stitesExpensify omg that screenshot is VERY helpful. I've been trying to recreate what I saw once which was that I wouldn't see the icons when I created the room. I think that's causing the crash. Works when you log back in but is like that the first time.

@TomatoToaster
Copy link
Contributor

Got a draft that I think will address the icon issue.

@TomatoToaster
Copy link
Contributor

Still working on the fix for this!

@MelvinBot MelvinBot removed the Overdue label Jan 18, 2022
@neil-marcellini
Copy link
Contributor

@TomatoToaster @Jag96 @stitesExpensify
Hey guys, while testing this PR I had a similar issue where just tapping on the name of a new room to view the details causes a crash. It happens on all platforms and it looks like the icons are the issue. Shall I update the issue to reflect this?

@TomatoToaster
Copy link
Contributor

@neil-marcellini does it happen when you are the only member of the workspace? I'm working on a fix for that and I've got a PR for it, but I just want to confirm I might not be missing another case.

@MelvinBot MelvinBot removed the Overdue label Jan 20, 2022
@neil-marcellini
Copy link
Contributor

Yep it only happens when you are the only member of the workspace. Thanks!

@TomatoToaster TomatoToaster added the Reviewing Has a PR in review label Jan 21, 2022
@botify botify closed this as completed Jan 25, 2022
@kbecciv
Copy link

kbecciv commented Feb 16, 2022

@Jag96 @TomatoToaster @neil-marcellini When tester was executing the PR #7692, we still have crash when tap on Settings. Do we need a separate issue?

Recording.222.mp4

@kbecciv kbecciv reopened this Feb 16, 2022
@TomatoToaster
Copy link
Contributor

@kbecciv Yes this should be its own issue I think, but could you assign me when it is created? I can look into it.

@kbecciv
Copy link

kbecciv commented Feb 16, 2022

@TomatoToaster assigned #7786

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Engineering Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

9 participants