search

meanjs / generator-meanjs

MEAN.JS Official Yeoman Generator
http://meanjs.org/
473 stars 177 forks source link

CRUD generator inconsistencies with app generated from meanjs master version #228

Closed demus closed 8 years ago

demus commented 8 years ago

Command running: yo meanjs:crud-module thing

When choosing the option to "add the CRUD module links to a menu" the wrong service names are used once the CRUD module is generated.

Generated client/config/things.client.config.js result: 

menuConfig.$inject = ['Menus'];
  function menuConfig(Menus) {
    Menus.addMenuItem('topbar', {
      title: 'Things',
      state: 'things',
      type: 'dropdown',
      roles: ['*']
    });

    // list item
    // create item
  }

'Menus' should actually be 'menuService'. Should we not use the master branch to generate the boilerplate?

demus commented 8 years ago

closing since I just saw this was fixed by pull request #215