mjml2json icon indicating copy to clipboard operation
mjml2json copied to clipboard

Bump MJML dependency version

Open arokanto opened this issue 5 years ago • 4 comments

This package still relies on mjml version ^3.2.2. When installing you get the following warnings:

warning mjml2json > mjml > mjml-cli > [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
warning mjml2json > mjml > mjml-cli > chokidar > [email protected]: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
warning mjml2json > mjml > mjml-button > react > fbjs > [email protected]: core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.

Maybe the mjml dependency should be bumped to something on the 4.x line.

arokanto avatar Sep 07 '20 07:09 arokanto

Any news on this? mjmj 3.2 has 279 low-severity vulnerabilities. All of them are because of outdated versions of node-fetch and ini, which are required by transitive dependencies of mjml 3.2.

Do you have plans to update this module to require mjml 4? For me mjml2json is still a useful module.

arokanto avatar Mar 03 '21 08:03 arokanto

I tried looking at this, but there are quite a few breaking changes and not all seem to be documented, so it requires some work looking into the internals of mjml: https://github.com/mjmlio/mjml2json/pull/25

stianjensen avatar Jun 07 '22 22:06 stianjensen

I provided #29 that works and replaces #25.

istr avatar Dec 20 '23 13:12 istr