search

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.33k stars 2.76k forks source link

Hybrid app- The LHN on hybrid app is wiggling #48434

Closed m-natarajan closed 6 days ago

m-natarajan commented 1 week ago

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.0.27-0 Reproducible in staging?: Needs reproduction Reproducible in production?: Needs reproduction 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: @quinthar Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1725302051557489

Action Performed:

  1. Launch the hybrid app and login
  2. Observe the LHN (Device tested : Pixel 8, Android 14)

    Expected Result:

    No visual issues

    Actual Result:

    LHN on the app is wiggling

    Workaround:

    Unknown

    Platforms:

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

    • [x] Android: Native
    • [ ] Android: mWeb Chrome
    • [ ] iOS: Native
    • [ ] iOS: mWeb Safari
    • [ ] MacOS: Chrome / Safari
    • [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/bb7d944d-4d24-45f0-a641-32d4800c470a

View all open jobs on GitHub

melvin-bot[bot] commented 1 week ago

Triggered auto assignment to @Gonals (AutoAssignerNewDotQuality)

melvin-bot[bot] commented 1 week ago

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

MelvinBot commented 1 week ago

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

trjExpensify commented 1 week ago

Asking in thread for the Android device and version number. I don't have a physical android device, but I can try and repro on the emulator perhaps.

melvin-bot[bot] commented 1 week ago

Triggered auto assignment to @srikarparsi (AutoAssignerNewDotQuality)

Gonals commented 1 week ago

Currently focusing on some travel issues I've been told to prioritize, so rerolling for another engineer!

trjExpensify commented 1 week ago

Alright, I tried testing this on browserstack using the release build in the OP on a Pixel 8, Android 14. I can't get the wiggler.

Note: I had to figure out how to do that, as the HybridApp builds aren't automatically added to browserstack. Added steps in an SO.

muttmuure commented 6 days ago

Checking if this is still happening in thread

trjExpensify commented 6 days ago

Sounds good, seems like this isn't reproducible anymore per the thread. We closing, Matt?

muttmuure commented 6 days ago

I think we can close it. SWM are doing a bunch of refactors so it's inevitable that some of these don't last long