-
Notifications
You must be signed in to change notification settings - Fork 155
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
Rework room summary #3631
Rework room summary #3631
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
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, thanks!
heroes = roomSummary.info.heroes, | ||
canonicalAlias = roomSummary.info.canonicalAlias, | ||
) | ||
} |
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.
Move function toSelectRoomInfo()
from the test class to this file and use it here?
|
||
@Immutable | ||
sealed interface ResolvedSuggestion { | ||
data object AtRoom : ResolvedSuggestion | ||
data class Member(val roomMember: RoomMember) : ResolvedSuggestion | ||
data class Alias(val roomAlias: RoomAlias, val roomSummary: RoomSummary) : ResolvedSuggestion | ||
data class Alias( |
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.
We will probably want to do the same type of rework for Member.roomMember
(later!)
Quality Gate passedIssues Measures |
Content
This PR takes care of 3 things :
getRoomInfoFlow
to not usegetRoom
internally.Motivation and context
Better code and prepare work for knocking feature.
Screenshots / GIFs
Tests
Tested devices
Checklist