-
Notifications
You must be signed in to change notification settings - Fork 26
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: Add missing staging backend verification for analytics (WPB-10736) #3366
fix: Add missing staging backend verification for analytics (WPB-10736) #3366
Conversation
…ifier Signed-off-by: alexandreferris <[email protected]>
app/src/main/kotlin/com/wire/android/analytics/ObserveCurrentSessionAnalyticsUseCase.kt
Show resolved
Hide resolved
Signed-off-by: alexandreferris <[email protected]>
Signed-off-by: alexandreferris <[email protected]>
…ing_backend_verification_for_analytics
Quality Gate passedIssues Measures |
Built wire-android-staging-compat-pr-3366.apk is available for download |
…6) (#3366) Signed-off-by: alexandreferris <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/candidate #3366 +/- ##
==================================================
Coverage 44.74% 44.75%
==================================================
Files 456 456
Lines 15401 15403 +2
Branches 2572 2574 +2
==================================================
+ Hits 6891 6893 +2
Misses 7783 7783
Partials 727 727
Continue to review full report in Codecov by Sentry.
|
…ifier
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
There was no verification for staging backend api when observing analytics toggle change
Causes (Optional)
Missed out this part
Solutions
Add staging backend api when observing analytics toggle change to propagate identifier and initialize analytics