Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support for mjml 4.1.0 #32

Closed
alebacca89 opened this issue Jul 2, 2018 · 9 comments
Closed

support for mjml 4.1.0 #32

alebacca89 opened this issue Jul 2, 2018 · 9 comments

Comments

@alebacca89
Copy link

alebacca89 commented Jul 2, 2018

The last version of mjml is 4.1.0 but mjml-rails not recognise this bin.
If I install mjml 4.0.0 with npm, npm install 4.1.0. (npm install [email protected])
If I use mjml-rails setup in a initializer and I put

Mjml.setup do |config|
config.mjml_binary_version_supported = "4.1."
end

not works.

@sighmon
Copy link
Owner

sighmon commented Jul 3, 2018

@alebacca89 did you mean to put this in the config:

Mjml.setup do |config|
  config.mjml_binary_version_supported = "4.0."
end

@sbiastoch
Copy link

Same problem here, when I install [email protected], mjml --version gives me 4.1.1
Seems like an old issue, see mjmlio/mjml#371

Setting mjml_binary_version_supported does not help neither.

@sighmon
Copy link
Owner

sighmon commented Jul 20, 2018

@biastoch2 are you installing it locally or globally? try npm install -g [email protected] and then let me know what mjml --version gives you.

@sbiastoch
Copy link

I tried both, locally and globally, as well as uninstalling, force cleaning the npm cache and reinstalling mjml. Always mjml --version gives

mjml-core: 4.1.1
mjml-cli: 4.1.1

I also tried installing [email protected] and [email protected] but still 4.1.1.

@sighmon
Copy link
Owner

sighmon commented Jul 20, 2018

@biastoch2 Oh I see - I just got home to try it and have the same result. Would npm installing from git work for the tagged release 4.0.5?

(I don't use npm much, so just guessing)

@sbiastoch
Copy link

No luck, there is no version-attribute in the package.json present, such that npm install mjmlio/mjml#v4.0.5 (which installs mjml directely from github) fails with:
npm ERR! Can't install github:mjmlio/mjml#ecfb19d227f232a29b7f4059849f3aa01ce22386: Missing package version.

@sbiastoch
Copy link

I was able to get mjml back to work using the master-branch of mjml-rails in my Gemfile. The master branch uses mjml in version 4.1., but when bundle install, rails pulls the latest version of mjml-rails, which is 4.0, which references mjml 4.0., which is due to the described bug of mjml currently not installable. A new release of mjml-rails would temporarily fix this problem until the releases get out of sync again.

Still weird, that the manual change of mjml_binary_version_supported from inside the initializer does not work...

@sighmon
Copy link
Owner

sighmon commented Jul 21, 2018

@biastoch2 I don't think I understand your problem - the latest version of mjml-rails in Rubygems is 4.2.0. Does it update to 4.2.0 when you do a bundle update?

@sighmon sighmon closed this as completed Aug 29, 2018
@sighmon
Copy link
Owner

sighmon commented Sep 1, 2018

Wondering if this was related to #36 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants