App
App copied to clipboard
[$500] Web - Searching for rooms for the first time without hashtags does not work.
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: 1.4.24-7 Reproducible in staging?: y Reproducible in production?: y 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: @justinpersaud Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1705002380962489
Action Performed:
- Create a room with workspace visibility as account A
- Invite user B
- Login as user B
- Search for the room created in step 1 without hashtag
Expected Result:
Room is able to find
Actual Result:
It does not show the result
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/38435837/3f2aaa04-9c35-42c3-b70c-0256b6487203
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~015f0d8aaddbf938d1
- Upwork Job ID: 1747032811175395328
- Last Price Increase: 2024-02-05
Triggered auto assignment to @muttmuure (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Job added to Upwork: https://www.upwork.com/jobs/~015f0d8aaddbf938d1
Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External
)
Hoping/guessing this can be External. @getusha , comment if you disagree. Thx
Proposal
Please re-state the problem that we are trying to solve in this issue
Searching for rooms for the first time without hashtags does not work
What is the root cause of that problem?
This happens because we have provided full room name with #
, which might make the search different from anticipated.
Function call
OptionsListUtils.getSearchText(report, reportName, participantPersonalDetailList, result.isChatRoom || result.isPolicyExpenseChat, result.isThread);
What changes do you think we should make in order to solve the problem?
We can add a stripped version of title in searchText here (similarly in other places too)
https://github.com/Expensify/App/blob/8e2fb675389bb84deee4b8e001a536252883e72f/src/libs/OptionsListUtils.js#L332
We can use similar approach to https://github.com/Expensify/App/blob/8e2fb675389bb84deee4b8e001a536252883e72f/src/libs/OptionsListUtils.js#L320
The motivation behind this is https://github.com/Expensify/App/issues/33029 will perform similar stripping before doing sorting on the display name. So its natural to add a stripped version in search text. We will also keep the original version in case user tries to search with stripped symbols.
What alternative solutions did you explore? (Optional)
@mallenexpensify Can you add this to #vip-vsb
, and maybe same priority as https://github.com/Expensify/App/issues/33029?
@muttmuure, @getusha Whoops! This issue is 2 days overdue. Let's get this updated quick!
@shubham1206agra I am not able to reproduce this on staging.
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
The bug seems to be a little inconsistent
I can only reproduce if I search for a room I haven't returned in search in that session yet (but I know exists and I am a member of it)
Still trying to reproduce this, will try to provide an update by tomorrow.
What @muttmuure described is exactly what the issue is. It only happens if you haven't found the room or searched for it before.
I can try to reproduce this today.
slack context: https://expensify.slack.com/archives/C02NK2DQWUX/p1706108522357449
Okay. I was able to reproduce this.
https://github.com/Expensify/App/assets/30054992/19a0f157-189a-4c84-a8bd-54eb60652885
@shubham1206agra Thanks for your proposal. I think your RCA is not very clear. Do you mind explaining why it works some times but not other?
Great, going to make @allroundexperts the C+ here
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
I guess @shubham1206agra is busy with other issues. Still looking for better proposals here.
Issue not reproducible during KI retests. (First week)
If we can't reproduce, can we close this?
I think it might still be repro, asking in #social for a few room names I haven't already visited
@allroundexperts, @muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
@allroundexperts @muttmuure this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
I can still reproduce this. I can't remember the last time I searched for this room, but I did it now:
Then after searching for it once, I searched again and it worked:
I remember we made some changes to OptionList
where we were limiting the number of chats returned by search, and we added a debounce.
It's just a theory, but I wonder if the debounce before querying the full list is engaged AFTER the initial input, and then if you search again we are actually querying from your "full" list. This would make sense in terms of the expected behavior here, because we can all retrieve a result that we have searched for recently.
asked about this here: https://expensify.slack.com/archives/C03UK30EA1Z/p1707158970024939
Hello, Im Artem from Callstack and would like to help with this issue!
Thanks @waterim , assigned!
Was trying to reproduce it for quite some time, but nothing. I think after sidebar refactor something was changed, because now it works good with newly created rooms, also tried old, can't reproduce too.
Can someone who could reproduce try it once again?
Uploaded two repro steps which I found here:
https://github.com/Expensify/App/assets/39777589/c022445b-fd4b-4bb9-af4b-0c19776db73f
https://github.com/Expensify/App/assets/39777589/458ff023-c7fc-4711-bb43-5ce9102c0928