Forum Replies Created
-
AuthorPosts
-
9 January, 2024 at 11:01 am in reply to: Reorder fonctionality not working when load JS in foo #65692
where can i see this please ?
works fine herePS: I’ll double check the messages displayed though. (1) Never really looked at that in too much detail I believe
re 1+2:
you are ignoring the fact that opening times can cross midnight. So, no, on both countsre 3:
The holidays option in the main plugin was added relatively recently and the current version of the preorder plugin does not account for that. I completely agree that I need to account for the holidays settings there too.
Thanks for pointing that out (Cannot give you a distinct ETA yet as to when this will be done, but hopefully not before too long)>….I can post this on the extension forum if necessary…
No need, but thank you for the consideration>You are still the best in the field.
I am most certainly not, but thanks for the vote of confidence anyway ….you are looking at the wrong plugin
i am talking about
https://support.wp-pizza.com/downloads/wppizza-delivery-by-postcode/
(just happens to have the same verion number now as the ingredients plugin)>the popup is working as I intended
the funny thing is, it doesnt actually work as intended at all in this case. it’s only becuase it throws severe errors, that it *seems* to be working as due to this error , the popup does not show …..(hence it took a bit of time to figure out what’s going on)in any event, I believe 5.2.2. has this now fixed (might take a little time to show up in your dashbard as being available as an update though)
thanks for your patience
yup yup, understood.
should be relatively easy to fix (though was all a bit obscured as the ingredienst plugin – using the same popup script library – was masking things somewhat )Hi Umut
can I double check something with you quickly :
The problem isn’t actually that the popup is empty but that it is there at all . correct ?
i.e your setting say (paraphrasing) : “only open on pages with wppizza items”
as your homepahe does not have any wppizza items on there, this popup should simply not “launch” – so to speak. Right ?(Am just trying to avoid going down the wrong rabbit hole …)
quick update:
I can reproduce this here too with a clone of your site (that’s a good thing) Just need to find out where the issue is – but that will have to wait ’til tomorrow as it’s kind of lateThis reply has been marked as private.This reply has been marked as private.Some more info re sessions: https://docs.wp-pizza.com/troubleshooting/?section=headers-already-sent
>I’ve previously reported the issue once here,
sure , and I responded by saying you need to fix the session_start issue you are having . From what I can see (assuming we are talking about the same site still and looking at your debug log there ) it is still an issue on that site
(naturally, I have double-checked this here with the settings you provided above and it works just fine)I believe this is fixed now with v3.18.5
Thanks for letting me knowthanks,
am I right in thinking that you only get this issue when setting a custom range (i.e settings the 2 distinct from/to dates) in the reports section ?you need to enable debug on your site and show me the debug.log
(https://docs.wp-pizza.com/troubleshooting/)
You *might* have to disable the fatal error handler too to get meaningful log entriesi.e in your wp config add
define('WP_DISABLE_FATAL_ERROR_HANDLER', true);Have a look in the plugin itself (sub)-directory /add-ons/gateway-development/ to see how to add your own gateway implementation (refer to the comments in the files).
However, if your question is actually about me doing this for you, I will have to disappoint unfortunately as I cannot offer customisations / one-off development .
Please see here https://docs.wp-pizza.com/faqs/?section=customisation.Oliver
I do not think I understand.
You updated to 3.8.4 and it’s working , so what is the issue ?So, I’ll have an update later today for this.
That said, just ignore my explanation regarding “>Issue 1” above. That’s not how it works and what I said is simply wrong.
The time do NOT get added to each other but are independenti.e “Minutes after opening” applies only to the (re)opening times. (e.g opening at 8:00, 45min time set, 8:45 earliest available. once it’s later that that , the 45min become irrelevant)
Preparation times: always added to current time as earliest available slot (provided shop is open).
Naturally, “minutres afer openeing” and “Preparation times” might compete – so to speak – early in the day , so longest delay will applyabove just as a – hopefully understandalbe – explanation how this works if someone reads this thread
Thanks for the details – helps a lot.
Referring to your original post:
>Issue 2
I can reproduce the same thing here and agree it’s not doing what it is supposed to be doing
Am pretty certain this worked as intended at some point, so something got lost somewhere in an update. My fault and will be fixed asap (later today / tomorrow I would have thought)>Issue 1
(assuming here the “issue 2” problem being fixed)
This works as it should from what I can make out.
I.e opening at 17:30.
preparation time 40 minutes , minutes after opening 40 min
== total 80 min after 17:30 opening -> 18:50point being: if you are saying delivery takes 40 minutes and preparation takes 40 minutes, customer will not be able to receive anything before 80 minutes after (re)opening .
The reason why there’s a pickup/delivery distinction in preparation times is because someone asked for it at some point, but I cannot remember why this made sense to be honest. Suffice to say I would not have added it if the case had not been made to do so.
In any event, you can of course set the times as they suit you and – I admit – the explanation next to each option could be more descriptive . I’ll see what I can do there. -
AuthorPosts