Making WordPress.org

Opened 8 years ago

Closed 8 years ago

#2713 closed defect (bug) (fixed)

Plugin Directory Admin: Show email addresses of committers

Reported by: ipstenu's profile Ipstenu Owned by: coffee2code's profile coffee2code
Milestone: Plugin Directory v3.0 Priority: normal
Component: Plugin Directory Keywords: has-patch has-screenshots
Cc:

Description

When we send out emails about security or guideline issues, we send them to everyone with commit access to a plugin. Not having this on the page for the plugin admin makes it a little harder to get the information.

Attached patch displays the email address.

Needs someone with a better DB of the directory to double check. I manually manipulated the DB I have to include committers.

Attachments (9)

2713.diff (1.0 KB) - added by Ipstenu 8 years ago.
2713.PNG (13.6 KB) - added by SergeyBiryukov 8 years ago.
2713.2.diff (1.1 KB) - added by SergeyBiryukov 8 years ago.
2713.2.PNG (10.4 KB) - added by SergeyBiryukov 8 years ago.
2713.3.diff (1.3 KB) - added by SergeyBiryukov 8 years ago.
2713.3.PNG (10.4 KB) - added by SergeyBiryukov 8 years ago.
2713.4.diff (682 bytes) - added by Ipstenu 8 years ago.
2713.5.diff (837 bytes) - added by SergeyBiryukov 8 years ago.
2713.5.PNG (8.4 KB) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (16)

@Ipstenu
8 years ago

@SergeyBiryukov
8 years ago

@SergeyBiryukov
8 years ago

#1 @SergeyBiryukov
8 years ago

Works as expected on my install, although emails don't quite fit in the narrow column: 2713.PNG.

Maybe a <a href="mailto:...">Email</a> link would be better (2713.2.diff, 2713.2.PNG)? Browsers would also show a "Copy email address" option in context menu in that case.

#2 @SergeyBiryukov
8 years ago

I'd also expect for the username to link to WP.org profile, as in the author card. 2713.3.diff addresses that.

@SergeyBiryukov
8 years ago

#3 @Ipstenu
8 years ago

Usability wise, we don't need to to be a mail-to, since right now we have to manually copy the emails into another form... Having it hidden behind link makes it a worse experience for how it's used.

We copy the email addresses to paste into a SupportPress email form.

What about putting the emails on a line below the user name, and moving 'remove' to be an X on the right most column?

@Ipstenu
8 years ago

#4 @Ipstenu
8 years ago

Attached a patch to show what I mean. We don't need a link to email anyone, we literally need to be able to copy the email because SupportPress is silly :)

This ticket was mentioned in Slack in #meta by ipstenu. View the logs.


8 years ago

@SergeyBiryukov
8 years ago

#6 @SergeyBiryukov
8 years ago

  • Keywords has-screenshots added
  • Milestone set to Plugin Directory v3.0

2713.5.diff is the same as 2713.4.diff, but with the link to WP.org profile, for consistency with the author card.

Screenshot: 2713.5.PNG

Last edited 8 years ago by SergeyBiryukov (previous) (diff)

#7 @coffee2code
8 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 5554:

Plugin Directory, Committers: Show email address of committers in "Plugin Committers" metabox.

Props SergeyBiryukov, Ipstenu.
Fixes #2713.

Note: See TracTickets for help on using tickets.