[wp-trac] [WordPress Trac] #40702: Add WordCamps and meetup events to the News Dashboard widget
WordPress Trac
noreply at wordpress.org
Thu May 18 15:56:30 UTC 2017
#40702: Add WordCamps and meetup events to the News Dashboard widget
----------------------------+-----------------------------------
Reporter: iandunn | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 4.8
Component: Administration | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch | Focuses: javascript, rest-api
----------------------------+-----------------------------------
Comment (by iandunn):
Replying to [comment:60 obenland]:
> @iandunn Can `$this->maybe_anonymize_ip_address(
$this->get_unsafe_client_ip() )` be merged into one function and called
`get_client_ip()`? They're always used in conjunction and feels overly
verbose
I don't see any problems with that. Do you have time to do that?
> Instead of using templates for translation strings, can we use the
dashboard l10n object?
Doh, good point. I'll make that change in the next iteration of the patch.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/40702#comment:61>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list