Repository navigation
Group - Website crashes when leaving group chat. #54693
Description
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.80-1
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5395271&group_by=cases:section_id&group_order=asc&group_id=229067
Email or phone of affected tester (no customers): Ibellicotest+133@gmail.com
Issue reported by: Applause Internal Team
Device used: Motorola Moto G60/ Android 12 - Windows 10/ Chrome
App Component: Other
Action Performed:
- Open the staging.new.expensify.com website.
- Open any group chat or create one if necessary.
- Tap on group header.
- Tap on "Leave"
Expected Result:
The user should land on LHN after leaving a group chat and this group should be automatically deleted.
Actual Result:
Website crashes when the user tries to leave a group chat.
Workaround:
Unknown
Platforms:
- Android: Standalone
- Android: HybridApp
- Android: mWeb Chrome
- iOS: Standalone
- iOS: HybridApp
- iOS: mWeb Safari
- MacOS: Chrome / Safari
- MacOS: Desktop
Screenshots/Videos
Bug6704688_1735688754617!log.txt
Activity
melvin-bot commentedon Jan 1, 2025
Triggered auto assignment to @marcaaron (
DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.melvin-bot commentedon Jan 1, 2025
Triggered auto assignment to @bfitzexpensify (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.melvin-bot commentedon Jan 1, 2025
💬 A slack conversation has been started in #expensify-open-source
github-actions commentedon Jan 1, 2025
👋 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:
NJ-2020 commentedon Jan 1, 2025
Offending PR: #53902
We remove the default ID
CONST.DEFAULT_NUMBER_IDhere, but since these value can undefined so it will throw an errorApp/src/pages/home/HeaderView.tsx
Line 77 in e5108a1
We can add the default ID in case if this value is undefined
melvin-bot commentedon Jan 1, 2025
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
melvin-bot commentedon Jan 1, 2025
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
isagoico commentedon Jan 6, 2025
Issue not reproducible anymore on 9.0.80-6 🎉
screen-20250106-133112.1.mp4
melvin-bot commentedon Jan 24, 2025
This issue has not been updated in over 15 days. @marcaaron, @bfitzexpensify 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!
bfitzexpensify commentedon Feb 26, 2025
Closing since this is not reproducible and we've had no further reports.