-
Notifications
You must be signed in to change notification settings - Fork 498
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
Device Manager: Filter sessions #6860
Conversation
# Conflicts: # RiotSwiftUI/Modules/UserSessions/UserOtherSessions/Test/Unit/UserOtherSessionsViewModelTests.swift # RiotSwiftUI/Modules/UserSessions/UserOtherSessions/UserOtherSessionsViewModel.swift
📱 Scan the QR code below to install the build for this PR. If you can't scan the QR code you can install the build via this link: https://i.diawi.com/F47gRn |
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! A couple of minor points line.
...ets/Images.xcassets/DeviceManager/user_other_sessions_filter_selected.imageset/Contents.json
Outdated
Show resolved
Hide resolved
RiotSwiftUI/Modules/UserSessions/UserOtherSessions/UserOtherSessionsModels.swift
Outdated
Show resolved
Hide resolved
RiotSwiftUI/Modules/UserSessions/UserOtherSessions/View/UserOtherSessions.swift
Outdated
Show resolved
Hide resolved
RiotSwiftUI/Modules/UserSessions/UserOtherSessions/View/UserOtherSessions.swift
Show resolved
Hide resolved
} | ||
} | ||
|
||
private func createEmptySessionsItemsSection(header: UserOtherSessionsHeaderViewData, title: String) -> some View { |
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.
Small personal nitpick: I generally have the attitude that if you need to use a func
to create a subview that's probably a code smell and it should likely be extracted out into its own struct
.
Codecov ReportBase: 11.58% // Head: 54.13% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #6860 +/- ##
============================================
+ Coverage 11.58% 54.13% +42.54%
============================================
Files 1553 410 -1143
Lines 154845 16162 -138683
Branches 62471 6504 -55967
============================================
- Hits 17946 8749 -9197
+ Misses 136292 7218 -129074
+ Partials 607 195 -412
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Co-authored-by: Doug <[email protected]>
…ssionsModels.swift Co-authored-by: Doug <[email protected]>
Kudos, SonarCloud Quality Gate passed! |
@pixlwave thanks for the comments 👍 fixed! |
Closes #6838
PSG-683 ACs: