-
Notifications
You must be signed in to change notification settings - Fork 216
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rooms without an unread badge show up in "unread" tab #1540
Rooms without an unread badge show up in "unread" tab #1540
Conversation
- First implementation
- Apply search query to the recents section
- update after code review
- Changed MXRecentsRoomListDataFetcher to MXBreadcrumbsRoomListDataFetcher
- Added change log
- Excluded room summaries without notifications from unread list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 😎
Codecov Report
@@ Coverage Diff @@
## gil/6079-Delight_Edit_layout_experiment #1540 +/- ##
============================================================================
- Coverage 42.09% 12.01% -30.08%
============================================================================
Files 515 515
Lines 84101 84115 +14
Branches 37059 35921 -1138
============================================================================
- Hits 35399 10109 -25290
- Misses 47672 73634 +25962
+ Partials 1030 372 -658
Continue to review full report at Codecov.
|
…up_in_unread_tab # Conflicts: # MatrixSDK/Data/MXRoomSummary.m
* release/nodefy_0.23.15: (175 commits) finish version++ version++ Stop using the room ID from the space child state. finish version++ version++ MXBeaconAggregations: Add debug log when MXBeaconInfoSummary associated to a stopped beacon info is not found. Implement new space selector bottom sheet (matrix-org#1541) Rooms without an unread badge show up in "unread" tab (matrix-org#1540) Delight edit layout experiment (matrix-org#1515) Send an empty dictionary when calling /join to be spec compliant. Empty commit to wake codecov AllWorkingTests: Disable tests that do not pass on GH actions changelog AllWorkingTests: Disabled commented tests Crypto tests CI: Expose tests report on failure and share code coverage Tests: Run working integration tests on every PR AllWorkingTests: Disable all unit tests AllWorkingTests: Disable the 22 tests that fail AllWorkingTests: Disable test classes with tests that crash Tests: Create AllWorkingTests test plan with stable tests ... # Conflicts: # MatrixSDK/Data/RoomSummaryStore/CoreData/Models/MXRoomSummaryMO.swift
resolves element-hq/element-ios#6511