[HOLD][$500] Feature request: Unable to add multiple emails at once as members to a room #26242

[mallenexpensify]

Coming from the below, which should be very similar

• [HOLD for payment 2023-10-23] [$1000] Feature request: Unable to add multiple emails at once as members to a workspace #26242

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 staging.new.expensify.com
2. Log in with any account
3. Add multiple emails at once as members to a room

Expected Result:

Allow for multiple emails addresses to be added at once via a comma-separated list when inviting new members to a room.

Actual Result:

Unable to add multiple emails at once as members to a workspace

Workaround:

Unknown

Platforms:

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

• Android / native
• Android / Chrome
• iOS / native
• iOS / Safari
• MacOS / Chrome / Safari
• MacOS / Desktop

Version Number: 1.3.95-5
Reproducible in staging?: y
Reproducible in production?: y
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
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @mallenexpensify
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1692837155454539

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~017b2a4652d665e512
• Upwork Job ID: 1699456983730249728
• Last Price Increase: 2023-11-06

[melvin-bot]

Current assignee @mallenexpensify is eligible for the NewFeature assigner, not assigning anyone new.

[mallenexpensify]

@Samueljh1 would you like to work on this since you worked on the other? If so, the job price is $500.

[mananjadhav]

@mallenexpensify in this gets worked upon, can I work as C+? Helped with the previous issue too.

[mallenexpensify]

Yes, good 👀 Manan, I didn't think about the benefit of having the same C+ too

[Samueljh1]

@Samueljh1 would you like to work on this since you worked on the other? If so, the job price is $500.

Yes, happy to work on this

[melvin-bot]

Current assignee @mallenexpensify is eligible for the Bug assigner, not assigning anyone new.

[melvin-bot]

Bug0 Triage Checklist (Main S/O)

• This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[melvin-bot]

Current assignee @mananjadhav is eligible for the External assigner, not assigning anyone new.

[mananjadhav]

@Samueljh1 Can you post a proposal so that we're aligned on the approach and the changes?

[Samueljh1]

• [HOLD for payment 2023-10-23] [$1000] Feature request: Unable to add multiple emails at once as members to a workspace [HOLD for payment 2023-10-23] [$1000] Feature request: Unable to add multiple emails at once as members to a workspace #26242

The code changes are basically the same.

Proposal

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

When adding new members to a room, we can only invite one at a time.

What is the root cause of that problem?

The handlers and code in RoomInvitePage.js only have logic that support inviting one member at a time (e.g., userToInvite is a single value, instead of an array).

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

Update single-user state variable userToInvite to an array, and update all handling logic to support multiple users (e.g., iterate through the invited emails when updating the list view instead of updating a single item). When handling updates to searchTerm, we can parse the string with the same parsing logic from WorkspaceInvitePage.

What alternative solutions did you explore? (Optional)

N/A.

[Samueljh1]

Screen.Recording.2023-11-08.at.00.26.37.mov

Any reason why this is happening? I am not able to open the invite page. I cannot work on the task until this is resolved. I hope this will be considered in the timing as well of the fix?

Thanks

[mallenexpensify]

I have no idea @Samueljh1 , what platform and version are you on?

[Samueljh1]

I have no idea @Samueljh1 , what platform and version are you on?

Chrome MacOS new.expensify.com. Same issue on iOS Safari.

[DylanDylann]

Proposal

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

• Feature request: Unable to add multiple emails at once as members to a room

What is the root cause of that problem?

• Before, In WorkspaceInvitePage, if user types multiple emails in the search box, for example, "user1@gmail.com, user2@gmail.com, user3@gmail.com", it will parse these text to the array [user1@gmail.com, user2@gmail.com, user3@gmail.com] and for each string in this array, we can call the OptionsListUtils.getMemberInviteOptions(props.personalDetails, props.betas, email, excludedUsers); to get the result, and then wrap all the results to display in the search result sections.
• But the Fix incorrect email validation msg caused comma separated search term #30859 removed this feature in WorkspaceInvitePage.

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

• If we still allow user to add multiple emails at once in room, we can apply the same logic as we did with WorkspaceInvitePage before, like below:

 useEffect(() => {
        let emails = _.compact(
            searchTerm
                .trim()
                .replace(/\s*,\s*/g, ',')
                .split(','),
        );

        if (emails.length === 0) {
            emails = [''];
        }

        const newUsersToInviteDict = {};
        const newPersonalDetailsDict = {};
        const newSelectedOptionsDict = {};

        _.each(emails, (email) => {
            const inviteOptions = OptionsListUtils.getMemberInviteOptions(props.personalDetails, props.betas, email, excludedUsers);

            // Update selectedOptions with the latest personalDetails and policyMembers information
            const detailsMap = {};
            _.each(inviteOptions.personalDetails, (detail) => (detailsMap[detail.login] = OptionsListUtils.formatMemberForList(detail)));

            const newSelectedOptions = [];
            _.each(selectedOptions, (option) => {
                newSelectedOptions.push(_.has(detailsMap, option.login) ? {...detailsMap[option.login], isSelected: true} : option);
            });

            const userToInvite = inviteOptions.userToInvite;

            // Only add the user to the invites list if it is valid
            if (userToInvite) {
                newUsersToInviteDict[userToInvite.accountID] = userToInvite;
            }

            // Add all personal details to the new dict
            _.each(inviteOptions.personalDetails, (details) => {
                newPersonalDetailsDict[details.accountID] = details;
            });

            // Add all selected options to the new dict
            _.each(newSelectedOptions, (option) => {
                newSelectedOptionsDict[option.accountID] = option;
            });
        });

        // Strip out dictionary keys and update arrays
        setUsersToInvite(_.values(newUsersToInviteDict));
        setPersonalDetails(_.values(newPersonalDetailsDict));
        setSelectedOptions(_.values(newSelectedOptionsDict));

        // eslint-disable-next-line react-hooks/exhaustive-deps -- we don't want to recalculate when selectedOptions change
    }, [props.personalDetails, props.policyMembers, props.betas, searchTerm, excludedUsers]);

What alternative solutions did you explore? (Optional)

• NA

[mallenexpensify]

Put on hold, removed reviewing label, left as a weekly

[mallenexpensify]

No action the past week, we'll want to prioritize before too long

[mallenexpensify]

Still held

[mallenexpensify]

on hold

[mallenexpensify]

Holding on a unified design proposal, bumping to monthly til we prioritize.

[mallenexpensify]

Added to #vip-vsb because it doesn't haven't to do with money.

[mallenexpensify]

From the other similar issue

• [HOLD for payment 2023-10-23] [$1000] Feature request: Unable to add multiple emails at once as members to a workspace #26242

This is actively being redesigned with Groups so I'm going to close. We can reopen if it's still behaving unexpectedly once we update.

Temped to close but I'm going to wait one more 'overdue' round then I'll retest and see if it makes sense to keep open or close.

[mallenexpensify]

Closing, will reopen if/when needed.