[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