linnovate / meanio

meanio core module
MIT License
54 stars 73 forks source link

fixes `menu` type, add icon support #27

Closed AlmogBaku closed 9 years ago

AlmogBaku commented 9 years ago

menu type support

As described by the documentation, and as generated by the scaffolding- the name of the property should be menu instead of path. This PR is allow you to set menu or path.

  Investor.menus.add({
    title: 'investors manage',
    link: 'investors.admin',
    icon: 'mean-admin/assets/img/icons/settings.png',
    roles: ['admin'],
    menu: 'admin'
  });

icon support

Also, this PR enable using the icon property(as used in the mean-admin)

AlmogBaku commented 9 years ago

also, this PR is matching the api that described here: http://learn.mean.io/#mean-packages-menu-system

AlmogBaku commented 9 years ago

ping: @liorkesos

liorkesos commented 9 years ago

Almog - YAGC (Yet Another Great Contribution)

AlmogBaku commented 9 years ago

thanks :)

On Wed, Dec 10, 2014 at 9:45 PM, Lior Kesos notifications@github.com wrote:

Almog - YAGC (Yet Another Great Contribution)

— Reply to this email directly or view it on GitHub https://github.com/linnovate/meanio/pull/27#issuecomment-66511182.