-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Search - selfDM is not shown in search #51471
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 @stitesExpensify ( |
💬 A slack conversation has been started in #expensify-open-source |
Left a comment on this PR that's potentially related |
hm no, i just can't reproduce it, neither on staging. I don't think this related to the mentioned PR |
Any ideas here about this being front of backend-related? 🤔 |
I could find the selfDM in search if I type |
@Beamanator but you weren't able to find it by searching for your mail address? |
I think this is a sorting priority issue. Try creating many chat reports (threads) that match the search term. In the example I posted in Slack, all the results seem to match the search input, but all the threads are prioritized above the "main" self-dm report |
Ooh now I can see it easily - I wonder if this was fixed by the revert that @luacmartins recommended here? |
as mentioned in slack here, this was fixed! I won't close it in case @stitesExpensify wants to try to get to the bottom of which PR revert fixed this |
I do not have time to dig on that with travel right now. I'm going to close since it is fixed |
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: 9.0.54-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5131233
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
SelfDM is shown in the search result
Actual Result:
SelfDM is not shown in the search result for a private domain.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6645542_1729862262578.2024-10-25_16.17.14.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: