-
-
Notifications
You must be signed in to change notification settings - Fork 256
Feat: Client Reports #829
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: Client Reports #829
Conversation
# Conflicts: # flutter/test/mocks.mocks.dart
Codecov Report
@@ Coverage Diff @@
## main #829 +/- ##
==========================================
+ Coverage 89.95% 90.33% +0.37%
==========================================
Files 114 119 +5
Lines 3703 3776 +73
==========================================
+ Hits 3331 3411 +80
+ Misses 372 365 -7
Continue to review full report at Codecov.
|
It needs testing with the iOS and Android bridges, but codewise looks good. |
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
@marandaneto Client reports are being send with the |
📜 Description
Feature branch collecting all sub-branches for the client reports implementation.
💡 Motivation and Context
Closes #812
💚 How did you test it?
Unit tests.
📝 Checklist
🔮 Next steps
sendClientReports
on native Android options when Java Client Reports are mergedCloses #735