brinchj / RndPhrase

RndPhrase: Auto-generated secure passwords.
BSD 2-Clause "Simplified" License
21 stars 4 forks source link

Make a rndphrase.js on www so it is possible to include it from appspot #11

Open rlindsgaard opened 11 years ago

rlindsgaard commented 11 years ago

Instead of the rndphrase javascript is included in index.html it should be located in a separate script

Munter commented 11 years ago

I suggest making it a proper module with an UMD wrapper and also release it on npm and bower. That gives users of the library maximum flexibility

Munter commented 11 years ago

I can take a stab at this. My idea is to split the javascript itself into a seperate repository. RndPhrase.js would be a good name. This repository contains all parts of the current RndPhrase repository that can be run in a browser or in node. It will be exposed in npm an in bower as a seperately versioned module, and the current RndPhrase browser extension repository will change it's build system to fetch the new javascript module with one of these package managers pre-build.

rlindsgaard commented 11 years ago

You took my idea a little bit further. :) I'm working on making the module work https://github.com/rlindsgaard/RndPhrase/tree/commonjs and putting

off-topic: Making a new repository is a good idea, shouldn't we get around to create an organization to keep the projects in one place? I'd be happy to create it and invite people interested.

Munter commented 11 years ago

Yes. RndPhrase org.

I'm porting cubehash to AMD/CommonJS right now so it can be pulled in using a proper package manager. We can plug that in in a minute.

Munter commented 11 years ago

Done with cubehash: https://github.com/Munter/cubehash.js

Can now be used via npm and bower