Skip to content
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

Add telemetry controls for internal reports. #1701

Merged
merged 1 commit into from
Jun 15, 2022
Merged

Conversation

kstenerud
Copy link
Contributor

Goal

See ROAD-1488-ED

@kstenerud kstenerud requested a review from lemnik June 14, 2022 14:52
@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Jun 14, 2022

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1855.52 1606.17
arm64_v8a 651.66 401.8
armeabi_v7a 586.13 340.37
x86 725.37 475.51
x86_64 692.61 446.85

Generated by 🚫 Danger

@kstenerud kstenerud force-pushed the PLAT-8556-telemetry branch 3 times, most recently from de57151 to 82ce802 Compare June 15, 2022 07:43
@lemnik
Copy link
Contributor

lemnik commented Jun 15, 2022

I think this PR is also missing a CHANGELOG entry?

@kstenerud kstenerud force-pushed the PLAT-8556-telemetry branch from 82ce802 to cbaac04 Compare June 15, 2022 09:11
@kstenerud kstenerud requested a review from lemnik June 15, 2022 09:12
Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants