[HOLD for payment 2024-07-22] [$250] [HOLD for payment 2024-07-17] [HOLD for payment 2024-07-10] Workspace - Inconsistent delete modal display and brief error when deleting a user

[lanitochka17]

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: 9.0.3-2 Reproducible in staging?: Y Reproducible in production?: N If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Email or phone of affected tester (no customers): shussain+chrome1@applausemail.com Issue reported by: Applause - Internal Team

issue found when executing PR https://github.com/Expensify/App/pull/44008

Action Performed:

Precondition: Create Control workspace in OldDot, make sure you can see it in NewDot In OldDot, make sure the Control workspace has "Advanced Approvals" set up (on the workspace settings -> Members tab) In OldDot, Add at least 2 other members (non-admin) In OldDot, Set up 1 of the members (Member A) as the submitsTo of the other member (Member B)

in NewDot

1. Go to the control WS created above
2. Open member page
3. Select user A and click on remove
4. Verify that the confirm modal appears with the warning prompt [Selected member] is an approver in this workspace. When you unshare this workspace with them, we'll replace them in the approval workflow with the workspace owner, [Workspace Owner].
5. Open the member detail page of member A
6. Click on Remove from workspace button
7. Verify that the confirm modal appears with the same warning prompt in the step 4
8. Click on remove
9. Verify that the user is removed from the WS

Expected Result:

The delete modal should consistently display the same user identifier (either "Username" or "Email") on both the list member page and the detail page. No error should appear when deleting a user

Actual Result:

Inconsistent delete modal display and brief error when deleting a user

Workaround:

Unknown

Platforms:

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

• [ ] Android: Native
• [ ] Android: mWeb Chrome
• [ ] iOS: Native
• [ ] iOS: mWeb Safari
• [x] MacOS: Chrome / Safari
• [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/b91344c2-bed5-4d73-ac26-e22e15ffced8

https://github.com/Expensify/App/assets/78819774/343d77d6-12de-4eb8-a90a-5e6f1eb8314f

View all open jobs on GitHub

Issue Owner

Current Issue Owner: @greg-schroeder

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01be201c5b6d5fc51e
• Upwork Job ID: 1811099866306463912
• Last Price Increase: 2024-07-10

[melvin-bot[bot]]

Triggered auto assignment to @yuwenmemon (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[github-actions[bot]]

:wave: 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:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[Beamanator]

Not a web-e blocker, looks like this is caused by https://github.com/Expensify/App/pull/44008

[Beamanator]

@nkdengineer it looks like your PR https://github.com/Expensify/App/pull/44008 isn't working perfectly in a few conditions, can you please take a look at this?

[nkdengineer]

It's fixed by @yuwenmemon

[Beamanator]

oh i should have looked at linked PRs 😅

[melvin-bot[bot]]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot[bot]]

The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.3-7 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

• https://github.com/Expensify/App/pull/44654

If no regressions arise, payment will be issued on 2024-07-10. :confetti_ball:

[ikevin127]

I'm the only one requiring payment here for C+ review of PR: https://github.com/Expensify/App/pull/44654#issuecomment-2197796335.

@yuwenmemon When you get a chance, mind adding the Bug label to this issue for payment ?

[melvin-bot[bot]]

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

[melvin-bot[bot]]

The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.5-13 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

• https://github.com/Expensify/App/pull/44654

If no regressions arise, payment will be issued on 2024-07-17. :confetti_ball:

[melvin-bot[bot]]

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [x] [@yuwenmemon] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/44008
• [x] [@yuwenmemon] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [x] [@yuwenmemon] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [x] [@yuwenmemon] Determine if we should create a regression test for this bug.
• [x] [@yuwenmemon] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [ ] [@greg-schroeder] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[ikevin127]

@greg-schroeder Payment is due today (10th) as this was deployed to production manually in 9.0.3-7 (reference).

[melvin-bot[bot]]

Job added to Upwork: https://www.upwork.com/jobs/~01be201c5b6d5fc51e

[melvin-bot[bot]]

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

[greg-schroeder]

Sent you an offer @ikevin127

[ikevin127]

@greg-schroeder Accepted, thank you!

[greg-schroeder]

Paid

[melvin-bot[bot]]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot[bot]]

The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.6-8 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

• https://github.com/Expensify/App/pull/44654

If no regressions arise, payment will be issued on 2024-07-22. :confetti_ball:

For reference, here are some details about the assignees on this issue:

• @hungvu193 requires payment through NewDot Manual Requests

[melvin-bot[bot]]

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [ ] [@hungvu193] The PR that introduced the bug has been identified. Link to the PR:
• [ ] [@hungvu193] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [ ] [@hungvu193] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [ ] [@hungvu193] Determine if we should create a regression test for this bug.
• [ ] [@hungvu193] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [ ] [@greg-schroeder] Link the GH issue for creating/updating the regression test once above steps have been agreed upon: