forwardemail / email-templates

Create, preview (browser/iOS Simulator), and send custom email templates for Node.js. Made for @forwardemail, @ladjs, @cabinjs, @spamscanner, and @breejs.
https://forwardemail.net/docs/send-emails-with-node-js-javascript
MIT License
3.67k stars 337 forks source link

Unable to install #243

Closed mrbinr closed 7 years ago

mrbinr commented 7 years ago

Hi, I've tried to install package with npm install email-templates --save but npm returns error :

npm ERR! Error while executing: npm ERR! /usr/bin/git ls-remote -h -t ssh://git@github.com/niftylettuce/juice.git npm ERR! npm ERR! fatal: failed to stat '.': Permission denied npm ERR! npm ERR! exited with error code: 128

Other package have been installed without problem, but not this one ..

Thanks for your help, Best Regards,

Pascall

OmgImAlexis commented 7 years ago

What version of npm are you on? Have you tried deleting the node_modules directory and running npm install?

mrbinr commented 7 years ago

npm -v : 5.4.2 removing node_modules has no effect

niftylettuce commented 7 years ago

@mrbinr I've fixed the issue, please try to install npm install --save email-templates@3.0.4

The reason why it was failing was because your system was trying to download it via git+ssh, and it needs to be git+https.

Unfortunately NPM has an issue per https://github.com/npm/npm/issues/18520 and you probably won't be able to use NPM probably due to this error output (at least on npm --version = 5.4.1). You'll probably have to downgrade npm or use yarn instead (which you should anyways, npm is horrid nowadays).

λ ~/Public/email-templates/ master* npm install

> typechecker@2.0.8 preinstall /Users/nexus/Public/email-templates/node_modules/extract-opts/node_modules/typechecker
> node ./cyclic.js

npm ERR! Cannot read property 'pause' of undefined

I'd recommend installing via yarn, see https://yarnpkg.com/en/docs/install.

mrbinr commented 7 years ago

Still not working with yarn

yarn add v1.1.0 warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected >behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || ^8.0.0" info No lockfile found. [1/4] Resolving packages... [2/4] Fetching packages... error get-paths@0.0.1: The engine "node" is incompatible with this module. Expected version ">=8.3". error Found incompatible module info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

niftylettuce commented 7 years ago

@mrbinr You're going to need v8.3 of Node anyways, so you might as well upgrade now.

I did however fix the issue with get-paths already, so you can try npm install --save email-templates@3.0.5 to get v3.0.5.

niftylettuce commented 7 years ago

Also hopefully to prevent other users from this headache, I've added a note regarding Node v7.6.0+ requirement to README per https://github.com/niftylettuce/email-templates/commit/5260d65d56c259af8d58ddbaee6912da13463d16.

niftylettuce commented 7 years ago

can you please try v3.1.5? npm install email-templates@latest or yarn add email-templates@latest. this package now supports Node v6.4.0+. I've also updated README with this note that v6.4.0+ is a requirement (instead of 7.x) also please see v2 to v3 upgrade guide at https://github.com/niftylettuce/email-templates#v3-breaking-changes

kunal-dethe-v2sol commented 6 years ago

Sometimes it even requires to npm install --save multer and npm install --save formidable after npm install --save email-templates@3.1.5

niftylettuce commented 6 years ago

@kunal-dethe-v2sol What are you talking about? This package does not require multer nor formidable. Please refrain from commenting without posting a full stack trace log or error output from npm.