-
Notifications
You must be signed in to change notification settings - Fork 324
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 android app id and ios bundle id in GA metrics #8177
base: master
Are you sure you want to change the base?
Conversation
@@ -326,6 +334,13 @@ class DeepLinksController extends DisposableController | |||
target: target, | |||
); | |||
_iosLinks[selectedAndroidVariantIndex.value] = result; | |||
ga.impression( |
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.
Do we have this method?
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.
yes, it's similar to ga.select but nothing is selected here.
external String? get android_app_id; | ||
external String? get ios_bundle_id; |
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.
These will need to be added to the custom dimensions and metrics in the GA console for the legacy DevTools analytics if we want to track there. Otherwise they should automatically be included in the unified_analytics data for DevTools.
looks like I need to update the event in pkgs/unified_analytics first, I opened PR dart-lang/tools#295 |
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
this.iosBundleId, | ||
}); | ||
|
||
/// The anroid app id of the flutter project. |
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.
sp: android
r/flutter/Flutter (same below)
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.
please also bump the min dep of unified_analytics to ^6.1.4
to include the changes here: dart-lang/tools#295
issue: (#8068)
Pre-launch Checklist
///
).If you need help, consider asking for help on Discord.