SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 In the configuration of the module, I assume you have Number of products set to > 0? Link to comment Share on other sites More sharing options...
Chandra Posted March 30, 2018 Share 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. Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 good plan Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 I do think the problem is that line now, change it please... Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share 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! Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share 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. Link to comment Share on other sites More sharing options...
Chandra Posted March 30, 2018 Share 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! Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 no problem, sorry I caused the problem to begin with, bad me..... :smiley: Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 I assumed it worked for you too? Did ya test it? Link to comment Share on other sites More sharing options...
Chandra Posted March 30, 2018 Share 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? Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share 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.... Link to comment Share on other sites More sharing options...
Chandra Posted March 30, 2018 Share Posted March 30, 2018 It worked, thanks so much! :) Link to comment Share on other sites More sharing options...
SLiCK_303 Posted March 30, 2018 Author Share Posted March 30, 2018 np, happy to help. Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 2, 2018 Author Share 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..... ;) Link to comment Share on other sites More sharing options...
Pedalman Posted April 3, 2018 Share 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. Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 10, 2018 Author Share Posted April 10, 2018 Thanks @colorful-ant for the German translations of the BO and emails! Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 10, 2018 Author Share 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. Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 11, 2018 Author Share 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. Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 12, 2018 Author Share 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. Link to comment Share on other sites More sharing options...
colorful-ant Posted April 12, 2018 Share Posted April 12, 2018 @slick_303 it works just fine, thank you Link to comment Share on other sites More sharing options...
SLiCK_303 Posted April 12, 2018 Author Share 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. Link to comment Share on other sites More sharing options...
Kleijn36 Posted May 8, 2018 Share Posted May 8, 2018 @SLiCK_303 Committed full translation in to Dutch on GitHub Link to comment Share on other sites More sharing options...
nickon Posted June 11, 2018 Share 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 Link to comment Share on other sites More sharing options...
SLiCK_303 Posted June 11, 2018 Author Share Posted June 11, 2018 @nickon 1) probably wont happen, sorry 2) Already is there Link to comment Share on other sites More sharing options...
nickon Posted June 11, 2018 Share 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 :-) Link to comment Share on other sites More sharing options...
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