Mark room as fully read when user goes back to the room list.

[bmarty]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

Send m.fully_read read marker once the user is going back to the room list.
Second commit is just removing noise in the logs.

Motivation and context

Closes #2281

Screenshots / GIFs

Tests

• See Own read marker (new) does not get updated when reading the room #2281, hence, I did not manage to see any "New" line on my messages during the tests...

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 23
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request includes a new file under ./changelog.d. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 80.40%. Comparing base (07af7cc) to head (ba1776c).
Report is 3 commits behind head on develop.

Files with missing lines	Patch %	Lines
...features/messages/impl/timeline/MarkAsFullyRead.kt	66.66%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2687   +/-   ##
========================================
  Coverage    80.40%   80.40%           
========================================
  Files         2149     2150    +1     
  Lines        56843    56854   +11     
  Branches      7125     7127    +2     
========================================
+ Hits         45703    45713   +10     
- Misses        8689     8690    +1     
  Partials      2451     2451           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/cwjaq3

[frebib]

I'm running this patch and it doesn't seem to do anything for me

[ganfra]

I'm running this patch and it doesn't seem to do anything for me

@bmarty I think the inner room can be closed before calling the method...

[bmarty]

OK, closing this for now, we will revisit when the New divider in the timeline will be working as expected.

[foss-]

OK, closing this for now, we will revisit when the New divider in the timeline will be working as expected.

@bmarty Can you link the relevant open ticket for what's blocking this currently?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud