Did a quick rebase to include #218 so we can see if the tests will pass.

Did a quick rebase to include #218 so we can see if the tests will pass.

Thanks! Looks like Cordova adapter › allow loaderror on InAppBrowser before finishing login is failing as expected, since there is no fix for #208 included at the moment. I could skip/remove this test for now.

Additionally I see the Firefox - Cross origin tests failing and will have a look at them, I think I missed those those locally some tweaks to the playwright setup.

Fix the tests in 7fbc7fb

@jonkoops just wanted to quickly follow up if I should continue in this direction?

Sorry for the late reply, I got bogged down in some other stuff. I think this is going in the right direction, I will try to get a more detailed review in when I can. Currently, I need to fix #209 and #208 and do a release before landing new features. If you have any idea what might be causing those, help is appreciated.

Sorry for the late reply, I got bogged down in some other stuff. I think this is going in the right direction, I will try to get a more detailed review in when I can. Currently, I need to fix #209 and #208 and do a release before landing new features. If you have any idea what might be causing those, help is appreciated.

No worries, I had my own fair share of other priorities, so apologies for my late response as well. I rebased my changes on latest master and adjusted the tests to latest changes of explicitly navigating in tests, so that they pass locally

My plan is to polish the PR this week and put it up for a proper review. Also will have a look at the issues you linked - not too confident that I can be of help there though.