Making WordPress.org

Opened 13 months ago

Closed 9 months ago

Last modified 4 months ago

#7405 closed enhancement (fixed)

Give plugin authors an easy way to run plugin-check on their pending submitted plugins

Reported by: tellyworth's profile tellyworth Owned by:
Milestone: Priority: normal
Component: Plugin Directory Keywords:
Cc:

Description

After submitting a plugin, we should make it easy for plugin authors to run plugin-check and view the results.

One way to do this would be to provide a link to Playground, and run PCP there for them.

Proposal: add a link or button on the plugin submission screen that does exactly that.

Attachments (3)

Screenshot 2024-01-12 at 5.33.09 pm.png (97.5 KB) - added by tellyworth 13 months ago.
Here's an example of what I mean. The "Run plugin check" link would open a new Playground window, with the user's plugin installed, and a copy of PCP installed and active.
7405.diff (2.5 KB) - added by dd32 12 months ago.
Screenshot 2024-01-16 at 3.39.46 pm.png (228.3 KB) - added by dd32 12 months ago.
Example of 7405.diff

Download all attachments as: .zip

Change History (16)

@tellyworth
13 months ago

Here's an example of what I mean. The "Run plugin check" link would open a new Playground window, with the user's plugin installed, and a copy of PCP installed and active.

#1 @tellyworth
13 months ago

The screen could include some text encouraging the author to review the plugin check results. Perhaps something like:

Please review the Plugin Check results for your plugin, and fix any significant problems. This will help streamline the preview process and reduce delays by ensuring your plugin already meets the required standards when the plugin review team examines it.

#2 follow-up: @foosantos
13 months ago

Thanks, @tellyworth! It is a great idea. We could add this for sure before the team works to have this check automatically as part of the review process.

For now, as a temporary solution, can we maybe do that before they apply and add a simple check box just saying something like, "I checked my plugin with the "Plugin Check."?

I believe it would be a similar complexity to implement it this way instead, right?

Last edited 13 months ago by foosantos (previous) (diff)

#3 in reply to: ↑ 2 @dd32
12 months ago

Replying to foosantos:

For now, as a temporary solution, can we maybe do that before they apply and add a simple check box just saying something like, "I checked my plugin with the "Plugin Check."?

See #7172 for that.

@dd32
12 months ago

@dd32
12 months ago

Example of 7405.diff

#4 @dufresnesteven
12 months ago

Can we do the following:

  • Communicate that the plugin is in the queue
  • Run the theme check
  • Alerts users of errors
  • Communicate that plugins with errors typically take X times longer to be approved
  • Communicate that they can fix the errors and not affect their place in the queue.

[Edit]
Ultimately, I think error-less plugins should make it to a priority queue but that probably falls out of the scope of this ticket.

Last edited 12 months ago by dufresnesteven (previous) (diff)

#5 @tellyworth
12 months ago

Those are excellent suggestions Steve. We can do the three "Communicate" ones easily in this patch just by adding some extra messaging. I'll tweak Dion's patch with some draft text.

Alerting the user to actual errors would require separate work to integrate PCP on the server side; that's in progress independently (see https://make.wordpress.org/pluginrepo/2024/01/08/plugin-check-next-steps/).

#6 @tellyworth
12 months ago

In 13150:

Plugin Directory: fix issue in moderator blueprints

Fixes a bug where the oldest helper plugin was used when plugin mods are reviewing a plugin, not the newest.

See #7405.

#7 @tellyworth
12 months ago

In 13179:

Plugin Directory: Add Plugin Check buttons on author pending page.

See #7405.

#8 @tellyworth
11 months ago

In 13211:

Plugin Directory: fewer Plugin Check buttons

On the plugin upload pending screen, only show the Plugin Check button for the most recent version of the plugin. This should eliminate confusion about which button to click.

See #7405.

#9 @dd32
10 months ago

In 13352:

Plugin Directory: Upload: Add a callout to the Plugin Check plugin, to encourage plugin authors to test with it prior to submission.

See #7405.

#10 follow-up: @dd32
10 months ago

Could [13352] be enhanced with a run interactively now link @tellyworth?

Perhaps it could just load up the requisite things and land on the wp-admin/plugin-upload.php screen? (I'm assuming it wouldn't be possible to inherit from the file upload pre-submission)

#11 in reply to: ↑ 10 @tellyworth
10 months ago

Replying to dd32:

(I'm assuming it wouldn't be possible to inherit from the file upload pre-submission)

Yeah I think it'd have to be a post-upload step.

I think it would be better to spend the effort on running plugin-check server-side during the upload process.

#12 @dd32
9 months ago

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

I think this is implemented.

I think it would be better to spend the effort on running plugin-check server-side during the upload process.

This is tracked in #7556

#13 @dd32
4 months ago

In 14078:

Plugin Directory: Plugin check on upload: When running plugin check against an uploaded ZIP file, attempt to force the plugin to be installed into the expected folder path.

This resolves issues of plugin check suggesting that the textdomain should be 2024_09_25-uploaded-file-SLUG rather than SLUG.

This is intended on being temporary until the upstream issue is resolved.

See #7405.
See https://github.com/WordPress/wordpress-playground/issues/1802.

Note: See TracTickets for help on using tickets.