SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 In the configuration of the module, I assume you have Number of products set to > 0?
Chandra Posted March 30, 2018 Posted March 30, 2018 I have Number of products = 0 (all), since I would like all products in the review request and typically my orders don't go beyond 10-15 products. Also, since an order will always have some products, I am not sure if the cond ition in the above post ever will be met. I suspect the template got screwed, let me refresh it from the zip and update it once more.
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 I do think the problem is that line now, change it please...
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 yup, worked for me. sorry, my logic was a little off in that statement, I'll fix it and do another release. Thanks for the bug report!
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 New version out, with above bugfix. Just install over your current version. (thanks @Chandra for the bug report). Or, you could change that line like I asked @Chandra to do, either way. This only affects people that set Number of products to 0.
Chandra Posted March 30, 2018 Posted March 30, 2018 @SLiCK_303 Made the change to the line and now installing the new version to be in sync with the releases. Thanks for fixing it super quick!
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 no problem, sorry I caused the problem to begin with, bad me..... :smiley:
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 I assumed it worked for you too? Did ya test it?
Chandra Posted March 30, 2018 Posted March 30, 2018 I can't since I don't have a test region now and my cron runs each Friday for the orders of the week. Is there a field I can update on an existing order to push the email request once again by running the cron?
SLiCK_303 Posted March 30, 2018 Author Posted March 30, 2018 you could goto your db, edit tblogsrr_email table, and delete the entries for the dates that didnt work, then manually run the cron job. backup your database first, in case something goes wrong....
SLiCK_303 Posted April 2, 2018 Author Posted April 2, 2018 New version released. This one has a little better hints (and I mean little). Also it fixes some logic so you can pick 0 for either Send after or Send before. This way you can pick 0 on Send after, and it will send emails, that hit your picked order state/s, today. Also you can set Send before to 0, so it will send requests for all your orders, not just ones that are say, 30 days old. I don't necessarily think you should set Send before to 0, as it will send out a LOT of emails that way, but you do as you wish. I think I'm making it more confusing by trying to explain it, I think you know what I mean, by setting these options to 0..... ;)
Pedalman Posted April 3, 2018 Posted April 3, 2018 I would like to have a form where we could enter test emails. I know about doing a test order and a manual cron with proper settings but I like it the easy way.
SLiCK_303 Posted April 10, 2018 Author Posted April 10, 2018 Thanks @colorful-ant for the German translations of the BO and emails!
SLiCK_303 Posted April 10, 2018 Author Posted April 10, 2018 I released a new version. It changes the main sql statement to use dateupd instead of dateadd. That way it looks at the last time you changed the state of the order, instead of when it was created. As the last couple versions have changed the logic of the module, I recommend everyone updates to this version.
SLiCK_303 Posted April 11, 2018 Author Posted April 11, 2018 I released a new version. For all of you that use Genzo_Krona Loyalty module, I have added a new email that ties into it, stating how many points a user will get when they submit a review. Maybe it will add more incentive to post a review, knowing a person will gain loyalty points for it. The idea, as well as the German translation are thanks to @colorful-ant.
SLiCK_303 Posted April 12, 2018 Author Posted April 12, 2018 Oh, as a note on the Krona usage in my module, you need to have review_created active and setup in Krona. If this is an issue for most of you I will make a setting in my module to choose which action name to use.
SLiCK_303 Posted April 12, 2018 Author Posted April 12, 2018 Well, I released another version. I added a setting for the Krona/Revws action, so you can pick which one of the two (reviewcreated or reviewapproved) to use. I figured it was better this way, so a person doesn't need to modify my module to make it work with his/her setup.
Kleijn36 Posted May 8, 2018 Posted May 8, 2018 @SLiCK_303 Committed full translation in to Dutch on GitHub
nickon Posted June 11, 2018 Posted June 11, 2018 @SLiCK303 Hi, Two feature suggestions :smilingfacewithsmiling_eyes: 1. make module gdpr complaint 2. Add option to send meial based on delivery status. Eg send email 1 day after delivery status is "Delivered" Can be usefull as not all packages are delivered after the same dasy Kind Regars
SLiCK_303 Posted June 11, 2018 Author Posted June 11, 2018 @nickon 1) probably wont happen, sorry 2) Already is there
nickon Posted June 11, 2018 Posted June 11, 2018 @SLiCK_303 1. Realy sorry to hear that. I thought it would be easy to do as (I think) is only need to add the psgdpr hooks. We only need to know when the email is send. As I don't see it collecting emails itself. Mainly just as mailalerts does. 2. Yeah missed that option in the screenshot :-)
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now