Olly

Forum Replies Created

Viewing 20 posts - 841 through 860 (of 4,446 total)
  • Author
    Posts
  • in reply to: Content of last checkout page is doubled #45123
    Olly
    Admin & Mod

      having had a quick test here, i get the same result (i.e multiple checkout pages if using that shortcode on the order page)
      as – in large parts and for consistency – that particular shortcode uses the same coding as the actual orderpage I really do not know if this is doable, but i’ll give it a go but as mentioned I cannot tell you when or if this will be possible I’m afraid…

      in reply to: Content of last checkout page is doubled #45122
      Olly
      Admin & Mod
        This reply has been marked as private.
        in reply to: Content of last checkout page is doubled #45120
        Olly
        Admin & Mod

          ok. thanks for narrowing things down
          the shortcode [wppizza type=orderhistory] is really designed to be on its own page.
          that said, i can somewhat see why one might perhaps want this to be on the orderpage too (though generally – in myopinion only though admittedly – once a customer is on the orderpage I am not sure if it’s such a great idea to confuse the customer with more stuff on the page that might stop him/her from completing the order right there and then . again, just my 2 cents)

          In any event, i’ll see what i can do. it will have something to do with some ajax call executing multiple times, my guess being you get 3 times that page if you had 3 previous orders, 4 times if you had 4 previous orders and so on

          Cannot give you an ETA though right now when (or even if) this will be doable I’m afraid

          in reply to: Content of last checkout page is doubled #45114
          Olly
          Admin & Mod

            without a url i cannot say anything other than the usual:
            i.e do you get the same issue when truning off all other plugins and using a default theme ?

            regarding cloudflare, can you send me a screenshot of what you are seeing (as i need some info displayed there to help to track things down , thanks)

            in reply to: Confirm SMS Text #44906
            Olly
            Admin & Mod

              You won’t find it in the “WordPress Environment” or in any directory. That’s not how WordPress filters work .
              You must familiarize yourself with this

              see here for a starting point
              https://docs.wp-pizza.com/developers/?section=filters-actions-functions
              – or in fact any WordPress actions/filters tutorial you can find with a quick search

              (at the most basic level, the filter/code above belongs into your [child]theme’s functions.php )

              in reply to: Confirm SMS Text #44876
              Olly
              Admin & Mod

                I assume you are referring to the Confirm/Reject/Notify plugin (you did not say) so I created a forum(s) for it that did not exist. Thanks for pointing that out.

                Regarding your question, the CRN plugin has a filter you can use to add your branding like so (as you know, this will only work if using one-way communications):

                
                add_filter('wppizza_crn_sms_branding', 'mysmsbranding');
                function mysmsbranding($sms_sender_id){
                /* 
                overwrite sender id as required (here set to MyShop123 )
                a-zA-Z0-9 (and spaces) only, 11 characters max
                */
                $sms_sender_id = 'MyShop123';
                return $sms_sender_id;
                }
                

                (not tested as such, but should work I would think)

                in reply to: Number of PayPal order per month #44689
                Olly
                Admin & Mod

                  simply go wppizza->reports , set the range you want and export
                  (PS: I am aware that – currently – the Range selector dropdown at the top left on that page only works when using the keyboard and not when using the mouse to select from one of the available options. That will be fixed in the next update of the plugin)

                  in reply to: Directions list for users #44601
                  Olly
                  Admin & Mod

                    WPPizza adds (additional – depending on what you have enabled in wppizza->order form) user fields to default wordpress fields via the filters wordpress has available (as you can see in your admin->users->your profile)

                    If the registration plugin does not display those fields you will have to speak to the author of that plugin .
                    My *guess* would be – though I cannot be sure as i know nothing about that plugin – they are simply using the default fields wordpress has but completely ignoring filters that allow to add to those )

                    in reply to: Directions list for users #44588
                    Olly
                    Admin & Mod

                      Sorry,but I dont understand what exactly you mean by “manage the user directions” (looking at your IP i assume you are referring to a user address ?! )

                      what exactly are you trying to manage ?

                      in reply to: PAYPAL NOT RECOGNISING ADDRESS PASSED TO IT #44586
                      Olly
                      Admin & Mod

                        >Is this likely to be a configuration issue at my end
                        No.

                        In fact it was quite a deliberate decision to completely disable the whole shipping address input when being redirected to paypal as – in my opinion – it only confuses the issue of paying for an order.

                        That being said, there is also a filter available in the plugin/gateway that allows to to override this behaviour as it suits your needs (i assume here you know what wordpress actions/filter are and how they work).

                        As a starting point you could do this sort of thing
                        you should be able to adjust things from here as you need them to be in your particular circumstances
                        see
                        https://developer.paypal.com/docs/classic/paypal-payments-standard/integration-guide/Appx-websitestandard-htmlvariables/#paypal-checkout-page-variables
                        for parameters paypal accepts

                        
                        add_filter('wppizza_filter_paypal_parameters', 'myprefix_paypalfilter', 10, 2);
                        function myprefix_paypalfilter($parameters, $order_formatted){
                        /* will revert the value to the default of 0 */
                        unset($parameters['no_shipping']);
                        /* alternatively override to the value you want there like so  */
                        $parameters['no_shipping'] = 0 ; /* 0. Prompt for an address, but do not require one */
                        /*
                        setting some default values according to your settings, using the values returned in $order_formatted.
                        do a print_r/vardump $order_formatted to see all order parameters . It should be obvious what is what
                        */
                        $parameters['last_name'] = $order_formatted['customer']['cname']['value'];
                        $parameters['address1'] = $order_formatted['customer']['caddress']['value'];
                        $parameters['email'] = $order_formatted['customer']['cemail']['value'];
                        /* and so on */
                        return 	$parameters;
                        }
                        

                        Hope that helps

                        in reply to: changed price position after update #44554
                        Olly
                        Admin & Mod

                          it really all depends on the amount of content you have and the device used (i.e ratio / width the screen has)
                          i dont know anything about the Mate20 Pro.
                          However ,having played with a few things, it seems to me it has a width of about 430 or so

                          i can see 2 options.
                          a) the media queries in the original css kick in at 420 i.e it’s set to do

                          @media only screen and (max-width: 420px){/* css stuff */}
                          you could change this to 440 for example (or something around that number). i.e
                          @media only screen and (max-width: 440px){/* css stuff */}
                          and the whole reshuffeling of things will occur earlier

                          another option would be to restrict the width that the title can take up , before it starts to linebreak
                          so, something like

                          @media only screen and (max-width: 440px) and (min-width: 420px) {.wppizza-article-h2{display:inline-block; max-width:120px}}

                          or similar – you’ll have to play with it a bit and or target only the items/categories where you need it

                          (or – as mentioned previously – use the “responsive style” layout to start off with which will eliminate a lot of this messing around. But I understand that you might not want to do that of course. )

                          PS: I would also suggest you add the float: none !important; i referred to earlier in this thread to center things under 420
                          but that’s up to you (if you dont like it centered that also just fine of course)

                          in reply to: changed price position after update #44548
                          Olly
                          Admin & Mod

                            generally that all sounds good.
                            just out of interest though, what phone are you using there you have an issue with ?

                            in reply to: changed price position after update #44527
                            Olly
                            Admin & Mod

                              coming back to the added css i suggested , you might even want to add this
                              float: none !important;
                              so the whole thing would read like this

                              
                              .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}
                              @media only screen and (max-width: 420px){
                              .wppizza-article-default .wppizza-article-sizes {position: unset !important;float: none !important;}
                              }
                              

                              that will keep th eprices centered on small screen devices (if you want that). obviously , you can tweak this as you wish

                              in reply to: changed price position after update #44526
                              Olly
                              Admin & Mod

                                PS: the easiest was/is always to simply use wppizza->layout->styles->responsive
                                (which always displays the title above everything else)

                                However, I just happen to like the default layout myself (and alwasy did), but it does need a lot of css to make it work on different devices as it also depends on how many sizes there are for an item etc …
                                (again, that’s why the changes were made in 3.10 – to make it as usable as possible “out of the box” with any – or at least most – themes)

                                …………

                                yes, i try to keep these relatively drastic changes to an absolute minimum, but it sometimes cannot be helped (to make things better for all) and if I do, I’m happy to hear about (and solve if at all possible) any issues

                                (Just felt the need to say this, lol)

                                in reply to: changed price position after update #44525
                                Olly
                                Admin & Mod

                                  >I’ve rolled back to a backup before the update, so that you can see, how it was and I want it to have in the future as well
                                  the thing is, i dont think you want it like that .(hear me out)
                                  have a look at it on a small screen , you will see that yoru prices are *above* the title.

                                  i have had quite a few reports where people ordered the wrong thing becuase they clicked on the price BELOW the title assuming – as would be pretty much expected throughout any ecommerce site – that the price below belongs to the title above it.

                                  this is precisely the reason why this got changed ……

                                  if you use the new version and add the below as the last of your custom css

                                  
                                  .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}
                                  @media only screen and (max-width: 420px){
                                  .wppizza-article-default .wppizza-article-sizes {position: unset !important;}
                                  }
                                  

                                  you will get the benefits from the above problem not happening on small screen devices , plus your “original” so to speak (i.e desktop/bigger screens) to be unaffected….

                                  in reply to: changed price position after update #44524
                                  Olly
                                  Admin & Mod

                                    i.e when resetting (i’m writing this out here – as reference to myself too – as i may even be adding this or similar as a default to the next update )

                                    
                                    @media only screen and (max-width: 420px){
                                    .wppizza-article-default .wppizza-article-sizes {position: unset !important;}
                                    }
                                    in reply to: changed price position after update #44522
                                    Olly
                                    Admin & Mod

                                      i.e you are doing

                                      .wppizza-article-default .wppizza-article-additives{ display: block; /* etc */}
                                      if you do/revert to
                                      .wppizza-article-default .wppizza-article-additives{ display: inline-block; /* etc */}
                                      additives are next to the title (as they are by default) and things dont move around all over the place

                                      if you DO want the additives to be below the title (i.e keep the display:block)

                                      then you could *add* this
                                      .wppizza-article-default .wppizza-article-sizes { position: absolute;top: 0;right: 0;}

                                      BUT MAKE SURE you reset/remove this one again below 420px (as that’s where the media query kicks in that moves things underneath each other to cater for mobile devices )

                                      in reply to: changed price position after update #44521
                                      Olly
                                      Admin & Mod

                                        > the additives are standing now directly after the article title
                                        i’m not with you.
                                        that’s the default behaviour (but you have overridden that span with display:block and whatnot which in turn pushes everything after down)

                                        it also resizes quite nicely when looking at it on a mobile device. i.e title above additives above prices – as intended and in fact the whole reason why the css/elements order was changed in wppizza 3.10

                                        you sure you are not looking at some sort of cached version in your browser ?

                                        in reply to: changed price position after update #44514
                                        Olly
                                        Admin & Mod

                                          >But with the custom css it worked very fine over a year and some updates
                                          sure , as unless it is strictly necessary (and in this case it was unfortunately) , i do not change these kind of things

                                          that aside.
                                          i get a a ton of javascript errors on you site (you have done something like this <!--?php . that’s really not a good idea as it will still be read and parsed in all sorts of places)
                                          i suspect you did that to disable your custom css (as i asked , and thanks for that)
                                          However, looking at what we have now, i think it will be better in this case if you just leave things as you had them – including the css you were overriding – and I will see if i can come up with a line or 2 of css you can additionally add (or change) to get you back to where we were from a layout perspective

                                          in reply to: changed price position after update #44512
                                          Olly
                                          Admin & Mod

                                            the order of the actual wrapping div, ul etc elements (prices, content text, title etc ) was changed slightly to make it possible to make things fit on a screen (affects default style only) when viewed on small screen devices while adding/changing *some* of the css to make this look as it did before

                                            this was simply not doable before dynamically using media queries css and in mamy cases will have resulted in quite a messed up screen when using mobile devices..

                                            that’s just as an explanation

                                            with your multitude of additives for your italien salad, this might well cause an issue and i’ll look if i can account for this too.
                                            In the meantime though – might also be the case – are you using any custom css on those elements ? perhaps simply removing any custom css there will also “fix” it so to speak (one might be overriding the other)

                                            I’ll have a look here too if i can reproduce your layout and see what happens, but if you could also check any custom css you might have and let me know , that would be great

                                            Could you also send me the url of your site so i can have a look at the actual css that is being applied ?

                                            thanks

                                          Viewing 20 posts - 841 through 860 (of 4,446 total)