Skip to content

[HOLD for payment 2024-04-09] [$500] Expense - Page scrolls back down when dismissing add receipt RHP in transaction thread #38935

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: 1.4.56-2
Reproducible in staging?: Y
Reproducible in production?: N
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):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

Action Performed:

  1. Go to staging.new.expensify.com
  2. Create a manual request without receipt
  3. Go to request details page
  4. Scroll to the top of the page
  5. Click on add receipt thumbnail
  6. Click outside the RHP to dismiss it

Expected Result:

The page will not scroll down when closing the RHP (production behavior)

Actual Result:

The page scrolls down when closing the RHP

Workaround:

Unknown

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

Add any screenshot/video evidence

Bug6425973_1711372872772.20240325_151949.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e4b313e99e0a0eca
  • Upwork Job ID: 1772334661375188992
  • Last Price Increase: 2024-03-25
Issue OwnerCurrent Issue Owner: @mananjadhav

Activity

  1. github-actions commented on Mar 25, 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.
  2. melvin-bot commented on Mar 25, 2024

    @melvin-bot

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

  3. lanitochka17 commented on Mar 25, 2024

    @lanitochka17
    Author

    We think that this bug might be related to #wave-collect - Release 1

  4. lanitochka17 commented on Mar 25, 2024

    @lanitochka17
    Author

    @flodnv 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

  5. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Mar 25, 2024
  6. changed the title [-]Expense - Page scrolls back down when dismissing add receipt RHP in transaction thread[/-] [+][$500] Expense - Page scrolls back down when dismissing add receipt RHP in transaction thread[/+] on Mar 25, 2024
  7. melvin-bot commented on Mar 25, 2024

    @melvin-bot
  8. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Mar 25, 2024
  9. melvin-bot commented on Mar 25, 2024

    @melvin-bot

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

  10. habasefa commented on Mar 25, 2024

    @habasefa

    I couldn't reproduce this on dev but it is still there on staging.

  11. 28 remaining items

  12. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Apr 2, 2024
  13. changed the title [-][$500] Expense - Page scrolls back down when dismissing add receipt RHP in transaction thread[/-] [+][HOLD for payment 2024-04-09] [$500] Expense - Page scrolls back down when dismissing add receipt RHP in transaction thread[/+] on Apr 2, 2024
  14. melvin-bot commented on Apr 2, 2024

    @melvin-bot

    Reviewing label has been removed, please complete the "BugZero Checklist".

  15. melvin-bot commented on Apr 2, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.58-8 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 2024-04-09. 🎊

    For reference, here are some details about the assignees on this issue:

    • @mananjadhav requires payment through NewDot Manual Requests
  16. melvin-bot commented on Apr 2, 2024

    @melvin-bot

    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:

  17. kadiealexander commented on Apr 9, 2024

    @kadiealexander
    Contributor

    @mananjadhav please complete the checklist!

  18. mananjadhav commented on Apr 9, 2024

    @mananjadhav
    Collaborator

    Thanks for the bump @kadiealexander. We already discussed the offending PR with the explanation.

    I don't think we need a regression test for this one.

  19. melvin-bot commented on Apr 9, 2024

    @melvin-bot

    Payment Summary

    Upwork Job

    BugZero Checklist (@kadiealexander)

    • 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/1772334661375188992/hired)
    • I have paid out the Upwork contracts or cancelled the ones that are incorrect
    • I have verified the payment summary above is correct
  20. JmillsExpensify commented on Apr 10, 2024

    @JmillsExpensify
    Contributor

    $500 approved for @mananjadhav based on checklist above.

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.DailyKSv2EngineeringExternalAdded to denote the issue can be worked on by a contributorHelp WantedApply this label when an issue is open to proposals by contributors

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions