-
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
[$500] Android - Chat - User suggestion list is not opening on entering @ in 2nd line. #37660
Comments
Triggered auto assignment to @trjExpensify ( |
@trjExpensify 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 |
We think that this bug might be related to #vip-vsp |
@lanitochka17 can we get the device and Android OS please? I'm trying to reproduce this on a Pixel 6, Android 12 using Browserstack and can't seem to. Thanks! |
@trjExpensify Hello |
Got it. Is it device specific? I don't have Redmi note 10s in Browserstack. I've tried in addition with Redmi 12 pro (Android 12) and Redmi 12 4G (Android 13), both of which I can't reproduce. Can we have it re-tested and also use a couple of different Android devices please, thanks! |
Awaiting the retest. |
Issue is still reproducible on the latest build 1.4.50-2 Record_2024-03-11-17-13-41_4f9154176b47c00da84e32064abf1c48.mp4Issue is still reproducible on the latest build 1.4.50-2 Screen_Recording_20240311_231829_New.Expensify.mp4 |
Thanks! |
@trjExpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
Job added to Upwork: https://www.upwork.com/jobs/~01c6808254765a233b |
Doesn't look like anyone in the thread can reproduce it on the devices they've used. @shubham1206agra thinks it might be something to do with Live Markdown, but it's a bit TBD on why only certain devices are impacted. |
@trjExpensify Contributor has mentioned here that issue is reproducible on the slow devices, though I have tested on 2 device and both worked well both has Android version 13, so not sure whats the next step here, should we try to fix to fix this issue for the some devices ? |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Do we have an idea of the root cause? As in, is it an issue on all devices under certain conditions? |
@trjExpensify No, issue reproducible conditions are vague, as issue reproducible for some devices (not for the exact android version or particular mobile brands), and to be honest I haven't faced issue in any of the devices that I have tested recently |
Hi @cubuspl42, @Gonals, @bernhardoj Can you please check if this issue is also reproducible? Here are the steps to reproduce the issue:
Screenrecordfasttype.mp4
Screenrecordslowmobile.mp4 |
I can reproduce it. Screen.Recording.2024-03-28.at.11.38.42.mov |
Thanks for the checking @bernhardoj |
Thanks for the help @bernhardoj , finally I was able to reproduce this issue using this emulator. |
@DrLoopFall's Proposal of passing the conditional leading and maxWait props and using lodashDefer to useCallback will solve this issue for the slow devices. Compared before and after applying the lodashDefer in Pixel 4a, it solves the current issue 🎀 👀 🎀 C+ reviewed |
Triggered auto assignment to @madmax330, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@madmax330 you good with this proposal? |
@madmax330, @trjExpensify, @jayeshmangwani Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Issue not reproducible during KI retests. (First week) |
Oh, interesting. Can you guys still reproduce? @madmax330 we're also awaiting the secondary proposal review here. |
Sorry, this was directed to @DrLoopFall & @jayeshmangwani! Let me know and we can close here if it's resolved. |
@trjExpensify From the latest Play Store build, I was able to reproduce this issue only once after testing it for 10-15 minutes. I am not able to reproduce this issue in the latest main branch; before, I was able to reproduce it easily from my local build, but now the problem is not happening to me. |
@trjExpensify, I'm also unable to reproduce this issue in the main branch. |
Okay, cool. Let's close it out! |
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.46
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: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
User suggestion list must be opened on entering @ in 2nd line
Actual Result:
User suggestion list is not opening on entering @ in 2nd line. It's slightly opening and closed again
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6399593_1709396827882.nv.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: