[$250] IOU - In newly created workspace, with no members invited, split expense is displayed

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: 9.0.16
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Action Performed:

1. Launch app
2. Tap profile icon -- workspaces -- new workspace
3. Navigate to newly created workspace in the list and using 3 dots on beside navigate to #admin room
4. Tap plus icon near compose and select split expense.
5. Enter an amount and tap next

Expected Result:

In newly created workspace, with no members invited, split expense option must not be displayed in #admin room. When no members invited in workspace, it must not show hidden member

Actual Result:

In newly created workspace, with no members invited, split expense option is displayed in #admin room. When trying to split expense, a hidden member is shown without invited

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/e01f812e-aab0-4f3a-bdb3-fa434f487b75

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0125de9b07488ab7a1
• Upwork Job ID: 1820610996701136238
• Last Price Increase: 2024-08-06
• Automatic offers:
• ahmedGaber93 | Reviewer | 103461530

Issue Owner

Current Issue Owner: @ahmedGaber93

[melvin-bot]

Triggered auto assignment to @adelekennedy (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.

[lanitochka17]

@adelekennedy 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

[lanitochka17]

We think that this bug might be related to #vip-split

[rlmorrison74]

I really apologize for commenting here, but I don't know where else to go that someone from expensify might see it. I emailed [email protected] a week ago requesting access to slack, along with a follow up a few days ago, and I haven't heard back. Are we just not accepting new contributors at the moment, or is there some other step I need to follow to gain access? Can someone please advise? Again, sorry for hijacking an issue thread.

[melvin-bot]

📣 @rlmorrison74! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[abzokhattab]

The issue is not reproducible on the latest main ... tried on IOS

[kpadmanabhan]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Workspace with 0 members shows Split expense option.

What is the root cause of that problem?

We do not check count of members for displaying Split expense option in context menu.

What changes do you think we should make in order to solve the problem?

Check for members count before showing the option in
src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.tsx

What alternative solutions did you explore? (Optional)

[daledah]

Proposal

Please re-state the problem that we are trying to solve in this issue.

In newly created workspace, with no members invited, split expense option is displayed in #admin room. When trying to split expense, a hidden member is shown without invited

What is the root cause of that problem?

Here is the condition to display the split option in admin room:

App/src/libs/ReportUtils.ts

Line 6103 in 8abdcca

(isChatRoom(report) && !isAnnounceRoom(report) && otherParticipants.length > 0) ||

with the otherParticipants is:

App/src/libs/ReportUtils.ts

Line 6090 in 8abdcca

const otherParticipants = reportParticipants.filter((accountID) => currentUserPersonalDetails?.accountID !== accountID);

which is not filtered out the participant that belongs to Expensify team.

What changes do you think we should make in order to solve the problem?

The otherParticipants should be:

    const otherParticipants = reportParticipants.filter((accountID) => {
        const detail = allPersonalDetails?.[accountID];
        return currentUserPersonalDetails?.accountID !== accountID && !PolicyUtils.isExpensifyTeam(detail?.login);
    });

What alternative solutions did you explore? (Optional)

Beside above changes, we can apply the same filter in:

App/src/pages/iou/request/step/IOURequestStepAmount.tsx

Line 256 in 8abdcca

const participantAccountIDs = Object.keys(report.participants).map((accountID) => Number(accountID));

and

App/src/components/MoneyRequestConfirmationList.tsx

Line 452 in 8abdcca

const splitParticipants = useMemo(() => {

to make sure the specialist is not included when splitting.

[hayes102]

I am not also able to reproduce the issue ... the split option doesn't appear in the admin room

[melvin-bot]

📣 @hayes102! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[melvin-bot]

@adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[adelekennedy]

reproduced - I'm also able to reproduce this on web, chrome v9.0.16-5, IMO this is a bug, we shouldn't show the split option until additional members are invited

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~0125de9b07488ab7a1

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @ahmedGaber93 (External)

[adelekennedy]

@ahmedGaber93 already some proposals to review!

[adelekennedy]

@rlmorrison74 Let me search for your email in the inbox - I'll reach out via email fro next steps

[ahmedGaber93]

I'm not able to reproduce on web, do I missed something?

20240807045722135.mp4

[daledah]

@ahmedGaber93 You can try the new account, with the first workspace in that account.

[ahmedGaber93]

You can try the new account, with the first workspace in that account.

@daledah Still not able to reproduce

20240807050838848.mp4

[trjExpensify]

is there is any case the guide member with email team.expensify.com can be normal user of admin room and not a guide member? For example, if we have a workspace for guide team, the guide team members here will be normal users on admin room not guide member

What do you mean by normal user? They can have their own workspaces for testing and demos on that domain, if that's the question.

[trjExpensify]

Does anyone have the latest on what we do with Guides/AMs now? Do we just add them as a member of the #admins room, versus adding them to the workspace when assigned to a lead? CC: @puneetlath

If that's the case, then I think it's probably fine to just show the guide/AM as a member of the #admins room and thus as a participant here:

[puneetlath]

Does anyone have the latest on what we do with Guides/AMs now? Do we just add them as a member of the #admins room, versus adding them to the workspace when assigned to a lead? CC: @puneetlath

Yes, that's correct. We add them to the admins room, but not the workspace.

[trjExpensify]

Okay cool, then I think it's fine to show the guide/AM as a participant of the #admins room and not hide that. Meaning, they can be selected for a Split in the room, but I highly doubt anyone will do that in reality.

So what we should try to fix here is why they're showing as hidden. Agreed?

[puneetlath]

Makes sense to me. Hiding seems to cause more problems than benefits.

[ahmedGaber93]

Thanks all for clearing the context.

[ahmedGaber93]

@adelekennedy @daledah The "hidden" participant not reproduced with me in this case, it always displays the Guide member. Is there anyone able to reproduce it?

20240905094216641.mp4

[ahmedGaber93]

@daledah the issue is not reproduced on latest main with me. Can you confirm?

[daledah]

@ahmedGaber93 For me, on the lastest main, the Split expense option is not displayed:

Screen.Recording.2024-09-17.at.11.29.40.mov

[ahmedGaber93]

@daledah can you try to clear the cache like you described here #46739 (comment) then retest.

[ahmedGaber93]

@daledah bump here #46739 (comment)

[daledah]

@ahmedGaber93 Doesn't seem like any different when I clear cache.

Screen.Recording.2024-09-19.at.17.02.25.mov

[ahmedGaber93]

@adelekennedy The "hidden" participant is not reproduced with me in this case, it always displays the Guide member. I think we can close it unless someone able to reproduce. #46739 (comment)

[adelekennedy]

agreed - let's close this one!

[daledah]

@adelekennedy @Gonals Sorry, should I and C+ have payment in this issue?
Based on the similar issue and issue: I belive we should be eligible for payment here, since the work was done by Contributor (me) and reviewed and approved by C+ (@ahmedGaber93 ). Then the issue is not reproduced anymore.

[daledah]

@adelekennedy Please help check my comment in case you miss it

[adelekennedy]

ah good catch - sorry about that I agree payment is due here

Payouts due:

• Contributor: $250 @daledah Upwork
• Contributor+: $250 @ahmedGaber93 Upwork

@daledah @ahmedGaber93 will you apply here? Upwork is letting he search talent at the moment

[daledah]

@adelekennedy My Upwork link is https://www.upwork.com/freelancers/~0138d999529f34d33f, could you kindly send the offer for me there?

[ahmedGaber93]

@daledah @ahmedGaber93 will you apply here? Upwork is letting he search talent at the moment

@adelekennedy I already applied here, can you complete it?

[adelekennedy]

@daledah sent @ahmedGaber93 just sent offer, please accept and I will pay you

[ahmedGaber93]

Hi, @adelekennedy @daledah what is left here to close this issue? Asking because of the ownership of the issue is assigned to me, but I don't have anything to do, thanks!

[daledah]

@ahmedGaber93 We're waiting on @adelekennedy to finish handling payment.

[adelekennedy]

I'm so sorry for the delay here - @daledah payment just made!

[ahmedGaber93]

I think we can close it now.