-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Group - All recents users selected when clicking on "Add to group" #35701
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
Comments
👋 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:
|
Triggered auto assignment to @marcaaron ( |
We think that this bug might be related to #vip-vsb |
Offending PR: #34847 |
@shubham1206agra, I don't think it's coming from that PR, still looking at it. |
I believe it is caused by this very large PR Debugging locally and there seems to be some issue with this commit |
The option selector
The problem is that the option selector does not know the type of list so even if we are supplying a list of account it still compares the policy id field (which is not unique among accounts). I think the best to do here is to compare |
This comment was marked as outdated.
This comment was marked as outdated.
@s77rt I feel like your solution works the best. Since you review the PR, could you raise a PR to fix this please? |
Job added to Upwork: https://www.upwork.com/jobs/~016be032c785f4143f |
Triggered auto assignment to @MitchExpensify ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @allroundexperts ( |
❌ There was an error making the offer to @s77rt for the Contributor role. The BZ member will need to manually hire the contributor. |
This can be closed. No payment is due here since it's part of the ideal nav. |
Closing as per the comment above |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.37-7 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-02-14. 🎊 For reference, here are some details about the assignees on this issue:
|
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:
|
|
Payment summary: $500 - @s77rt (NewDot) |
@MitchExpensify No payment is required here. |
yep this is an issue for a regression stemmed from the Ideal Nav PR sorry about that I should have specified that |
Uh oh!
There was an error while loading. Please reload this page.
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: v1.4.36-0
Reproducible in staging?: y
Reproducible in production?: yn
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: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
Only one user should be selected when clicking on "Add to group"
Actual Result:
All recents users selected when clicking on "Add to group"
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6365181_1706899051810.2024-02-02_21-31-14.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: