search

neos / form-builder

Flow Form Framework integration into Neos CMS
GNU General Public License v3.0
18 stars 31 forks source link

TASK: Remove plow-js from PlaceholderInsert plugin #141

Closed markusguenther closed 1 month ago

markusguenther commented 1 month ago

Since we changed the build stack in the Neos-UI to esbuild we are able to use the optional chaining (?.) operator in JavaScript. As the modern JavaScript is pretty powerful, we don’t need the helper methods from the plow-js library anymore.

The library served us well and has been deprecated, we also throw console messages when we still use the functions from plow-js. To prevent bloating the console with Neos packages, we should remove the usage also in our own plugins. With this PR we use esbuild instead of webpack for the Plugin and drop plow-js.

bwaidelich commented 1 month ago

image

woohoo, thanks a lot :)

I assume that this works as expected, otherwise we can always tweak it in a follow-up