-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[REPORTING BONUS PAYMENT] [Hold Auth #8141] Web - Search - Two concierge chats and user's own account appears in search #20965
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @arosiclair ( |
@kbecciv can you try reproducing again? |
Alright the root cause of this seems to be an API issue. We build the options list using reports and personal details. We filter duplicates here by login. However, the personal details for Concierge has no login for some reason so both the Concierge chat and personal detail are returned. I can confirm we're not returning the login for Concierge in ![]() Other personal details are unaffected and have logins populated. |
Alright the proper fix for this is #20933 which will filter duplicates by |
This Auth PR seems to fix the issue. Gonna remove the label and put this issue on hold for the Auth deploy. We should confirm it's fixed on staging once we deploy Auth. |
@arosiclair, I think it should be eligible for the reporting compensation. |
Triggered auto assignment to @jliexpensify ( |
This comment was marked as off-topic.
This comment was marked as off-topic.
@jliexpensify can you handle paying @Puneet-here for his report? |
Job added to Upwork: https://www.upwork.com/jobs/~01518684b253e81d9a |
Current assignee @jliexpensify is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh ( |
@fedirjh sorry, no need for C+, just sorting out payment. @Puneet-here - have invited you for the $250 payment. |
@jliexpensify applied thank you. |
No worries, once accepted I shall pay - cheers! |
@Puneet-here - bumping for you to accept so I can pay you :) |
@jliexpensify, I have accepted the offer |
Paid and job closed! |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
Only one concierge chat should appear
Actual Result:
Two concierge chats and user's own account appears in search
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.29.0
Reproducible in staging?: yes
Reproducible in production?: no
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: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686987884987839
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: