Skip to content

Conversation

@bizz84
Copy link
Owner

@bizz84 bizz84 commented May 8, 2025

Replaces #160 by implementing a workaround for this issue:

Workaround is based on this comment:

flutter/flutter#112196 (comment)

Example log while navigating the app:

[log] screenName: signIn, pathParams: {}
[log] screenName: jobs, pathParams: {}
[log] screenName: job, pathParams: {id: 6NDSdYl00ZSDTbfY1zoX}
[log] screenName: jobs, pathParams: {}
[log] screenName: entries, pathParams: {}
[log] screenName: profile, pathParams: {}

@bizz84 bizz84 merged commit 821e212 into master May 8, 2025
1 check passed
@bizz84 bizz84 deleted the go-router-delegate-listener branch May 8, 2025 13:24
@bizz84 bizz84 changed the title Add GoRouterDelegateListener class Add GoRouterDelegateListener class to track screen views May 8, 2025
@atnpem
Copy link

atnpem commented May 8, 2025

Hello,
Doesn't injecting the Firebase observer directly into the “observers” property track all pages?

GoRouter(
        navigatorKey: _rootNavigatorKey,
        observers: [FirebaseAnalyticsObserver(analytics: FirebaseAnalytics.instance)],
        refreshListenable: GoRouterRefreshStream(...),
        routes: [...])

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