Muslim

Another little bug you probably want to fix.
I noticed the button here (/wp-admin/upload.php?page=wr2x_dashboard) that links to the pro version page goes here: https://meowapps.com/products/wp-retina-2x-pro/wp-retina-2x/ while you probably want it to go here: https://meowapps.com/products/wp-retina-2x-pro/ instead.

Update: It seems to be going to (https://meowapps.com/plugin/wp-retina-2x/) from the plugin but then redirects to (https://meowapps.com/products/wp-retina-2x-pro/wp-retina-2x/) on your site.

• This reply was modified 1 year, 8 months ago by Muslim.

Thanks for such a swift response and thanks for such a great plugin.

Best regards.

Thanks for your swift response. I’ve opened a ticket for WP Mail Catcher explaining your suggestion.

Thanks for your response. I tried your suggestion and that did remove the warning from PageSpeed Insights.

It would be even better if Judge.me could include the dimensions so every user doesn’t have to manually add the above CSS to their site. Thanks.

Thank you for your prompt response and recommendation. I appreciate you investigating the issue and reaching out to the developers of the other plugin for a possible solution.

I find your plugin to be very impressive and am actually considering replacing GA with it. If I encounter any further issues or concerns, I won’t hesitate to contact you. Once again, thank you for your assistance.

Hi George, it’s my pleasure. Many thanks everyone behind this awesome plugin!

Regarding improvements, for a start, there seems to be a problem with the logging of lockouts and blocks. I’m pretty sure it’s related the database ‘country_iso_code’ error I reported in the support threads.

This does seems to be a recent issue as Adam mentioned in his reply because the logs were working before and suddenly stopped working most likely after a recent update.

The lockouts are working but they are not being logged for me at all. I’ve manually triggered a lockout by reaching the threshold I set for failed logins but nothing gets logged and it always says ‘There have been 0 lockouts in the last 24 hours.’ in the view logs area.

For any other improvements I’ll have to take some time to put together some of my opinions and hopefully post them here later.

• This reply was modified 2 years, 10 months ago by Muslim.

Thank you for your reply Adam. I hope you are doing good as well.

It does look like the error is consistently occurring every time a failed login or lockout is logged to the DB. So I definitely think it needs to be permanently fixed.

WP Defender is such a great plugin and fixing this issue will only make it even better. A big thanks to all the good guys behind it. Please keep up the great work.

I have a new update. The plugin doesn’t seem to be updating the order status correctly via the webhook.

I decided to dig into your plugin files and found the webhook file at commerce-coinbase-for-woocommerce/includes/webhook.php

The first thing that came to my mind was to try and call global $woocommerce; on line 50 right before $order = new WC_Order( $order_id ); and voila! The order status is updating just fine now.

I hope this information is somehow helpful. Best regards.

Sorry for the late reply. Yes, I can confirm it is working fine now. Thank you very much.

Update! I realized your route accepts the POST method therefor I can’t test it in chrome.

I did a quick test with the webhook payload example from https://commerce.coinbase.com/docs/api/#webhooks in Postman and everything seems to be working just fine.

I’m pretty sure the reason the test via coinbase commerce dashboard is getting a 404 response is because that is actually what your plugin API route returns when there is no order associated with the ID and the coinbase test doesn’t show the message:

{
    "message": "No order associated with this ID."
}

Sorry for the false alarm. Have a great day.

• This reply was modified 3 years ago by Muslim. Reason: Marking as resolved

Sure, here’s a screenshot of my permalink settings: Removed screenshot.

Yes I followed all the steps in your screenshots.

• This reply was modified 3 years ago by Muslim. Reason: Remove screenshot