Skip to content

[$250] Report - Report view shows skeleton during each visit #58805

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

Closed
6 of 8 tasks
lanitochka17 opened this issue Mar 20, 2025 · 9 comments
Closed
6 of 8 tasks

[$250] Report - Report view shows skeleton during each visit #58805

lanitochka17 opened this issue Mar 20, 2025 · 9 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Hourly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Mar 20, 2025

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 9.1.16-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
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): [email protected]
Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Go to any report
  3. Go to another report
  4. Switch back and forth between reports

Expected Result:

If the report is opened previously, it should not show skeleton after the subsequent visits

Actual Result:

When the report is opened previously, it stll shows skeleton after the subsequent visits

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
Bug6777006_1742484711621.20250320_232332.mp4

This issue happens when submitting expense and closing receipt view
https://github.com/user-attachments/assets/b56056b0-e5ab-4919-9f58-30528c541ffb

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021902762585241549475
  • Upwork Job ID: 1902762585241549475
  • Last Price Increase: 2025-03-20
Issue OwnerCurrent Issue Owner: @ZhenjaHorbach
@lanitochka17 lanitochka17 added DeployBlocker Indicates it should block deploying the API DeployBlockerCash This issue or pull request should block deployment labels Mar 20, 2025
Copy link

melvin-bot bot commented Mar 20, 2025

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

Copy link

melvin-bot bot commented Mar 20, 2025

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

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.

@marcochavezf marcochavezf added the External Added to denote the issue can be worked on by a contributor label Mar 20, 2025
@melvin-bot melvin-bot bot changed the title Report - Report view shows skeleton during each visit [$250] Report - Report view shows skeleton during each visit Mar 20, 2025
Copy link

melvin-bot bot commented Mar 20, 2025

Job added to Upwork: https://www.upwork.com/jobs/~021902762585241549475

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Mar 20, 2025
Copy link

melvin-bot bot commented Mar 20, 2025

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

@nkdengineer
Copy link
Contributor

Proposal

Please re-state the problem that we are trying to solve in this issue.

When the report is opened previously, it stll shows skeleton after the subsequent visits

What is the root cause of that problem?

This comes from #58654.

We always show the skeleton view when the OpenReport API is called.

if (isLoadingInitialReportActions && !isOffline) {
return <ReportActionsSkeletonView />;
}

What changes do you think we should make in order to solve the problem?

We need to undo this change.

if (isLoadingInitialReportActions && !isOffline) {
return <ReportActionsSkeletonView />;
}

For this bug #58388, it's currently a backend bug. After we request money, the backend returns reportID of transaction thread report as number then this function returns early until OpenReport API of the expense report is complete. If we want to a hot fix we can update this to

if (!transactionThreadReportID || isSentMoneyReport) { 
     return reportActions; 
 }

if (isEmpty(transactionThreadReportID) || isSentMoneyReport) {
return reportActions;
}

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

None

What alternative solutions did you explore? (Optional)

Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

@lanitochka17
Copy link
Author

This issue happens when submitting expense and closing receipt view

20250321_004746.mp4

@marcochavezf
Copy link
Contributor

@marcochavezf
Copy link
Contributor

This was already reverted, tested on staging and the bug is gone.

@marcochavezf marcochavezf removed DeployBlockerCash This issue or pull request should block deployment DeployBlocker Indicates it should block deploying the API labels Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Hourly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants