gulp scaffold: fix issues and improve

unic / estatico

[DEPRECATED] Estático – Static site generator for frontend unicorns

Other

121 stars 18 forks source link

gulp scaffold: fix issues and improve #16

Closed tschuems closed 7 years ago

tschuems commented 7 years ago

Like @backflip said in #15 fix issues with JS class name

prettify scaffolding by adding new variables. now you can choose between camelCased-, pascalCased-, original- and lowerCased-Name.

the camelCased name can be used i.e. for the initialisation of react modules.

orioltf commented 7 years ago

Isn't it needed to patch *.data.js modules files too? I specially mean it for scaffold:rename, scaffold:copy and scaffold:delete to work. At least this was my experience when adapting scaffold for React.

tschuems commented 7 years ago

i don't think so it could be if you have some additional module types

backflip commented 7 years ago

We will need automated tests at one point (I have started with this already). But for now, I'd be fine with manual (i.e. "works on my machine") tests.

orioltf commented 7 years ago

gulp scaffold:delete> modules > all fails because of the use of _.pluck => We should merge #27 in here to solve this.
gulp scaffold:delete> modules > all fails. This may have to do with missing attributes in the *.data.js files.

lbsonley commented 7 years ago

This pull request looks good to me.

gulp scaffold:delete > Demo Pages > * (all) only fails when you run gulp scaffold:delete > Demo Modules > * (all) first

this is due to a requireNew in the pages data files pointing to one of the demo modules

So... when removing the demo modules and pages, make sure to remove the pages first and everything will work