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

Cannot find module './zxcvbn/zxcvbn' (coffee script problem) #77

Closed
dominictarr opened this issue Jun 10, 2015 · 5 comments
Closed

Cannot find module './zxcvbn/zxcvbn' (coffee script problem) #77

dominictarr opened this issue Jun 10, 2015 · 5 comments

Comments

@dominictarr
Copy link

This module needs to ship with prebuilt javascript code, otherwise it doesn't work without coffeescript.
the simplest way to do this would be to call a build step from within a prepublish script in the package.json.

here is the error I get

$ npm install zxcvbn
$ node
> require('zxcvbn')
Error: Cannot find module './zxcvbn/zxcvbn'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/home/dominic/c/experiments/node_modules/zxcvbn/index.js:1:80)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
@binarykitchen
Copy link

agree with that and not minified please - in my case i need it to debug other zxcvbn related bugs ...

@binarykitchen
Copy link

@dominictarr you can use my npm-published fork until it's fixed https://www.npmjs.com/package/zxcvbn-npm

@lowe
Copy link
Collaborator

lowe commented Jul 13, 2015

npm fix coming soon -- I'm working w the original publisher to get access
to the registry.

On Sunday, July 12, 2015, Michael Heuberger notifications@github.com
wrote:

@dominictarr https://github.com/dominictarr you can use my
npm-published fork until it's fixed
https://www.npmjs.com/package/zxcvbn-npm


Reply to this email directly or view it on GitHub
#77 (comment).

@lowe
Copy link
Collaborator

lowe commented Jul 14, 2015

I fixed the npm package and updated the README -- this should just work now.

@binarykitchen for debugging I would recommend the setup described in the README vs directly including the zxcvbn.js bundle (whether minified or not).

@lowe lowe closed this as completed Jul 14, 2015
@binarykitchen
Copy link

Your updates are good, thanks. But there are still things you need to change. Will write new tickets.

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