-
Notifications
You must be signed in to change notification settings - Fork 428
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
[FR] Able to drop analytics events before CheckAndFixDependenciesAsync() completes #853
Comments
I found a few problems with this issue:
|
Hi @AntonPetrov83, any events happens before Firebase Initialization can't be logged properly by firebase. That should be one of the earliest init of the app. I'd like to get more context of what you might want to log earlier. |
Hi @cynthiajoan , I would like to log very early events like "loading_start" to build funnels later. The problem with Firebase SDK is that it requires async initialization so this only SDK blocks the entire logging system from early start. As a workaround I implemented a queue to collect calls into Firebase until it is initialized. And this is ugly. I think such design shifts a lot of complexity from the Firebase SDK to the client side. Events logging must work through a simple static API and managing initialization and connection issues must be hidden on your side. With all my respect to your hard work. |
Unfortunately this is how the API is designed now. I understand your concern. This seems to be a feature request for a design change (quite major one actually). Please allow me to change the label and title a bit. Shawn |
@AntonPetrov83 you inspired me to write this short article ;) |
@chkuang-g Well, it's really a bad design. I create a script for handling this stuffs ( It use both Firebase and Facebook) maybe it helps someone in the future
Then add it to an object in your first scene. And use it with :
|
The official guide recommends to set a flag when
CheckAndFixDependenciesAsync
completes to indicate that Firebase is ready.Otherwise, an exception gets thrown:
So how am I supposed to log early events? What are the best practices?
The text was updated successfully, but these errors were encountered: