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

screenName not working for ios. #828

Closed
newuser44 opened this issue Sep 6, 2018 · 8 comments
Closed

screenName not working for ios. #828

newuser44 opened this issue Sep 6, 2018 · 8 comments

Comments

@newuser44
Copy link

New to this all.
calling the firebase service setScreenName("Login");
This seems to work fine on android, but on ios we are not seeing the firebase_screen parameter.
In Firebase Analytics go to events, go to screen view, filter by ios and change user engagement to screen name. We see Screen Class "(not set)" % 100%.
Its like it got the data, but doesn't know the class to group it with?

@newuser44
Copy link
Author

Both posted the same bug.
In xcode while running the app, I'm seeing the message
"setScreenName:screenClass: must be called from the main thread to ensure that the events are associated with the intended screen."
???

@briantq
Copy link
Contributor

briantq commented Sep 7, 2018

What version of the plugin are you using?

@newuser44
Copy link
Author

newuser44 commented Sep 10, 2018

cordova-plugin-firebase 1.0.5 "Google Firebase Plugin"

@briantq
Copy link
Contributor

briantq commented Sep 10, 2018

@newuser44 Running cordova plugin list should give you a list of the plugins you have installed and their version

@newuser44
Copy link
Author

I updated my last comment

@briantq
Copy link
Contributor

briantq commented Sep 10, 2018

@newuser44 Thanks for that. I don't have the time to dig into what the issue/fix for this is, but the code you are looking for on iOS is in FirebasePlugin.m, specifically the setScreenName method. It seems pretty straight forward, but if you can figure out what we might need to change to resolve the error, I would be happy to review a Pull Request and get it fixed in the plugin.

@soumak77
Copy link
Contributor

@newuser44 you might want to try an updated version of the plugin. #751 was merged recently which fixed an issue with setScreenName on iOS.

@soumak77
Copy link
Contributor

closing as fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants