quickstart-unity icon indicating copy to clipboard operation
quickstart-unity copied to clipboard

Initialization of the static member of FirebaseAnalytics(Unity)

Open std-forward opened this issue 5 years ago • 4 comments

we find this: BAED4EB4-8B36-4B0B-B794-2BA659C41452

But I think we should use lazy initialization here, like this: 817A0C8A-06EF-49C1-800C-336D86305002

why do i report this issue: we hava a analytics wrapper for FirebaseAnalytics, and there is a static map here: BFE8C602-C3FA-44D2-A44C-6505DD205397 but when we use the public static member of FirebaseAnalytics, it constructs a FirebaseApp first, which is not expected, due to the static member FirebaseAnalytics.app (As you say, we must check dependencies first, then construct the app~)

std-forward avatar Jul 28 '20 04:07 std-forward

This issue does not seem to follow the issue template. Make sure you provide all the required information.

google-oss-bot avatar Jul 28 '20 04:07 google-oss-bot

This issue does not seem to follow the issue template. Make sure you provide all the required information.

Sure

std-forward avatar Jul 28 '20 04:07 std-forward

Hi @std-forward, Thank you for bringing this up. We understand your suggestion and will review it to improve the usability of our SDK. Am flagging this as a feature request.

vimanyu avatar Jul 31 '20 21:07 vimanyu

Hi @std-forward, Thank you for bringing this up. We understand your suggestion and will review it to improve the usability of our SDK. Am flagging this as a feature request.

Nice! But it does not really matters! Stability matters!😀

std-forward avatar Aug 03 '20 19:08 std-forward