Skip to content
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

Scan - Header in 1:1 DM turns to skeleton after creating Scan request #39163

Closed
6 tasks done
izarutskaya opened this issue Mar 28, 2024 · 13 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2

Comments

@izarutskaya
Copy link

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:

  1. Go to staging.new.expensify.com.
  2. Go to chat with user that has no chat history.
  3. Create a scan request.
  4. Immediately after requesting, tap on the IOU preview twice to go to transaction thread.
  5. Wait until the receipt is uploaded, which is when the receipt preview reloads.
  6. Swipe right (iOS app) to return to 1:1 DM.

Expected Result:

The header in 1:1 DM will show the user name and avatar.

Actual Result:

The header in 1:1 DM turns to skeleton loading.

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

Bug6429898_1711617204516.RPReplay_Final1711616962.mp4

View all open jobs on GitHub

@izarutskaya izarutskaya added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 28, 2024
Copy link

melvin-bot bot commented Mar 28, 2024

Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Mar 28, 2024
Copy link
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.

Copy link

melvin-bot bot commented Mar 28, 2024

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

@izarutskaya
Copy link
Author

@muttmuure 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.

@izarutskaya
Copy link
Author

We think this issue might be related to the #vip-bills.

@izarutskaya
Copy link
Author

Production

Record_2024-03-28-12-26-21_4f9154176b47c00da84e32064abf1c48.mp4

@yuwenmemon
Copy link
Contributor

Hmmm... not able to reproduce so far in staging

@yuwenmemon
Copy link
Contributor

RPReplay_Final1711661027.MP4

Can't reproduce on staging on iOS. Gonna remove the blocker label.

@yuwenmemon yuwenmemon added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Mar 28, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 1, 2024
@Julesssss
Copy link
Contributor

Hey @muttmuure, I can't repro this on the staging Android app. Could you please check iOS?

@melvin-bot melvin-bot bot removed the Overdue label Apr 2, 2024
Copy link

melvin-bot bot commented Apr 5, 2024

@Julesssss, @muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@Julesssss
Copy link
Contributor

Bump on this @muttmuure

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 8, 2024
@Julesssss Julesssss added Weekly KSv2 and removed Daily KSv2 labels Apr 11, 2024
@melvin-bot melvin-bot bot removed the Overdue label Apr 11, 2024
Copy link

melvin-bot bot commented Apr 11, 2024

@Julesssss @muttmuure this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Apr 19, 2024
@muttmuure
Copy link
Contributor

Can't repro, closing

@melvin-bot melvin-bot bot removed the Overdue label Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants