[Hold for Payment 13th August] Deselect attendees when splitting a bill in group DM.

[rushatgabhane]

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. Go to a group.
2. Split a bill in the group.
3. Enter amount, go next.
4. Unselect attendees.

Expected Result:

Should be able to unselect attendees.

Actual Result:

Can't unselect attendees.

Workaround:

User can split the bill from outside the group.

Platform:

Where is this issue occurring?

Web ✔
iOS ✔
Android ✔
Desktop App ✔
Mobile Web ✔

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:

View all open jobs on Upwork

[MelvinBot]

Triggered auto assignment to @MitchExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[rushatgabhane]

Proposal

In IOUConfirmationList.js
Modify OptionsList props, getSplits(), getSections(), define toggleOption() etc to get the desired behaviour.

Split with the selected attendees only.
Disable button when no attendee is selected.

[trjExpensify]

Thanks, @rushatgabhane! So a couple of additional things we'll want to consider in the proposal for this:

• when you deselect the attendee, that attendee isn't removed from the list completely, but rather the checkmark is removed from the radio button and the amount is updated to %0
• The amounts for the attendees still selected are then increased to the new even split values
• We still add the split comment to the group DM you actioned on this from with all members, it's just modified to only include the attendees that were selected for the split.

CC: @shawnborton @Julesssss @kevinksullivan

[rushatgabhane]

Yes, definitely!
That is how it'll be implemented

[MitchExpensify]

Reproduced as triager, labelling engineering! Looks like a good external issue but will let an engineer make that call (I can create the Upwork job once thats confirmed!)

[MelvinBot]

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

[jasperhuangg]

Yup, writeup looks good to me, and was also able to reproduce it.