- Issue created by @IT-Cru
- last update
over 1 year ago 12 pass - @it-cru opened merge request.
- last update
over 1 year ago 12 pass - last update
over 1 year ago 12 pass - Status changed to Needs review
over 1 year ago 7:53am 17 May 2023 - 🇩🇪Germany IT-Cru Munich
Changes of MR solve the issue that only 1 file was added to Image Style Warmer queue.
I don't know if this is really a good idea, because if a lot of files should be warmup (for example: from bulk upload, bulk warmup action or warmup via drush) we maybe run into PHP memory limit issues.
Maybe it also would be better to run only initial warmup from destruct() and add each file to queue outside of destruct() method.
- last update
over 1 year ago 12 pass - last update
over 1 year ago 12 pass - last update
over 1 year ago 12 pass - last update
over 1 year ago 12 pass - 🇩🇪Germany IT-Cru Munich
I think adding to queue and also collecting files for initial warmup could be improved to reduce duplicate file processing here, but I will add this improvement at some later point in follow up release 8.x-1.3 to get first a stable D10 release with 8.x-1.2.
I hope this will be fine for all.
- Status changed to Fixed
over 1 year ago 3:01pm 23 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.