[wp-meta] [Making WordPress.org] #1109: Site Cloner: Add support for importing menus
Making WordPress.org
noreply at wordpress.org
Wed Dec 28 20:12:52 UTC 2016
#1109: Site Cloner: Add support for importing menus
--------------------------------------+-----------------------
Reporter: iandunn | Owner: iandunn
Type: enhancement | Status: accepted
Priority: normal | Milestone:
Component: WordCamp Site & Plugins | Resolution:
Keywords: good-first-bug has-patch |
--------------------------------------+-----------------------
Changes (by iandunn):
* keywords: needs-patch good-first-bug => good-first-bug has-patch
Comment:
This looks pretty good to me at first glance, thanks!
I think the only thing I'd suggest improving would be to modularize
`import_menus()`, since right now it's doing 2-3 different things. At the
very least, there should be a separate function for creating the new menus
There's a backlog of contributions right now, so it might be awhile before
I have time to do a full review, but I've added this to my list.
--
Ticket URL: <https://meta.trac.wordpress.org/ticket/1109#comment:6>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org
More information about the wp-meta
mailing list