feat: show security providers info screen (WPB-27916) - #5156
Conversation
…-info-screen # Conflicts: # app/src/main/kotlin/com/wire/android/di/metro/WireMetroViewModelBindings.kt # app/src/main/kotlin/com/wire/android/ui/debug/DebugInfoViewModelFactory.kt
Codecov Report❌ Patch coverage is ❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #5156 +/- ##
===========================================
- Coverage 52.77% 52.41% -0.36%
===========================================
Files 659 667 +8
Lines 23589 23846 +257
Branches 3881 3932 +51
===========================================
+ Hits 12448 12499 +51
- Misses 9994 10186 +192
- Partials 1147 1161 +14
... and 15 files with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
|
this will show all providers which is not the intendet behaivour |
The ticket asks for a section that "displays entries for security providers," which is what this implements. the installed provider list with their entries What you're describing is different. Happy to build it, but could you update the AC since the scope changed? |
Good point, my bad I oversaw some info, I've updated the ticket(s) with a couple of things to consider:
|
| <string name="debug_settings_force_repair_faulty_keys" translatable="false">Reset</string> | ||
| <string name="debug_settings_feature_flags" translatable="false">Feature Flags</string> | ||
| <string name="debug_settings_conversation_crypto_stats" translatable="false">Conversation Crypto Stats</string> | ||
| <string name="debug_settings_security_providers" translatable="false">Security Providers</string> |
There was a problem hiding this comment.
| <string name="debug_settings_security_providers" translatable="false">Security Providers</string> | |
| <string name="debug_settings_security_providers" translatable="false">Security Diagnostics</string> |
yamilmedina
left a comment
yamilmedina
left a comment
There was a problem hiding this comment.
lgtm ! just pls let's change the name of the Security providers item to Security Diagnostics before merging as indicated and be aligned with the docs/ticket :D
|
APKs built during tests are available here. Scroll down to Artifacts! |
3 participants
https://wearezeta.atlassian.net/browse/WPB-27916
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764The PR Description
What's new in this PR?
Issues
Show security providers info screen in debug settings screen
Screen_recording_20260811_122754.mp4
Needs releases with:
Testing
Test Coverage (Optional)
How to Test
Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.
Notes (Optional)
Specify here any other facts that you think are important for this issue.
Attachments (Optional)
Attachments like images, videos, etc. (drag and drop in the text box)
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.