Making WordPress.org

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#2386 closed enhancement (fixed)

Plugin Directory: Restrict Devs concurrent submissions

Reported by: ipstenu's profile Ipstenu Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Plugin Directory v3.0 Priority: normal
Component: Plugin Directory Keywords: needs-patch
Cc:

Description

The queue is rarely longer than 7 days. As such, the burden of reviews can get a little weird when people submit upwards of 10 plugins at once. Wish it was a joke. The problem is most of the time the reviews are all the same results, with the same errors repeating in each separate plugin.

It would be nice if we could restrict submissions as follows:

If a developer has NO active plugins, they can submit one plugin

Otherwise, if a developer has 3+ plugins in 'draft' or 'pending' (which means in review), they cannot submit any more. Give them a nice message of "You have too many plugins in the queue. Please wait for them to be approved before submitting any more."

This would mitigate repetition of reviews, which can be pretty onerous.

Change History (15)

#1 @SergeyBiryukov
8 years ago

  • Component changed from General to Plugin Directory

#2 @SergeyBiryukov
7 years ago

The patch on #2594 introduces an Upload::get_submitted_plugins() method that could be used here.

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


7 years ago

#4 @SergeyBiryukov
7 years ago

  • Keywords needs-patch added

#5 @SergeyBiryukov
7 years ago

  • Milestone changed from Plugin Directory v3 - Future to Plugin Directory v3.0

#6 @Ipstenu
7 years ago

  • Keywords needs-patch removed
  • Milestone changed from Plugin Directory v3.0 to Plugin Directory v3 - Future

Actually ... can we make it one in review at a time?

Make them decide what the heck to do before submitting more.

This would help keep the pending-reply queue lower. it's almost 700 from people who want multiple plugins and never finish.

#7 @Ipstenu
7 years ago

  • Keywords needs-patch added
  • Milestone changed from Plugin Directory v3 - Future to Plugin Directory v3.0

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


7 years ago

#9 @SergeyBiryukov
7 years ago

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

In 6023:

Plugin Directory: Limit plugin submissions to one at a time.

Remove the upload form if the developer already has a plugin in the review queue.

Fixes #2386.

#10 @SergeyBiryukov
7 years ago

In 6025:

Plugin Directory: Change "You have 1 plugin in the review queue" notice wording to "You already have a plugin in the review queue".

See #2386.

#11 @Ipstenu
7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

We missed an edge case.

If you upload a plugin, the submit box remains on the success page. :facepalm: We should hide it there, as it would permit users to submit multiple plugins in a row.

#12 @SergeyBiryukov
7 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 6045:

Plugin Directory: Remove the upload form after a successful upload.

Make Upload_Handler::process_upload() return WP_Error on failure.

Fixes #2386.

#13 @SergeyBiryukov
7 years ago

In 6046:

Plugin Directory: Adjust the condition for displaying the upload form for better readability.

See #2386.

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


7 years ago

#15 @SergeyBiryukov
7 years ago

In 6119:

Plugin Directory: Stop processing the upload if the developer already has a plugin in the review queue.

Fixes #3251. See #2386.

Note: See TracTickets for help on using tickets.