[$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

Issue Owner

Current Issue Owner: @roryabraham

[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!

[roryabraham]

Thanks @rlinoz

[rlinoz]

@janicduplessis can I assign you to this one?

[janicduplessis]

@rlinoz Yes!

[janicduplessis]

@rlinoz Is there a way I can test what you mentioned here? I can look at the issues you encountered and then it sounds like this would be solved.

[rlinoz]

@janicduplessis just sent you a DM with how to access that

[rlinoz]

We have draft PRs for backend and frontend

[ishpaul777]

PR still in WIP, @janicduplessis Any updates?

[janicduplessis]

The PR is mostly working already, but there is an edge case that will require some refactoring of ReportActionsView / ReportActionsList so we can reuse the logic to start the list at a specific reportActionID. I will work on this in a separate PR to avoid making the current one giant.

[janicduplessis]

Have a new PR up to refactor how we handle opening the list at a specific index. Before it was mixed with the comment linking code and now it is extracted out to BaseInvertedFlatList so it will make it easy to use for opening the report at last unread message too.

#52149, currently in draft, will test more and complete checklist later today.

[trjExpensify]

@janicduplessis is the PR ready for a re-review from @ishpaul777?

[janicduplessis]

Yes, I have fixed the issue he found

[ishpaul777]

I'll rereview in couple hours

[trjExpensify]

What's the latest here?

[rlinoz]

#52149 (comment)

@janicduplessis is finding time to fix a bug @ishpaul777 found in the first PR

[janicduplessis]

#52149 is ready. After that it will make implementing the functionality simple

[trjExpensify]

Looks like we have conflicts to resolve, @janicduplessis. Can you jump on those? Thanks!

[johncschuster]

@janicduplessis how's this one going?

[rlinoz]

First PR is in the last steps I think, just some lint fix missing and I will review it again