Changes to make setScreenName run in main thread.

[sandfield-andy]

I've made changes to the setScreenName function to run in the main thread. This is needed to make it work for iOS.

[remisture]

Just curious, what is it that's not working on iOS? Screen name is apparently working for me - it appears in the Analytics dashboard, at least.

[karlbohlmark]

Whats the status here? Will this be merged? @soumak77

[soumak77]

@sandfield-andy could you fix the merge conflicts so I can merge this PR?

[soumak77]

fixed via bce8acd

[JonSmart]

@soumak77 will you be doing a new release version with this fix? 1.1.1?

[soumak77]

@JonSmart sure, publishing now

[JonSmart]

@soumak77 perfect. Thanks.

[newuser44]

Has this been pushed in?

[soumak77]

@newuser44 yes, try version 1.1.4

[newuser44]

it seems to be pulling firebase 2.0.0 I know there is a way to pull a specific version.

[soumak77]

2.0 should be fine too. 2.0 upgrades firebase SDK which you might want as well. See the CHANGELOG for more info.

If you want 1.1.4, run cordova plugin add cordova-plugin-firebase@1.1.4

[newuser44]

it looks like I have 1.1.4
The error I'm seeing is
ERROR: Unhandled Promise rejection: Analytics isn't initialised ; Zone: ; Task: setTimeout ; Value: Analytics isn't initialised

[soumak77]

@newuser44 I believe #832 will fix your problem. If you are able to test that PR, we can get a new version published sooner

[newuser44]

Sorry, not really sure how I would test that seperate build. Are these problems solved in the 2.0 version?

[soumak77]

@newuser44 the setScreenName running on the main thread is fixed, but we need #832 merged to fix the issue you're running into with analytics not being initialized