Closed garystubbings closed 4 years ago
So with this we can just call advancedWebview.close() and it would close?
That would be awesome!
So with this we can just call advancedWebview.close() and it would close?
That would be awesome!
Yeah, it would only work for iOS though, android doesn't support this. I am using this with my authorization_code grant flow, opening the advancedWebView when dispatching login and then closing it after the token response. It's worth noting that android closes automatically when the activity is setup correctly.
@garystubbings - cool, we have a similar flow that we use for our app, where normally I would send the user back with a deep-link after successful authentication (which works for the regular browser), but in the advanced version it just sorta "freezes", so this should fix it!
Thanks, hope it will be merged soon.
@garystubbings - cool, we have a similar flow that we use for our app, where normally I would send the user back with a deep-link after successful authentication (which works for the regular browser), but in the advanced version it just sorta "freezes", so this should fix it!
Thanks, hope it will be merged soon.
Exactly the same here @Reached , the deeplink callback dispatches an event to fetch the token and now the advancedWebView is persisted from the authorize request it just needs to simply call advancedWebView .close() as you say.
I think @bradmartin has some uncommitted changes (looking at the source on npm) so not sure if he'll want me to rebase - if he approves of the changes. Or he can integrate the changes with his uncommitted version as there isn't that many.
Based on one other request I also see the need to close the ios web view so have prepared this pull request.