App icon indicating copy to clipboard operation
App copied to clipboard

[$500] Web - Searching for rooms for the first time without hashtags does not work.

Open m-natarajan opened this issue 1 year ago • 29 comments

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:

  1. Create a room with workspace visibility as account A
  2. Invite user B
  3. Login as user B
  4. 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

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~015f0d8aaddbf938d1
  • Upwork Job ID: 1747032811175395328
  • Last Price Increase: 2024-02-05

m-natarajan avatar Jan 13 '24 16:01 m-natarajan

Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

melvin-bot[bot] avatar Jan 13 '24 16:01 melvin-bot[bot]

Job added to Upwork: https://www.upwork.com/jobs/~015f0d8aaddbf938d1

melvin-bot[bot] avatar Jan 15 '24 23:01 melvin-bot[bot]

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

melvin-bot[bot] avatar Jan 15 '24 23:01 melvin-bot[bot]

Hoping/guessing this can be External. @getusha , comment if you disagree. Thx

mallenexpensify avatar Jan 15 '24 23:01 mallenexpensify

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)

shubham1206agra avatar Jan 16 '24 08:01 shubham1206agra

@mallenexpensify Can you add this to #vip-vsb, and maybe same priority as https://github.com/Expensify/App/issues/33029?

shubham1206agra avatar Jan 16 '24 08:01 shubham1206agra

@muttmuure, @getusha Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Jan 19 '24 17:01 melvin-bot[bot]

@shubham1206agra I am not able to reproduce this on staging.

Screenshot 2024-01-22 at 3 02 40 PM

getusha avatar Jan 22 '24 12:01 getusha

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] avatar Jan 22 '24 16:01 melvin-bot[bot]

The bug seems to be a little inconsistent

muttmuure avatar Jan 22 '24 16:01 muttmuure

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)

muttmuure avatar Jan 22 '24 16:01 muttmuure

Still trying to reproduce this, will try to provide an update by tomorrow.

getusha avatar Jan 23 '24 21:01 getusha

What @muttmuure described is exactly what the issue is. It only happens if you haven't found the room or searched for it before.

justinpersaud avatar Jan 24 '24 14:01 justinpersaud

I can try to reproduce this today.

allroundexperts avatar Jan 24 '24 16:01 allroundexperts

slack context: https://expensify.slack.com/archives/C02NK2DQWUX/p1706108522357449

roryabraham avatar Jan 24 '24 19:01 roryabraham

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?

allroundexperts avatar Jan 24 '24 22:01 allroundexperts

Great, going to make @allroundexperts the C+ here

roryabraham avatar Jan 24 '24 23:01 roryabraham

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] avatar Jan 29 '24 16:01 melvin-bot[bot]

I guess @shubham1206agra is busy with other issues. Still looking for better proposals here.

allroundexperts avatar Jan 29 '24 20:01 allroundexperts

Issue not reproducible during KI retests. (First week)

mvtglobally avatar Jan 30 '24 04:01 mvtglobally

If we can't reproduce, can we close this?

quinthar avatar Feb 01 '24 04:02 quinthar

I think it might still be repro, asking in #social for a few room names I haven't already visited

mallenexpensify avatar Feb 02 '24 02:02 mallenexpensify

@allroundexperts, @muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Feb 02 '24 15:02 melvin-bot[bot]

@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!

melvin-bot[bot] avatar Feb 03 '24 15:02 melvin-bot[bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] avatar Feb 05 '24 16:02 melvin-bot[bot]

I can still reproduce this. I can't remember the last time I searched for this room, but I did it now:

image

Then after searching for it once, I searched again and it worked:

image

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.

muttmuure avatar Feb 05 '24 18:02 muttmuure

asked about this here: https://expensify.slack.com/archives/C03UK30EA1Z/p1707158970024939

muttmuure avatar Feb 05 '24 18:02 muttmuure

Hello, Im Artem from Callstack and would like to help with this issue!

waterim avatar Feb 06 '24 10:02 waterim

Thanks @waterim , assigned!

mallenexpensify avatar Feb 06 '24 23:02 mallenexpensify

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

waterim avatar Feb 07 '24 14:02 waterim