setUserId accept null too #49

[markxoe]

I've solved #49 so that null will be accepted by setUserId and in the web-version it gets converted to an empty string

[MNorgren]

Any update on this PR? I am also waiting on this fix

[brownoxford]

I have some time set aside later this week for this.

[edouardruiz]

Hi, I also need to unset the userId via setUserId. @brownoxford do you think you need help to merge this PR?

[sergeivoroshilov]

@brownoxford @edouardruiz any updates?

[sneko]

Also interested by this. Any news @brownoxford?

[sneko]

@markxoe it seems your PR is not fixing the issue unfortunately. For example on iOS:

firebase-analytics/ios/Plugin/Plugin.swift

Lines 25 to 29 in 962a666

	if let userId = call.getString("userId") {
	Analytics.setUserID(userId)
	call.resolve()
	} else {
	call.reject("userId property is missing")

Since it expects a "string" it will raise an error in all cases. Did you test your implementation? (maybe I'm doing something wrong)