Older report comments are not loaded once we scroll to the top - reported by @santhosh-sellavel

[chiragsalian]

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

---

Action Performed:

1. Go to https://staging.new.expensify.com
2. Click on any search that contains a good amount of history.
3. Scroll up to the top.

Expected Result:

Once we reach the top more comments should have loaded.

Actual Result:

No comments are loaded once we scroll to the top. So for a chat we can only see around 50-100 comments and not more.

Workaround:

Unkown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.60-0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

Logs: https://expensify.slack.com/archives/C01GTK53T8Q/p1652463943094999

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Issue reported by: @Santhosh-Sellavel

Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1652463943094999

View all open jobs on GitHub

[chiragsalian]

cc @thesahindia

[OSBotify]

👋 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.

[melvin-bot]

Triggered auto assignment to @Luke9389 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[chiragsalian]

Its being worked on here

[thesahindia]

PR is up #9001.

[melvin-bot]

Current assignee @michaelhaxhiu is eligible for the External assigner, not assigning anyone new.

[chiragsalian]

PR has been merged
@michaelhaxhiu, feel free to compensate @Santhosh-Sellavel for finding the issue and @thesahindia for helping resolve the issue when appropriate.

[chiragsalian]

Reopening since i'll ask QA to test when this is on staging.

[roryabraham]

Looks like this passed QA

[michaelhaxhiu]

PR has been merged @michaelhaxhiu, feel free to compensate @Santhosh-Sellavel for finding the issue and @thesahindia for helping resolve the issue when appropriate.

Yep! So here's what we should have:

@Santhosh-Sellavel - $250 for reporting the bug
@thesahindia - $250 for assisting in finding/fixing the deploy blocker

Can you both please apply to the upwork job here

[Santhosh-Sellavel]

Applied, thanks!

[thesahindia]

Done ✅

[michaelhaxhiu]

Paid both!