firebase-unity-sdk icon indicating copy to clipboard operation
firebase-unity-sdk copied to clipboard
verified publisher icon firebase

[minor] FirebaseAnalytics.SetCurrentScreen should have screen class be optional

Open Thaina opened this issue 5 years ago • 2 comments

FirebaseAnalytics.SetCurrentScreen has 2 argument. But second argument should be optional for convenient, no need to FirebaseAnalytics.SetCurrentScreen("name",null)

Thaina avatar Dec 24 '19 13:12 Thaina

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

google-oss-bot avatar Dec 24 '19 13:12 google-oss-bot

Hi @Thaina The screenClass is actually needed for the SetCurrentScreen method when a user intends to use another screen class that isn't the default Activity or UIViewController. I'll still treat this as a feature request, so I created an internal FR for this. With that being said, I'll keep this open. Thanks!

paulinon avatar Jan 08 '20 15:01 paulinon