Skip to content

Conversation

@pixlwave
Copy link
Member

This PR makes 2 changes:

  • Enable the newer latest event sorter.
  • Decorate rooms in the room list with badges for Sending and Failed Sending states.

This is the iOS part for element-hq/element-meta#2945 and element-hq/element-meta#2946

@pixlwave pixlwave requested a review from a team as a code owner November 27, 2025 13:39
@pixlwave pixlwave requested review from Velin92 and removed request for a team November 27, 2025 13:39
@pixlwave pixlwave added the pr-change for updates to an existing feature label Nov 27, 2025
@pixlwave pixlwave changed the title Latest event tweaks Show the send state in the room list. Nov 27, 2025
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

Warnings
⚠️ You seem to have made changes to views. Please consider adding screenshots.

Generated by 🚫 Danger Swift against fd299b6

@pixlwave pixlwave force-pushed the doug/latest-event-tweaks branch 2 times, most recently from 3abe3dd to 314b351 Compare November 27, 2025 15:37
@pixlwave pixlwave force-pushed the doug/latest-event-tweaks branch from 314b351 to fd299b6 Compare November 27, 2025 15:54
@sonarqubecloud
Copy link

@element-hq element-hq deleted a comment from codecov bot Nov 27, 2025
@pixlwave pixlwave merged commit f6d0b2c into develop Nov 27, 2025
8 checks passed
@pixlwave pixlwave deleted the doug/latest-event-tweaks branch November 27, 2025 16:43
@codecov
Copy link

codecov bot commented Nov 27, 2025

Codecov Report

❌ Patch coverage is 95.45455% with 4 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@051e5ce). Learn more about missing BASE report.
⚠️ Report is 3 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ervices/Room/RoomSummary/RoomSummaryProvider.swift 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #4791   +/-   ##
==========================================
  Coverage           ?   79.41%           
==========================================
  Files              ?      878           
  Lines              ?    84097           
  Branches           ?        0           
==========================================
  Hits               ?    66784           
  Misses             ?    17313           
  Partials           ?        0           
Flag Coverage Δ
unittests 71.31% <95.45%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-change for updates to an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants