[wp-trac] [WordPress Trac] #42791: wp_add_dashboard_widget() should accept location and priority parameters
WordPress Trac
noreply at wordpress.org
Thu Feb 8 15:47:01 UTC 2018
#42791: wp_add_dashboard_widget() should accept location and priority parameters
-------------------------------------------------+-------------------------
Reporter: johnbillion | Owner:
Type: enhancement | davidjlaietta
Priority: normal | Status: assigned
Component: Administration | Milestone: Awaiting
Severity: normal | Review
Keywords: good-first-bug has-patch needs- | Version:
unit-tests | Resolution:
| Focuses:
-------------------------------------------------+-------------------------
Changes (by DrewAPicture):
* keywords: good-first-bug has-patch => good-first-bug has-patch needs-
unit-tests
* owner: => davidjlaietta
* status: new => assigned
Comment:
Hi @davidjlaietta, thanks for the patch! I've assigned the ticket to you
to mark this `good-first-bug` as "claimed". Sorry for the delay in
response.
I did a first pass on the patch, and the big standouts that are still
missing are changelog entries for the new parameters in the DocBlock, and
we could probably also benefit from unit tests here as well.
For info on changelog entries, check out
[https://make.wordpress.org/core/handbook/best-practices/inline-
documentation-standards/php/#since-section-changelogs this article] in the
core handbook.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/42791#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list