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

Angular:crud #99

Merged
merged 1 commit into from
Nov 21, 2012
Merged

Angular:crud #99

merged 1 commit into from
Nov 21, 2012

Conversation

jacobmumm
Copy link

Description:
Creates new AngularJS crud routes

Example:
yeoman init angular:crud thing [--coffee]

This will create:
    app/scripts/controllers/thing/thingIndex.js
    app/scripts/controllers/thing/thingCreate.js
    app/scripts/controllers/thing/thingUpdate.js
    app/scripts/controllers/thing/thingView.js
    app/views/thing/thingIndex.html
    app/views/thing/thingCreate.html
    app/views/thing/thingUpdate.html
    app/views/thing/thingView.html
And add routing to:
    app.js
    /thing/index
    /thing/create
    /thing/update/:id
    /thing/view/:id

@btford Here we have the ability to generate all crud routes in one command. Further, they will be organized into folders and the routes will look better.

@btford
Copy link
Contributor

btford commented Oct 17, 2012

I'll have to take a closer look, but at first glance, I like what I see. Nice work!

@addyosmani
Copy link
Member

@btford could you have another look at this when you have time?

@btford
Copy link
Contributor

btford commented Nov 7, 2012

I like it! I'm going to run this by @IgorMinar, but tentatively I think it's worth merging. Thanks, @jacobmumm :)

@sindresorhus
Copy link
Member

ping

@addyosmani
Copy link
Member

Nice work! Just pinging again as we'd love to be able to get this added if @IgorMinar and c/o feel its worthwhile.

@sindresorhus sindresorhus merged this pull request into yeoman:master Nov 21, 2012
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

Successfully merging this pull request may close these issues.

4 participants