-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD for payment 2023-05-16] [$1000] App displays fixed set of results for few seconds and closes when using symbols like (,@,?,# on first open in address1 field of home address #16367
Comments
Triggered auto assignment to @johncschuster ( |
Bug0 Triage Checklist (Main S/O)
|
@johncschuster Eep! 4 days overdue now. Issues have feelings too... |
@johncschuster Still overdue 6 days?! Let's take care of this! |
@johncschuster Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it! |
@johncschuster 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! |
@johncschuster 12 days overdue now... This issue's end is nigh! |
This issue has not been updated in over 14 days. @johncschuster eroding to Weekly issue. |
@johncschuster this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
Triggered auto assignment to @lschurr ( |
John is OOO, reassigning so this gets actioned. thanks! |
Bug0 Triage Checklist (Main S/O)
|
I'm not able to reproduce this. When I enter a symbol, I get no suggested results at all. |
Triggered auto assignment to @flodnv ( |
@flodnv are you able to reproduce this? |
@Pujan92 Thanks for the proposal. Your RCA is correct. However I think the fix should be slightly altered, I think it's intended to use an object for the Also please hide your first proposal to avoid confusion. 🎀 👀 🎀 C+ reviewed cc @flodnv |
@s77rt Thanks for your comment. I don't actually understand your mean. Why do we create the _request function only once? I don't see the author using useCallback for Screen.Recording.2023-04-29.at.22.14.27.mp4The _requests is always empty, but if I move it outside of the component, it's not empty. Please pardon me if I'm wrong something. |
To use the first initialised Actually we don't really create the function only once I said once just to make the idea clear. We do re-create it (
We are using
I think my first points clarifies this. Let me know if this is still unclear. |
@s77rt Thanks so much. I totally understand your mean |
Makes sense to restrict the creation of new ref from our side. Thank @s77rt |
@MelvinBot is OOO 😅. 📣 @Pujan92 You have been assigned to this job by @flodnv! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.12-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-05-16. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
|
@dhanashree-sawant @Pujan92 @s77rt - please apply to the job in Upwork if you haven't already: https://www.upwork.com/jobs/~017c127888b5f727a7 |
It looks like the job posting expired - @dhanashree-sawant @Pujan92 @s77rt can you apply here: https://www.upwork.com/jobs/~019c7c7b19cc3a8141 |
@lschurr Applied. |
Applied! |
Hi @lschurr, thanks I have applied too |
Paid and closing! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
If no results are present for the entry, app shouldn't display any result and if there are results for the entry, app should display those results
Actual Result:
App displays fixed set of result everytime for symbols and when trying with any letter, it again displays same fixed set of result first for 1-2 seconds before displaying actual results
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.88-0
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
Notes/Photos/Videos:
Recording.17.mp4
results.issue.home.address.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1679409645222509
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: