-
Notifications
You must be signed in to change notification settings - Fork 263
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
Add auth hash to external Id #797
Conversation
309e9b7
to
9589356
Compare
* Add auth to setExternalId * Add tests for new functionality
9589356
to
14d80dc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to add remote param check for require_user_id_auth
return; | ||
} else if (!self.currentSubscriptionState.userId || !self.app_id) { | ||
[OneSignal onesignal_Log:ONE_S_LL_WARN message:[NSString stringWithFormat:@"Attempted to set external user id, but %@ is not set", self.app_id == nil ? @"app_id" : @"user_id"]]; | ||
return; | ||
} else if (self.currentEmailSubscriptionState.requiresEmailAuth && !hashToken) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self.currentEmailSubscriptionState.requiresEmailAuth
should be checking requiresUserIdAuth
instead
1e66130
to
9af58b5
Compare
* Add auth to setExternalId * Add tests for new functionality * Add failure block to external id methods * Add tests for external id auth validation
Needs name params validation
This change is