Forum Replies Created
-
Posts
-
Thanks, Anna– that’s indeed what was expected and why the issue seems odd, and I just wanted to make sure I didn’t miss anything, and your affirmation is what was needed. I’ll dig in further and indeed maybe try a different SMTP host. I hadn’t tried Postman SMTP but will try it if other attempts don’t pan out.
Many thanks.
Thanks, Anna! That’s awesome to know. (And a relief.)
Please do extend my gratitude to Jamie as well.
All the best,
Nicholas
Update: I got a very confident note from Cloudways about replacing the same line, then moving on to other things as if they were sure this was the fix. And sure enough, the moment I changed it, that fixed it. The message is now gone, and I didn’t even have to restart the server. I reckon they knew about it directly based on your (Jamie’s) earlier assessment.
This is their fix:
Hi Nicholas,
Sorry for the delay, regarding wcvendors, can you please replace code with this :
if ( in_array( $_SERVER[ 'HTTP_X_REAL_IP' ], $whitelist ) ){…
Noting this here to update you but also in the off chance that it helps with WCV going forward.
Speaking of which, may I just ask, will you be adapting that to WCV-Pro or is this specific enough for my/our install that I will need to update the plugin manually like this at every upgrade/update?
Thanks!
Nicholas
Hi Jamie,
Thank you for looking into this. Yes, I too am just relieved the code is functioning despite the error/warning, whch I did too reckon it shouldn’t have worked for everyone else. I also hope the fact that the site was migrated from a WHM/CPanel account and onto this (Cloudways control using DigitalOcean droplet) doesn’t have anything to do with it.
I have done what you instructed and changed the line (and commenting-out the previous line). Additionally, I purged Varnish from the server and restarted it. Unfortunately, the symptom persists.
I have indeed tried to engage Cloudways into this and they have tried (in their limited capacity in the context of the plugins) to look into it, but to no avail.
I hope that helps? (Please let me know if you prefer to have server admin access, and I’ll be happy to set such an account for you.)
All the best,
Nicholas
Just an update that the database had been updated to 5.6. (Same issue remains, though.) Thanks.
Also, I didn’t realize the first post was public, so I deleted the screenshot of our admin dashboard (and moved it here). I was hoping the first post was private for support only.
Murat, Allow me to chime in here. With all due respect.
The product (WCV) works. If you want it to do more and it isn’t available, you just have to add the necessary products/extensions as needed. And be prepared to spend to get that functionality if need be.
Just because you want it to do something more– that isn’t available out of the box because it is beyond the (current) scope of the product– is not the product’s fault. It’s just that your needs require more specific elements that the product doesn’t have (at least not yet), and they are working on functions that are of a higher priority to more customers.
You’re already getting an amazing bargain with the product, and a lot of people have a million specific features they want for their own use/sites, that the product just can’t answer all for, but that the general product can try to accommodate for the most impact/use for the majority of users.
If a feature you want is not available, you can ask. And if it isn’t available then, you do like the rest of us and find a custom solution or buy what you need. But it isn’t the product or the developer’s fault because they advertised what you assumed the product to have.
+1 on Bookings integration to WCV. Been waiting for this a long time. Thanks and looking forward to this.
It would indeed be a worthwhile undertaking (something Woo/Automattic should do), and I’d reckon would do wonders for WCV overall. But alas, it’ll likely remain to be seen for a long time to come, and worthwhile undertaking may not necessarily translate to realistic, the same way I’ve seen other WP forks before (like the single-db MU-like fork one from ibiblio) getting shelved by the time Automattic comes around to integrating that functionality. Only question now is if they’d have this in the roadmap someday.
Thanks for clearing that up, Ben.
