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

removeMenu() not implemented #5210

Closed
lkcampbell opened this issue Sep 13, 2013 · 2 comments
Closed

removeMenu() not implemented #5210

lkcampbell opened this issue Sep 13, 2013 · 2 comments
Assignees
@lkcampbell
Milestone
Sprint 31

Comments

@lkcampbell
Copy link
Contributor

Per src/command/Menus.js:

    /** NOT IMPLEMENTED
     * Removes Menu
     */
    // function removeMenu(id) {
    //     NOT IMPLEMENTED
    // }

We will need this for issue #5078 or any situation where we need to clean up non-default menus on the fly.
@lkcampbell lkcampbell mentioned this issue Sep 13, 2013
Closed
@lkcampbell
Copy link
Contributor Author

Assigning to myself since I am working on a fix.

@ghost ghost assigned lkcampbell Sep 14, 2013
@lkcampbell lkcampbell mentioned this issue Sep 14, 2013
Merged
JeffryBooher added a commit that referenced this issue Sep 17, 2013
@JeffryBooher
Merge pull request #5217 from lkcampbell/fix-issue-5210
78814b4 
Fix issue #5210: Adding Menus.removeMenu() and Menus.getAllMenus()
@lkcampbell
Copy link
Contributor Author

Wasn't implemented...now it is :).

Closing as fixed.

@core-ai-bot core-ai-bot mentioned this issue Aug 29, 2021
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lkcampbell lkcampbell

Labels
None yet
Projects
None yet
Milestone
Sprint 31
Development

No branches or pull requests
1 participant
@lkcampbell