Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Menus should support params #1304

Closed
scfox opened this issue Apr 12, 2016 · 0 comments
Closed

Menus should support params #1304

scfox opened this issue Apr 12, 2016 · 0 comments
Assignees
@mleanos
Labels
enhance:feature
Milestone
0.5.0

Comments

@scfox
Copy link
Contributor

scfox commented Apr 12, 2016

Use case is to dynamically generate sub-menu items where all would go to same state but have a parameter to identify target object, e.g. optionId. Right now the only option I see would be to create a dynamic state name that would embed the params, but that doesn't seem like a good option.
scfox added a commit to scfox/mean that referenced this issue Apr 12, 2016
@scfox
feat(core): Add support for params in menu items
2f7ed3e 
Proposed by @scfox

Fixes meanjs#1304
@scfox scfox mentioned this issue Apr 12, 2016
Merged
@lirantal lirantal added this to the 0.5.0 milestone Jul 7, 2016
mleanos pushed a commit that referenced this issue Jul 25, 2016
@scfox @mleanos
Add support for params in menu items
 (#1305)
5817a48 
* Support params to state for menu items.

* feat(core): Add support for params in menu items

Proposed by @scfox

Fixes #1304

* Removed trailing space on test.

* Changed default params to be empty object instead of empty string.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mleanos mleanos

Labels
enhance:feature
Projects
None yet
Milestone
0.5.0
Development

No branches or pull requests
3 participants
@lirantal @mleanos @scfox