search

OneSignal / react-onesignal

React OneSignal Module: Make it easy to integrate OneSignal with your React App!
Other
74 stars 23 forks source link

logoutEmail() should probably return a promise #53

Closed statico closed 2 years ago

statico commented 2 years ago

Right now it returns void, but clearly there's a network request happening. Chrome is showing me an uncaught exception when there's an error with the call. It would be nice to be able to catch this in my own code.

CleanShot 2021-11-09 at 13 56 22

rgomezp commented 2 years ago

Howdy, Thanks for reporting. You are right, this should return a promise.

Thanks for your patience while we resolve.