Skip to content

[Due for payment 2025-03-17] [Due for payment 2025-03-13] LHN - FAB does not close when clicking FAB button again when FAB is opened #57797

Closed

Description

@lanitochka17

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


Version Number: 9.1.9-0
Reproducible in staging?: Y
Reproducible in production?: N
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): applausetester+ps309@applause.expensifail.com
Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Go to the report page (search page)
  3. Click FAB
  4. Click FAB again (now X button)

Expected Result:

FAB closes (production behavior)

Actual Result:

Nothing happens after clicking FAB button when FAB is already opened

Workaround:

Unknown

Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence
Bug6761270_1741128000831.Screen_Recording_2025-03-05_at_05.56.59.mp4

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @sonialiap

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    DeployBlockerCashThis issue or pull request should block deployment
    and removed
    BugSomething is broken. Auto assigns a BugZero manager.
    on Mar 4, 2025
  2. melvin-bot commented on Mar 4, 2025

    @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 Mar 4, 2025

    @melvin-bot

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

  4. melvin-bot commented on Mar 4, 2025

    @melvin-bot

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

  5. github-actions commented on Mar 4, 2025

    @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.
  6. 16 remaining items

  7. tgolen commented on Mar 7, 2025

    @tgolen
    Contributor

    I think can be closed now. It's not subject to the regression period since the result was reverting the original PR.

  8. changed the title [-][Due for payment 2025-03-13] LHN - FAB does not close when clicking FAB button again when FAB is opened[/-] [+][Due for payment 2025-03-17] [Due for payment 2025-03-13] LHN - FAB does not close when clicking FAB button again when FAB is opened[/+] on Mar 10, 2025
  9. melvin-bot commented on Mar 10, 2025

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.10-6 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 2025-03-17. 🎊

  10. rushatgabhane commented on Apr 2, 2025

    @rushatgabhane
    Member

    @sonialiap requesting payment summary for review of the linked PR, thank you!
    #57890 (comment)

  11. garrettmknight commented on Apr 10, 2025

    @garrettmknight
    Contributor

    @sonialiap can you help me out with a payment summary, please?

  12. sonialiap commented on Apr 10, 2025

    @sonialiap
    Contributor

    Payment summary:

    EDIT: updating to $250 since Rushat confirmed that he did test the PR

    cc @garrettmknight

  13. rushatgabhane commented on Apr 11, 2025

    @rushatgabhane
    Member

    i think it should be standard amount because i did test the PR with tests for FAB

  14. sonialiap commented on Apr 11, 2025

    @sonialiap
    Contributor

    Thanks Rushat. Updating the summary to $250

  15. sonialiap commented on Apr 11, 2025

    @sonialiap
    Contributor

    Updated payment summary:

    cc @garrettmknight

  16. garrettmknight commented on Apr 22, 2025

    @garrettmknight
    Contributor

    $250 approved for @rushatgabhane

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

Metadata

Metadata

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions