Simplicity icon indicating copy to clipboard operation
Simplicity copied to clipboard

Callback not called

Open Alex293 opened this issue 9 years ago • 6 comments

Hi, when the user press the Done button at the top of the safari view the simplicity callback is not called. I think it should be because now the code can't have any clue if the safari view is cancelled like this.

Alex293 avatar Nov 12 '16 14:11 Alex293

Yes, that's correct. If the user "cancels" the login, it's not called. I'd be open to having it call back with an error; is there a specific use case you're looking for, where you need that information?

edjiang avatar Nov 14 '16 20:11 edjiang

Nothing specific I just want to be called when its cancelled. I'm modifying the status bar style before opening the modal and I'd like to restaure it after closing as I do when the user accept or deny the login.

I think it would be great if you use the same callback with an additional error type for this kind of cancel because someone could use that info. It's still different than denying the login.

Alex293 avatar Nov 14 '16 23:11 Alex293

OK, I looked into this more today, and determined that it may not be a good thing to return it in the regular callback, since the callback is designed to be kind of "fool proof". If you're a lazy developer, you can just make an UIAlertView if error is not nil, and print out the error's description. Having a dialog saying "Error: you cancelled the login" would not make sense in normal circumstances.

Any ideas for how I can incorporate this without introducing undesired behavior?

I think earlier someone asked to get the Safari object from Simplicity so they can "present" it themselves; if that's the case you could attach to the delegate yourself. Or, I can modify Simplicity to just give you the URL, and have you initiate the Safari / app:openURL flow.

edjiang avatar Nov 18 '16 01:11 edjiang

The why not another optional callback? I think it's nice to have the safari view inside the lib it provide a cleaner usability for the lib users

Alex293 avatar Nov 18 '16 01:11 Alex293

Any updates?

smahajan28 avatar May 29 '17 12:05 smahajan28

We can add an optional callback; feel free to submit a pull request :)

edjiang avatar Jun 02 '17 02:06 edjiang