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

[firebase-analytics] Add DebugView support on Expo client #7796

Merged
merged 21 commits into from Apr 15, 2020

Conversation

IjzerenHein
Copy link
Contributor

Why

This PR adds support for using the DebugView mode in Google Analytics. This option is only supported on the standard Expo client which uses the pure JS implementation. All other environments are required to use the platform specific way of enabling debug mode.

How

  • The pure JS implementation has been extended to support the debugMode option. Additionally a method called setDebugModeEnabled() has been added to enable/disable the debug-mode. Any events sent afterwards will be tracked in the DebugView console.
  • On all other environments, except to the standard Expo client, this method throws an exception with a link on how to use the debug-mode (e.g. native iOS, Android or Web)
  • docs have been updated in unversioned and sdk36
  • changelog-entry has been added

Test Plan

  • Tested within expo-firebase-demo demo app (bare)
  • Tested that setDebugModeEnabled() throws on native iOS
  • Tested on Expo client (ios)

Debug events recorded with the pure JS implementation

image

Copy link
Contributor

@EvanBacon EvanBacon left a comment

Choose a reason for hiding this comment

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

Good work

@IjzerenHein IjzerenHein force-pushed the @hein/expo-firebase/analytics-web-debugview branch from e6f9385 to 09b0d08 Compare April 14, 2020 12:02
@IjzerenHein IjzerenHein merged commit b9fac3f into master Apr 15, 2020
@IjzerenHein IjzerenHein deleted the @hein/expo-firebase/analytics-web-debugview branch April 15, 2020 09:27
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.

None yet

2 participants