Skip to content
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

[$250] Android-Room-After creating room, entering @ not showing contact suggestions #42766

Closed
1 of 6 tasks
m-natarajan opened this issue May 29, 2024 · 29 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor

Comments

@m-natarajan
Copy link

m-natarajan commented May 29, 2024

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.77-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4585586
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:

  1. Launch app
  2. Tap fab -- start chat
  3. Create a new room
  4. Enter @ and wait few seconds
  5. Navigate to LHN
  6. Navigate to room again
  7. Keep cursor after @

Expected Result:

After creating room, entering @ must show contact suggestions

Actual Result:

After creating room, entering @ not showing contact suggestions but shown on revisit.

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6495138_1716990722444.Screenrecorder-2024-05-29-19-16-06-795_compress_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016d65d15eadf82bb4
  • Upwork Job ID: 1795831001903484928
  • Last Price Increase: 2024-05-29
  • Automatic offers:
    • alitoshmatov | Reviewer | 102529959
    • ishpaul777 | Contributor | 102529960
Issue OwnerCurrent Issue Owner: @jliexpensify
@m-natarajan m-natarajan added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. DeployBlocker Indicates it should block deploying the API labels May 29, 2024
Copy link

melvin-bot bot commented May 29, 2024

Triggered auto assignment to @bondydaa (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

Copy link

melvin-bot bot commented May 29, 2024

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels May 29, 2024
Copy link
Contributor

👋 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:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@m-natarajan
Copy link
Author

m-natarajan commented May 29, 2024

@bondydaa FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@m-natarajan
Copy link
Author

We think that this bug might be related to #vip-vsb

@mountiny mountiny added External Added to denote the issue can be worked on by a contributor and removed DeployBlocker Indicates it should block deploying the API labels May 29, 2024
@melvin-bot melvin-bot bot changed the title Android-Room-After creating room, entering @ not showing contact suggestions [$250] Android-Room-After creating room, entering @ not showing contact suggestions May 29, 2024
Copy link

melvin-bot bot commented May 29, 2024

Job added to Upwork: https://www.upwork.com/jobs/~016d65d15eadf82bb4

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 29, 2024
Copy link

melvin-bot bot commented May 29, 2024

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

@bondydaa
Copy link
Contributor

hmm the site has been pretty slow most of the day, if this relies on a network request that could explain it. or is it pretty consistent that first time @ is type nothing is triggered?

@bondydaa
Copy link
Contributor

hmm okay was able to reproduce on android only. it doesn't look like you have to create a new room. i'm able to consistently reproduce by just going into any existing room and trying typing @. if i leave and go back into the same room then @ triggers the list of members to show up.

@bondydaa
Copy link
Contributor

i think the code for this is roughly these components

https://github.com/Expensify/App/blob/main/src/pages/home/report/ReportActionCompose/SuggestionMention.tsx
https://github.com/Expensify/App/blob/main/src/pages/home/report/ReportActionCompose/Suggestions.tsx

I don't really see any recent changes in those files though.

There are these 2 PRs that had changes to the ReportActionCompose.tsx component but neither stand out to me as the root cause

#42180
#42615

@ishpaul777
Copy link
Contributor

👋

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label May 30, 2024
Copy link

melvin-bot bot commented Jun 3, 2024

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

@jliexpensify
Copy link
Contributor

Hmm not overdue, right? Waiting for it to be deployed?

@tomekzaw
Copy link
Contributor

tomekzaw commented Jun 5, 2024

There were no Android-specific changes between [email protected] and 0.1.76 so I have no idea what could have caused this regression.

edit: I've bumped Live Markdown to 0.1.76 and I cannot reproduce the original problem with suggestion box, it works for me.

@fabOnReact
Copy link
Contributor

fabOnReact commented Jun 6, 2024

I quote Skalakid comment.

After a quick investigation I found out that the live markdown parser is using a different commit of expensify-common than E/App. Live Markdown parser uses a newer version of ExpeniMark that contains changes that block blockquote creation without space after

I will create PR that bumps the version in the Expensify App
I pulled the latest main and E/App has a newer version of expensify-common than live markdown. I can't reproduce this issue now

57d8b2b

Maybe with the latest version of expensify-common and react-native-live-markdown the issue will not reproduce anymore. I will verify this on Monday (unluckily now I have other commitments). Thanks

@melvin-bot melvin-bot bot added the Overdue label Jun 6, 2024
Copy link

melvin-bot bot commented Jun 7, 2024

@bondydaa, @jliexpensify, @ishpaul777 Whoops! This issue is 2 days overdue. Let's get this updated quick!

@bondydaa
Copy link
Contributor

Thanks @tomekzaw and @fabOnReact let me know if you need anything from me.

@fabOnReact
Copy link
Contributor

fabOnReact commented Jun 11, 2024

I tested and I can not reproduce this issue anymore.

CLICK TO OPEN TESTS RESULTS

Screen.Recording.2024-06-11.at.11.59.08.AM.mov

CLICK TO OPEN TESTS RESULTS

@ishpaul777
Copy link
Contributor

@jliexpensify Can we release bounty for the issue please

Copy link

melvin-bot bot commented Jun 11, 2024

@bondydaa, @jliexpensify, @ishpaul777 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@jliexpensify
Copy link
Contributor

There wasn't an actual regression here right? Also, what's the original PR? Melvin hasn't updated the date so just want to check everything has been deployed.

@melvin-bot melvin-bot bot removed the Overdue label Jun 12, 2024
Copy link

melvin-bot bot commented Jun 12, 2024

@bondydaa @jliexpensify @ishpaul777 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@jliexpensify
Copy link
Contributor

jliexpensify commented Jun 12, 2024

PR - #42843

Will pay out today, confirmed with Bondy no checklist needed. Payment summary.

@jliexpensify
Copy link
Contributor

Paid and job closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor
Projects
No open projects
Archived in project
Development

No branches or pull requests

10 participants