Skip to content
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

Missing create options from the DM create menu in some DMs and not others #48282

Closed
1 of 6 tasks
m-natarajan opened this issue Aug 29, 2024 · 6 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed

Comments

@m-natarajan
Copy link

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:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs Reproduction
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: @trjExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724860697269619

Action Performed:

  1. Go to staging.new.expensify.com
  2. Initiate a DM with people on expensify.com
  3. Click the +

Expected Result:

All create menu options displayed (Split, Submit, Pay, Assign task and add)

Actual Result:

Only Assign task and add attachment options displayed for people on expensify.com

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

View all open jobs on GitHub

@m-natarajan m-natarajan added Daily KSv2 Needs Reproduction Reproducible steps needed Bug Something is broken. Auto assigns a BugZero manager. labels Aug 29, 2024
Copy link

melvin-bot bot commented Aug 29, 2024

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

@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@neonbhai
Copy link
Contributor

Can reproduce:
Screenshot 2024-08-29 at 11 52 22 PM

@neonbhai
Copy link
Contributor

Proposal

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

Missing create options from the DM create menu in some DMs and not others

What is the root cause of that problem?

This happened after #47214

We use isExpensifyTeam here to check if we should show the money request options. This function includes the check for @expensify domain, hence the options are not shown.

App/src/libs/ReportUtils.ts

Lines 6385 to 6387 in e054a71

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

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

We don't want to show the Create Options for only the Expensify guide emails (team.expensify.com). We'll create a uti isExpensifyGuide to help us

function isExpensifyGuide(email: string | undefined): boolean {
    const emailDomain = Str.extractEmailDomain(email ?? '');
    return emailDomain === CONST.EMAIL.GUIDES_DOMAIN;
}

Change here to:

const otherParticipants = reportParticipants.filter(
    (accountID) => currentUserPersonalDetails?.accountID !== accountID && !PolicyUtils.isExpensifyGuide(allPersonalDetails?.[accountID]?.login),
);

@trjExpensify
Copy link
Contributor

That's not great, I've commented in the parent issue of the PR that introduced it as it's within the regression period: #46739

@melvin-bot melvin-bot bot added the Overdue label Sep 2, 2024
@twisterdotcom
Copy link
Contributor

This was reverted, so, going to close this one.

@melvin-bot melvin-bot bot removed the Overdue label Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed
Projects
None yet
Development

No branches or pull requests

5 participants