-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[ON HOLD] [MEDIUM] [Splits] [$250] Split bill - Split amount is lost when adding receipt and splitting bill #37491
Comments
Triggered auto assignment to @isabelastisser ( |
👋 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:
|
Triggered auto assignment to @grgia ( |
@grgia 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. |
I believe this is happening with manual requests too |
ProposalPlease re-state the problem that we are trying to solve in this issue.When What is the root cause of that problem?Here, when we split the bill with a receipt, it was thought to be done via the scan tab where we do not enter the amount. In such a case, the flow starts with a split bill request and is completed later. However, as done in this issue, it is also possible to split the bill via I do not think this is a regression. What changes do you think we should make in order to solve the problem?We can support split bill request for manual request type with receipt by making use of the
However, this would also need BE support as What alternative solutions did you explore? (Optional)Alternatively, if we want to do this via |
FYI, this is likely fixed by this revert. Will have QA re-test once it is CP'd. |
Looks like that revert ^ didn't fix the issue. So we still need to figure out what is going on here @grgia. |
Another regression from #33482 |
ok, if this bug happens on production when split bill with workspace (not single user), not blocker. |
have you tested this? |
checking |
we can't test that scenario in prod, because the PR that fixes detecting 'Split' from the route is only in staging. Notice how the request never becomes a split. Screen.Recording.2024-02-29.at.22.09.51.mov |
@bernhardoj I think this is an edge case we missed, do you mind taking a look? |
👍 |
I think demoting is fine here. |
@rojiphil are you able to work on this soon? |
Lot to unpack here but, yes, will pick this up today in some time. |
@rojiphil can you please provide an update? Thanks! |
The current status is that @youssef-lr is taking a look (mentioned here) at any needed BE changes for this feature based on the FE changes made thus far in a draft PR |
I haven't had time to prioritize this yet, I'm on it next week. |
Not overdue. Any updates, @youssef-lr? Thanks! |
No update yet @isabelastisser :/, I'm focusing on Aug 15th issues |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@rojiphil @youssef-lr Bump on the status here. |
Any updates here @youssef-lr ? thanks! |
I think we can put this on monthly for as long as vip-split is paused |
vip-split is paused; issue is on hold. |
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: v1.4.45-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers): natnael.expensify+3@gmail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal Team
Action Performed:
Expected Result:
Entered amount shouldn't be lost, and request shouldn't change to scan request
Actual Result:
Entered amount is lost and request is changed to a scan request
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6396788_1709201886587.Screen_Recording_2024-02-29_at_10.49.14_in_the_morning.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: