[$250] [Search v2.2] Chat is scrolled to the bottom when refreshing the page

[IuliiaHerets]

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: v9.0.31-0
Reproducible in staging?: Y
Reproducible in production?: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

1. Log in to Newdot
2. Go to Search option from bottom tab
3. Click on "Chat" type from LHP
4. Click on any message which is not at the bottom of a chat report
5. When the report view is opened in RHP refresh the browser
6. Now notice how the chat report is scrolled to the bottom

Expected Result:

The chat report in RHP is not scrolled down to the bottom and stays in the same report action opened before refreshing the page

Actual Result:

The chat report in RHP is scrolled down to the bottom and user has to scroll up to find the chat report opened before refreshing the page

Workaround:

Unknown

Platforms:

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Bug6597778_1725902355625.2024-09-09_19_35_43.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021833217065296959946
• Upwork Job ID: 1833217065296959946
• Last Price Increase: 2024-09-16
• Automatic offers:
• ikevin127 | Contributor | 103886648
• huult | Contributor | 104028956

Issue Owner

Current Issue Owner: @rojiphil

[melvin-bot]

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

[melvin-bot]

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

[IuliiaHerets]

We think that this bug might be related to #wave-control

[github-actions]

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

[luacmartins]

I'll take this since it's part of Search

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Upwork job price has been updated to $125

[luacmartins]

We should navigate the user to the reportActionID as well, not only the report

[luacmartins]

@shubham1206agra

[melvin-bot]

📣 @ikevin127 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[melvin-bot]

📣 @huult 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[huult]

Thank you for everything. I will create a pull request for this ticket tonight.

[Pujan92]

The report action list is missing the initialScrollIndex in the InvertedFlatList.

@huult, I was just wondering if this is the issue then how it is working for the central pane ReportActionsList

[huult]

Proposal updated

• Added a new solution

Hi @ikevin127 , While creating the pull request, I found a better solution. The pull request is already open. Can you check it? Thanks!

[ikevin127]

Thanks @Pujan92!

I was just wondering if this is the issue then how it is working for the central pane ReportActionsList

@huult This is a valid question when it comes to the RCA, which I wasn't able to figure out.

I checked out your updated proposal and to be honest I'm reluctant when it comes to adding more useEffects / timeouts in a component which is already complex because these can degrade performance.

I think the best way to go here would be to dig into why the link / scroll to message works when the list is in central pane but not when it's in RHP.

If you're wiling to dig deeper and update your proposal to expand on the RCA and fix the issue such that it would work in RHP as well please let us know - otherwise we can re-open this for proposals 👍

[huult]

Proposal updated

• Updated RCA
• Updated solutions to fix this issue
• Added new POC based on the new solution

@ikevin127, I've updated the proposal. Can you help check it again? Thank you!

[ikevin127]

@huult 🟢 The latest proposal update, RCA and solution are the perfect fix here, I tested it and can confirm that it works as expected without any scroll jumps, works exactly like on the central pane on browser refresh 👍

You can proceed to implementing the latest solution in the opened PR and tag me there once it's ready for review.

Notes for PR:

• the onStartReached function could become:

const onStartReached = useCallback(() => {
    InteractionManager.runAfterInteractions(() => requestAnimationFrame(() => loadNewerChats(false)));
}, [loadNewerChats]);

which is preferred over the inline function suggested because of performance reasons.

[alexpensify]

@ikevin127 are we waiting for an update here? Sorry, I don't see the extra PR. Thanks!

[ikevin127]

@alexpensify PR #49477 for this issue was just deployed to production, no extra PR needed. ⚠️ Automation failed, this should be on [HOLD for Payment 2024-10-04] according to today's production deploy from #49477 (comment).

[alexpensify]

Thank you for flagging the payment date and clarifying the status.

[alexpensify]

Payouts due: 10-04-24

• Contributor: $250 @huult
• Contributor+: $250 @ikevin127

Upwork job is here.

Closing - I believe that I sent the payment via Upwork, but something looks off in Upwork. Please reply if you didn't get the payment and go through the payment process again.

[ikevin127]

@alexpensify I only got $125, was that on purpose or a mistake ?

[alexpensify]

No, there was an issue in Upwork, so I input a Bonus too. Let me know if that second payment didn't go through. Here is what I see on my side:

[ikevin127]

@alexpensify Indeed, Upwork is acting weird today (reported in other issues too).

So far nothing changed, I got $125 for this issue, looks same as #48808 (comment) and in Overwiew I only see the $125 transaction:

Contract details look like this right now:

[alexpensify]

Ok, I have a note to check again on Monday. Maybe there is a delay or something on the Upwork side. All my payment states are not updating correctly in Upwork today. If there is still an issue on Monday, I'll reach out to their support since I input a Bonus to correct the price issue.

[ikevin127]

Understood!

For context: the same thing happened in this issue: #48161 and the BZ wasn't able to do anything with the payment once the initial contract was ended without me being paid anything, so I'm assuming that your additional $125 bonus did not go through once the initial contract was ended after the $125 payment.

In the other issue, the BZ sent me another contract saying:

No idea what happened there. It doesn't show as being paid either. I'll send you another offer, but if you happen to get a rogue $125 show up - I'll trust your honesty to let me know. :)

To which I replied with:

Sure, I accepted the new contract and will let you know in case it comes twice 👍

[huult]

@alexpensify , Can you also check for me on Upwork? My contract for this ticket has ended, but the status on Upwork is still 'work in progress. thank you

[ikevin127]

@alexpensify I still did not get the other $125, only got half of the payment for this issue so far.

[alexpensify]

Thanks for these updates! There is an issue an Upwork that is being investigated to address the payment process. I'm going to hold until there is a fix. Convo: https://expensify.slack.com/archives/C01GTK53T8Q/p1728319262830679

[alexpensify]

@huult and @ikevin127 - our team figured out the Upwork bug, and I tried to complete the payment via another payment path in Upwork. Their support team is working on a permanent fix for the bug. I'd really appreciate it if you could confirm today's payment was sent successfully. Thanks!

[ikevin127]

✅ Confirmation that I just got the other $125, thanks Al !

[huult]

It was sent to me. Thank you, @alexpensify

[alexpensify]

Awesome, thank you for confirming! This bug is a wild one on the Upwork side, so I appreciate your patience.