-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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] mWeb/Safari - Chat - Space appears above chat header after opening and closing attachment #39065
Comments
Triggered auto assignment to @anmurali ( |
👋 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:
|
Triggered auto assignment to @AndrewGable ( |
@anmurali 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. |
Production RPReplay_Final1711549477.MP4 |
#36658 looks like it could be suspect, can someone confirm? cc @SzymczakJ @puneetlath @s77rt @kubabutkiewicz @BartoszGrajdek |
Seems likely. Can you please take a look @SzymczakJ? |
I was not able to reproduce. I doubt it's related to the attachment carousel. |
Reproduce at all? |
@AndrewGable I was also not able to reproduce, both locally and on staging.new.expensify.com |
OK thanks - @izarutskaya can you try to reproduce once more? And try to find more detailed reproduction steps? |
Agree - I don't see the banner either, we should attempt to reproduce while the banner is up. |
Job added to Upwork: https://www.upwork.com/jobs/~01ac4eb38e7291bc5c |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @dukenv0307 ( |
I can't reproduce this either. How do I setup an account such that there is a banner shown on mWeb? |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
1 similar comment
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
I believe you have to go to mWeb on a device that doesn't have the iOS app installed |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
|
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (First week) |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
If it's not reproducible anymore, let's just close. It's been hard to reproduce and track down to begin with. |
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.57-2
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
There should be no extra space at the top above the chat header and below the suggestion to open the new.expensify in the app
Actual Result:
There is an extra space appears above the chat header and below the suggestion to open the new.expensify in the app
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6428725_1711547758276.RPReplay_Final1711547352.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @anmuraliThe text was updated successfully, but these errors were encountered: