-
Notifications
You must be signed in to change notification settings - Fork 28
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
fix: leaking UI context GetE2EICertificateUseCase (WPB-6648) #2717
fix: leaking UI context GetE2EICertificateUseCase (WPB-6648) #2717
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2717 +/- ##
===========================================
+ Coverage 42.87% 43.01% +0.13%
===========================================
Files 408 408
Lines 13524 13529 +5
Branches 2392 2392
===========================================
+ Hits 5799 5819 +20
+ Misses 7047 7025 -22
- Partials 678 685 +7
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
APKs built during tests are available here. Scroll down to Artifacts! |
Build 3179 succeeded. The build produced the following APK's: |
Cherry pick from the original PR:
kalium
PR Submission Checklist for internal contributors
The PR Title
The PR Description
What's new in this PR?
Issues
Leaking Context from
Causes (Optional)
Context was being passed from UI
Solutions
Inject on ViewModel that is calling