datakick

I believe @lesley is planning to move away from patreon to some different solution. What exactly is not 'handy for business user' ?

There are multiple commits [ 1 , 2 , 3 ]related to this, you'll need all 3. Then you get this:

The fix for this, including new back office switch, is scheduled for the next release. @Smile there's no need to sponsor individual fixes, especially when they are trivial 🙂 But feel free to join patreon program (if you haven't already), that's always appreciated!

The bug that was causing 500 error code was fixed, so Full Page cache should be usable in the upcoming version. Note that by default every hooks are non-cached, so when you turn it on, you might even experience some slowdown. To get the benefits of FPC, you will need to turn on caching for hooks that *you believe* should be cached. And this is the hard part -- nobody knows if the hook has some side effects other than rendering the output. If it has side effects, turning caching on will break stuff. So you need to be very careful, and test everything.

Tracked here: https://github.com/thirtybees/thirtybees/issues/848

It is a bug, but it's also very specific. You need to have multistore enabled, and distinct set of languages used by each stores. With this configuration, tb will indeed emit alt links for languages used in other stores. This should be fixed. Should thirtybees emit alt link for different store? That's questionable. Probably not, imho. While it would make sense for your specific use case, when you use multistore as a language selector, it would not work in general. Especially when stores have their own products. I also agree that there should be a switch to turn off this feature, and a hook to alter alt links just before they get rendered to the html.

The auto-login functionality exists only in premium version, and it's tied up to email record. Every sent review request email is tracked in database, and it includes private security key needed to perform the auto login. I've tested on my environment, and image aspect ratio is maintained correctly. I've tested with image sizes 150x300 and 300x150, and in both scenarios the result was ok. It's possible that you have modified/outdated revws css, or that some other css is applied to review list as well.

Another free version 1.0.23 released Enhancements Issue 86 - custom placement for review average - you can now position stars/ratings widget anywhere on your product page. For example prominently below product title Issue 97 - added shortcuts from settings page to reviews, import, criteria pages Issue 100 - added configuration option for number of reviews to be displayed on all-reviews / my-reviews page Issue 48 - you can now edit review date from back office Issue 47 - review lists can now be filtered by category and brand. Bugfixes Issue 90 - fix pagination problem that occured on some versions of MariaDB without proper support for SQL_CALC_FOUND_ROWS Issue 96 - fixed overflow problems with long review content Issue 20 - read review link now works in quick view Issue 99 - don't filter reviews on my-reviews page by selected language Tip and tricks Try adding this short snippet somewhere into your smarty template. It will display product reviews associated with category 4 {hook h='displayRevwsReviewList' category=4 }

what do you mean change the classes/tools.php ?

I believe that new core updater module handles database migration as well: https://github.com/thirtybees/coreupdater/blob/e58130881076265a0bc1a

That's correct. It's stupid, isn't it 🙂

You need to run your store on https

migration module does not replace any module, unless it's one of the few known incompatible. It's up to merchant to decide if he wants to upgrade to Thirtybees native module, or keep prestashop version

@Séane it's possible to override module's primary file, so you are in luck. Just create file overrides/modules/blocktopmenu/blocktopmenu.php with content like this: class BlocktopmenuOverride extends Blocktopmenu { protected function generateCategoriesMenu($categories, $isChildren = 0) { // your custom implementation } }

Yes, this is already in 1.0.8. Please keep the issue opened, as traumflug wants to use it to track the original performance problem

I'm just wondering what this has to do with gdpr module...? Anyway, the code is pretty much ok. I would drop $this->need_instance = 1; because it's tells thirtybees to instantiate this module every time it renders Modules page in back office, which can slow it down a bit. But other than that, there's nothing wrong with it. And if you have your skroutz javascript on your order confirmation page, then the module must be installed properly.

The easiest way is to edit Module.php file, and just before the line where it failed add this code: die((new ReflectionClass('Validate'))->getFileName()); Then refresh your front page. Instead of your beautiful thirtybees shop you will see file path to the Validate class. Don't forget to remove this line after 🙂

Also, you can use Override Check module to quickly find all orphaned overrides. This module is available to install in your backoffice

Looks like it is indeed obsolete, and you should be able to remove it. These overrides remnants are, unfortunately, common problem with module upgrades. Another question is why is there an override for sendChangedNotification method. This method only triggers actionOrderEdited hook. By overriding this method this hook is not executed, which might cause problems with other modules integration.

https://github.com/thirtybees/thirtybees/commit/8e89acf9061e88bc106658891c8bd5abb83eeb9c

There's a bug in 1.0.8 regarding saving Conifguration, that's probably it. Since you are using gitupdater, you can try to forward your shop to bleeding edge. It's fixed there

I've tested mollie's ps module version 3.3.5. Klarna payment method in test mode seems to be working correctly for me. Question is, if the webhook is triggered in production mode at all. Those of you who are having issues with Klarna -- please go to Mollie module, and set LogLevel to Everything. With this option enabled, there should eventually appear some Mollie entry in your Advanced Parameters > Logs Can somebody confirm such entry recently?

I believe you can expect next version very soon. Maybe even in two weeks time. And after the new @Traumflug's core updater is released, you will be able to deploy latest changes almost immediately

This is definitely a good idea. The implementation is tricky, though. The best way would be to have dedicated task queue running in separate process. But that would require your own server, and it would not be trivial to configure. Alternative is cron-based task queue. There are always some problems with crons, timeouts, etc. But it's definitely a good thing to think about. Especially if we combine this with task scheduler to run maintenance tasks

As others said -- full page cache has few problems in 1.0.8. These are fixed in upcoming version. But back to your question - when you enable debug mode in FPC, then there will be special http header in response with information if cache was hit or missed. That's all it does.