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
feat: observe team app lock config (WPB-4476) #2380
Conversation
# Conflicts: # kalium
Test Results663 tests +2 663 ✔️ +2 5m 53s ⏱️ - 4m 58s Results for commit 4d9e335. ± Comparison against base commit 0584961. This pull request removes 2 and adds 4 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Build 1567 failed. |
Codecov Report
@@ Coverage Diff @@
## develop #2380 +/- ##
=============================================
+ Coverage 41.51% 41.60% +0.08%
Complexity 1077 1077
=============================================
Files 343 343
Lines 12252 12271 +19
Branches 1608 1612 +4
=============================================
+ Hits 5087 5105 +18
Misses 6685 6685
- Partials 480 481 +1
Continue to review full report in Codecov by Sentry.
|
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1571 succeeded. The build produced the following APK's: |
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.
great work! just a tiny question on the commented code 🤔
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1600 succeeded. The build produced the following APK's: |
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1611 succeeded. The build produced the following APK's: |
# Conflicts: # app/src/test/kotlin/com/wire/android/ui/home/appLock/LockCodeTimeManagerTest.kt # kalium
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1636 succeeded. The build produced the following APK's: |
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Description
In this PR, I updated mainly
ObserveAppLockConfigUseCase
so it will observe team app lock config as well and return the valid status for the appNext PR will be for displaying SetAppLockScreen when it got enabled by the team
Needs releases with:
Testing
Test Coverage (Optional)
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
.