[$1000] Jump to last unread chat when opening a report

[kbecciv]

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: 1.4.28.0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @MonilBhavsar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1705558628672569

Action Performed:

1. A is participant of a chat or room
2. As another user send some good number of messages to chat report, so new messages can be scrolled up and down
3. As user A, open the chat report

Expected Result:

The chat report view opens from where the messages are unread and user can scroll down

Actual Result:

The chat report view is scrolled down to the last message and user needs to scroll up to find last read message

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

Add any screenshot/video evidence

Recording.5931.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01fc144b2f2f5df388
• Upwork Job ID: 1749920553887232000
• Last Price Increase: 2024-06-11
• Automatic offers:
• ikevin127 | Contributor | 0
• ishpaul777 | Contributor | 102670752

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[jeremy-croff]

This PR #28793 specifically makes it scrolled to the bottom, seems like NAB

[trjExpensify]

I'm a bit confused by the bug report @MonilBhavsar, so asking about it here. I thought this was something we'd implement via comment linking.

[roryabraham]

HOLD for comment linking: #30269

[melvin-bot]

Current assignee @trjExpensify is eligible for the NewFeature assigner, not assigning anyone new.

[trjExpensify]

No change, still on hold Melv!

[trjExpensify]

Samesies, Melv!

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.88-7 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Move logic to start report actions list at index to BaseInvertedFlatList #52149

If no regressions arise, payment will be issued on 2025-01-30. 🎊

For reference, here are some details about the assignees on this issue:

• @janicduplessis does not require payment (Contractor)
• @ishpaul777 requires payment automatic offer (Contributor)

[melvin-bot]

BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [@ishpaul777] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
• [@trjExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

[rlinoz]

We need one more PR here, so no payment just yet

[janicduplessis]

Had to deal with a little regression from #52149, last PR should be done Friday or Monday.

[trjExpensify]

How we looking here?

[rlinoz]

FE Draft PR is here #51366

But we will need a backend PR to continue, I will work on that today

[rlinoz]

PRs in review

[rlinoz]

Last PR Web-E PR was merged today, so we probably can continue with the PR tomorrow @janicduplessis

[rlinoz]

@janicduplessis PR is in staging

[ishpaul777]

PR is WIP, @janicduplessis can you please update the status on when can we expect it to open, TY!

[janicduplessis]

Working on last adjustments as per #51366 (comment), then will need to do a good round of testing. If all goes well I am expecting it to be ready by start of next week.

[trjExpensify]

Great stuff!

[rlinoz]

Hey @janicduplessis any updates here?

[janicduplessis]

Sorry about the delay, I didn't end up having much time to work on this this week. It should be ready by early next week :)

[janicduplessis]

Still working on this, I hit a little issue where for some reason the first unread action is now wrong. It seems to be a regression since it happened after merging main. Trying to pinpoint what caused this.

[janicduplessis]

Good news, after updating to latest main the regression is gone. PR #51366 is now ready for review and further testing!

[rlinoz]

PR is being reviewed, looks like there is one outstanding bug reported here #51366 (comment)