Andrew Gavgavian
Andrew Gavgavian
@Demesmaeker Thanks for the review, I have adjusted everything to your recommendations besides the one comment on AstroPay TEF. You were right that combining the controllers worked and I was...
@Demesmaeker Thanks for the follow up review. I've pushed all of the requested changes, and then just now pushed a commit adding the author key to the manifest to help...
@Demesmaeker thank you for the review, I've adjusted the code to your suggestions and I added the tests that logically make sense to me but I might be missing some...
@Demesmaeker I really appreciate you taking the time to look so in depth on the PR! I have made the changes requested, added a few more tests, and added back...
@AntoineVDV The only changes in the fixup are: - Removed Uruguay from AstroPay since Local Payments covers more + everything covered in AstroPay. - Changed `payment_method_code.capitalize()` -> `payment_method_id.name` - Added...
@mmahrouss No worries, grouped definitely felt like a longshot. Glad to hear filtered is working soon :)
@auva-odoo Thanks for the review, I agree with you on all cases, and have updated the documentation as requested.
Thanks for the once over @auva-odoo, we were considering adding a table to the page showing what payment methods Nuvei supports in what specific countries, would something like this (Either...
@auva-odoo Thanks for the review, I have made the two adjustments.