Skip to content

[HOLD for payment 2024-07-22] [HOLD for payment 2024-07-17] Search - Empty list does not turn to empty state screen until page is revisited #44803

Closed

Description

@m-natarajan

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Issue found when validating #44385
Version Number: 9.0.4-0
Reproducible in staging?: y
Reproducible in production?: no
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:

Precondition:

  • User has only unpaid expenses.
  1. Go to staging.new.expensify.com
  2. Go to Search > Shared.
  3. Select all the expenses.
  4. Click on the dropdown > Delete.
  5. Delete the expenses.

Expected Result:

The empty state screen will show up after deleting all the expenses.

Actual Result:

The list is empty with field row after deleting all the expenses.
It only shows empty state screen after revisiting the page.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

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

Bug6532525_1720043955916.20240704_055548.mp4

View all open jobs on GitHub

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    DeployBlockerIndicates it should block deploying the API
    on Jul 4, 2024
  2. melvin-bot commented on Jul 4, 2024

    @melvin-bot

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

  3. melvin-bot commented on Jul 4, 2024

    @melvin-bot

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

  4. github-actions commented on Jul 4, 2024

    @github-actions
    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.
  5. m-natarajan commented on Jul 4, 2024

    @m-natarajan
    Author

    @MariaHCD 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-vsb

  6. kosmydel commented on Jul 4, 2024

    @kosmydel
    Contributor

    Hey, I'm from Software Mansion and I will investigate this issue.

  7. MariaHCD commented on Jul 4, 2024

    @MariaHCD
    Contributor

    cc: @luacmartins @WojtekBoman for visibility

  8. 28 remaining items

  9. sonialiap commented on Jul 18, 2024

    @sonialiap
    Contributor

    @MariaHCD @luacmartins I see three people on the PR, two are freelancers. I see fedirjh completed the checklist and neonbhai left a comment. Should they both be paid for the review?

  10. MariaHCD commented on Jul 18, 2024

    @MariaHCD
    Contributor

    I believe only fedirjh should be compensated for their C+ review. Will let @luacmartins confirm.

  11. luacmartins commented on Jul 18, 2024

    @luacmartins
    Contributor

    Correct, we only need to compensate @fedirjh for the review!

  12. sonialiap commented on Jul 22, 2024

    @sonialiap
    Contributor

    Payment summary:

  13. melvin-bot commented on Jul 22, 2024

    @melvin-bot

    Payment Summary

    Upwork Job

    • Reviewer: @fedirjh owed $250 via NewDot

    BugZero Checklist (@sonialiap)

    • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
    • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
    • I have paid out the Upwork contracts or cancelled the ones that are incorrect
    • I have verified the payment summary above is correct
  14. JmillsExpensify commented on Jul 22, 2024

    @JmillsExpensify
    Contributor

    $250 approved for @fedirjh

  15. sonialiap commented on Jul 24, 2024

    @sonialiap
    Contributor

    Last step here is the checklist

    @fedirjh I believe this falls to you as the C+?

  16. fedirjh commented on Jul 24, 2024

    @fedirjh
    Contributor

    BugZero Checklist:

    • Link to the PR: [Search v1] Add bulk actions #44385
    • Link to comment: Author was aware and fixed it
    • Link to discussion: N/A
    • Determine if we should create a regression test for this bug: Yes

    Regression Test Proposal

    Precondition: User has only unpaid expenses.
    
    1. Go to Search -> Shared.
    2. Select all the expenses.
    3. Click on the dropdown > Delete.
    4. Delete the expenses.
    5. Verifythat  empty state screen will show up after deleting all the expenses.
    
    • Do we agree 👍 or 👎
  17. sonialiap commented on Jul 25, 2024

    @sonialiap
    Contributor

    Thanks Fedi!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2Engineering

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions