-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fropay for Russia Users [Android & Desktop] #1046
Conversation
@atavism Mind taking a look? |
…client into jigar/fropay-payment
@atavism Fixed issue payment methods not updating, Do you mind taking look at it again? |
@atavism If all is okay, I like to merge this and do a bit of testing with the main, I am planning to upload the beta tomorrow if all is okay. |
Thanks! Testing now |
@jigar-f Everything seems to be working well for me. Changing my language on desktop returns the correct plans now! I did run into one issue with Fropay -- and it's not an issue with the other payment providers -- the checkout page never loads for some reason: Here's my logs:
|
Just had one comment. LGTM otherwise! |
Same for me. It's getting stuck with these logs, its loading same page again and again.
|
@atavism Fropay recently added human verification that why its not working, Fropay open fine with system browser, Do you have any idea what might causing this issue? |
Oh, interesting.. I thought it might just be an issue on my side. Here's the URL of the checkout page the webview is having trouble loading.. https://sci.fropay.art/get?amount=5580.00&shop_id=4515&label=cd81b13a-5f0e-4413-a6dd-2ab6085acc25 This loads fine for me in a browser |
I think this is the issue |
@jigar-f Not sure. I tried changing several InAppBrowser settings: https://github.com/pichillilorenzo/flutter_inappwebview/blob/4c58653113573558ea5be880aca931e85b23da7e/flutter_inappwebview_platform_interface/lib/src/in_app_browser/in_app_browser_settings.dart#L95 but I still ran into the same issue. I think we can either try opening the URL in the system browser on macOS or just merge this for now? I can check quickly to see if it works on Windows |
@atavism Yes, I did the same, Just give me a few mins, I am trying something that works, otherwise, I will merge this. |
Depends on -:getlantern/flashlight#1370