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] Expense - Recipient list auto scrolls down to reveal only "Contacts" after clearing search field #55715

Open
6 of 8 tasks
vincdargento opened this issue Jan 24, 2025 · 19 comments
Assignees
Labels
Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@vincdargento
Copy link

vincdargento commented Jan 24, 2025

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: 9.0.89-2
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 15.0 / Chrome
App Component: Other

Action Performed:

Precondition:

  • Account has many contacts (Expensifail account).
  • There are a few contacts under Recent section in contact list (create chat with a few users to have some contacts under Recent section).
  1. Go to staging.new.expensify.com
  2. Open FAB > Create expense > Manual.
  3. Enter amount > Next.
  4. Type search term that will return a few members in the search result.
  5. Clear the search field.

Expected Result:

The recipient list will not scroll down after clearing the search field.

Actual Result:

The recipient list auto scrolls down to reveal only "Contacts" after clearing the search field.

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

bug.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021882867353031260828
  • Upwork Job ID: 1882867353031260828
  • Last Price Increase: 2025-01-31
Issue OwnerCurrent Issue Owner: @Pujan92
@vincdargento vincdargento added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 DeployBlockerCash This issue or pull request should block deployment labels Jan 24, 2025
Copy link

melvin-bot bot commented Jan 24, 2025

Triggered auto assignment to @laurenreidexpensify (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.

Copy link

melvin-bot bot commented Jan 24, 2025

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

Copy link

melvin-bot bot commented Jan 24, 2025

💬 A slack conversation has been started in #expensify-open-source

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Jan 24, 2025
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.

@vincdargento
Copy link
Author

@laurenreidexpensify 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.

@srikarparsi
Copy link
Contributor

Adding the external label

@srikarparsi srikarparsi added the External Added to denote the issue can be worked on by a contributor label Jan 24, 2025
@melvin-bot melvin-bot bot changed the title Expense - Recipient list auto scrolls down to reveal only "Contacts" after clearing search field [$250] Expense - Recipient list auto scrolls down to reveal only "Contacts" after clearing search field Jan 24, 2025
Copy link

melvin-bot bot commented Jan 24, 2025

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 24, 2025
Copy link

melvin-bot bot commented Jan 24, 2025

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

@Pujan92
Copy link
Contributor

Pujan92 commented Jan 24, 2025

I am not able to reproduce, @vincdargento can you plz retest and confirm

@mountiny
Copy link
Contributor

I cannot reproduce either and its a minor issue

@melvin-bot melvin-bot bot added the Overdue label Jan 27, 2025
@mountiny mountiny added Daily KSv2 and removed Engineering Hourly KSv2 Overdue Bug Something is broken. Auto assigns a BugZero manager. labels Jan 27, 2025
@melvin-bot melvin-bot bot added the Overdue label Jan 27, 2025
@laurenreidexpensify
Copy link
Contributor

Waiting for @vincdargento to retest

@vincdargento
Copy link
Author

@laurenreidexpensify Tester can reproduce when the search result list is scrollable, not reproducible when search result list is not scrollable

20250128_121627.mp4

@yuwenmemon
Copy link
Contributor

yuwenmemon commented Jan 29, 2025

Removing the blocker label for this one, as it's quite minor and shouldn't hold up the deploy.

@yuwenmemon yuwenmemon removed the DeployBlockerCash This issue or pull request should block deployment label Jan 29, 2025
Copy link

melvin-bot bot commented Jan 31, 2025

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

Copy link

melvin-bot bot commented Jan 31, 2025

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

@laurenreidexpensify
Copy link
Contributor

Let's wait for some more proposals to come in

Copy link

melvin-bot bot commented Feb 4, 2025

@Pujan92, @srikarparsi, @laurenreidexpensify Still overdue 6 days?! Let's take care of this!

Copy link

melvin-bot bot commented Feb 6, 2025

@Pujan92, @srikarparsi, @laurenreidexpensify 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

@laurenreidexpensify
Copy link
Contributor

Still waiting on proposals here

@melvin-bot melvin-bot bot removed the Overdue label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

6 participants