- Issue created by @JoaoCoelho13
- ๐ต๐นPortugal JoaoCoelho13
I have uploaded the patch 'd10_previewer_once_fix.patch' as a trial. Since I am unable to test it in my environment, I am not sure that this patch works.
- ๐ฎ๐ณIndia dineshkumarbollu
Hi @JoaoCoelho13
The issue in the repo code is they updated the js code without jquery dependency and using once dependency in it. If we use js once code without Jquery dependency then we need to update the 'each' to 'forEach'.
once('previewerRefresh', '.js-previewerRefresh', context).each(() => {
to
once('previewerRefresh', '.js-previewerRefresh', context).forEach(() => {
Your patch is also correct with jquery dependency.
- ๐ณ๐ฑNetherlands interactivex
Thanks for the great job @JoaoCoelho13. Your patch is only missing the missing dist JS. I've added it to a new patch.
-
ThomasDik โ
committed 5f8d6732 on 1.0.x authored by
interactivex โ
Issue #3408154 by JoaoCoelho13, interactivex, dineshkumarbollu: jQuery...
-
ThomasDik โ
committed 5f8d6732 on 1.0.x authored by
interactivex โ
- Issue was unassigned.
- Status changed to Fixed
11 months ago 8:54pm 9 January 2024 - ๐ณ๐ฑNetherlands thomasdik
Hi All,
Thanks for the help!
This kind of slipped through when updating multiple modules.
I've added the latest version of the patch and released it.Kind regards,
Thomas Dik
Automatically closed - issue fixed for 2 weeks with no activity.