You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Group Split - Error shows for Merchant and not Amount, while Merchant leads to error page #37523
Reproducible in staging?: Yes Reproducible in production?: No If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs:https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: Applause internal team Slack conversation:
Action Performed:
Go to staging.new.expensify.com
Go to group chat.
Click + > Split bill > Scan.
Create a scan split bill.
Click on the split preview.
Click Split.
Note that it shows error for Merchant and not amount.
Click Merchant.
Expected Result:
In Step 6, it will show error for Amount only.
In Step 7, Merchant editor page will open without issue.
Actual Result:
In Step 6, it shows error for Merchant and not Amount.
In Step 7, Merchant leads to not here page.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
👋 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:
Identify the pull request that introduced this issue and revert it.
@deetergp FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.
Activity
melvin-bot commentedon Feb 29, 2024
Triggered auto assignment to @stephanieelliott (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.github-actions commentedon Feb 29, 2024
👋 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:
melvin-bot commentedon Feb 29, 2024
Triggered auto assignment to @deetergp (
Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.m-natarajan commentedon Feb 29, 2024
We think that this bug might be related to #vip-split-p2p-chat-groups
cc @arielgreen
m-natarajan commentedon Feb 29, 2024
@deetergp FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.
hoangzinh commentedon Feb 29, 2024
I found this bug and reported here #37515 (comment). @kubabutkiewicz I think we can fix this issue in your current PR #37515
puneetlath commentedon Feb 29, 2024
Looks like the revert fixed this.