dpash
Forum Replies Created
-
We’ve implemented caching of the failed http request to the available-payment-methods url and our home page load time has dropped 33% from ~3 seconds to ~2 seconds for loading the HTML.
This is definitely a bug in the revolut plugin not handling HTTP failures correctly.Hi, I’ve also been seeing this behaviour in our own site. I’ve added some logging to the plugin to make sure it was making HTTP requests and I can confirm that it is indeed making multiple requests to your servers for each request: I’ve counted 27 outgoing requests during a single wordpress request.
GET https://merchant.revolut.com/api/public/available-payment-methods?amount=0¤cy=EUR | Status: 403 | Duration: 0.094s | Caller: Revolut\Plugin\Infrastructure\Api\MerchantDetails\MerchantDetailsApi::availablePaymentMethods
It’s also being called from WC_Payment_Gateway_Revolut::fetch_available_payment_methods_and_brand_logos. All requests result in a 403 error.
I suspect there’s meant to be some caching that’s not working, but it’s causing a considerable slowdown on our site. I can provide a full log file if you’d like and let me know if you’d like me to open a new issue instead of reviving this one.Yes, I read this post. The prices on our products are all exclusive of VAT on the product edit screen, but all the settings on the Woocommerce Settings page for tax are disabled and “Display prices in the shop” and “Display prices during basket and checkout” are hardcoded to be “Excluding tax” and is exactly what the description says will happen:
“Automated taxes take over from the WooCommerce core tax settings. This means that “Display prices” will be set to Excluding tax and tax will be Calculated using Customer shipping address”Additionally, “Prices entered with tax” is also disabled, so can’t be changed.
So there doesn’t seem to be any way to display prices to the customer including tax using this plugin.- This reply was modified 8 months, 1 week ago by dpash.
Having read this topic, I’ve disabled the woocommerce checkout blocks on the checkout page and now the payment methods appear to be loading correctly, so it seems to be an issue with Funnels forcing the classic checkout and revolut attempting to use the blocks
Forum: Plugins
In reply to: [Revolut Gateway for WooCommerce] Duplicated payments on SubscriptonsI can confirm that our most recent subscription renewal did not have a double charge.
Forum: Plugins
In reply to: [Revolut Gateway for WooCommerce] Duplicated payments on SubscriptonsThanks for the speedy fix. I’ve deployed 4.16.2 and will let you know tomorrow if we have any additional issues as that’s our next scheduled renewal.
Forum: Plugins
In reply to: [Revolut Gateway for WooCommerce] Duplicated payments on SubscriptonsHaving checked the dates of releases and the difference in the code, I’ve now downgraded to 4.15.1
If other people want to find older versions, they can be downloaded at https://wordpress.org/plugins/revolut-gateway-for-woocommerce/advanced/
Forum: Plugins
In reply to: [Revolut Gateway for WooCommerce] Duplicated payments on SubscriptonsWe are also seeing this issue. Duplicate payments on subscription orders. Running 4.16.1 on WooCommerce 9.1.4 and WordPress 6.6.1. The issue started on July 30th, which matches the update to 4.16.1.
I can confirm that this issue is only happening on Revolut subscription renewals. Non-subscription orders do not have an issue and neither do non-Revolut subscriptions.
I have downgraded to 4.16.0 and have a renewal in 13 hours, so we’ll see what happens with that order.- This reply was modified 1 year, 7 months ago by dpash. Reason: Updated to include comment about downgrading
Forum: Plugins
In reply to: [Nomad World Map] Waymark resets to nearest cityThank you very much. Let me know if you’d like me to test anything.
Forum: Plugins
In reply to: [Nomad World Map] Waymark resets to nearest cityThe map is being used on http://experimentalnomad.com/lost-sea-came-desert-island/
You’ll notice that it’s currently working, because I hacked the hidden location field to be visible and then pasted in the right coordinates.
(I also hacked up a change to allow me to override map type on a per-map basis. Do you want me to open a new issue for that?)