Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Marketplace: Global plugins install is slow and error state UX is incorrect for erroring plugins #92251

Open
ebinnion opened this issue Jul 1, 2024 · 1 comment
Assignees

Comments

@ebinnion
Copy link
Contributor

ebinnion commented Jul 1, 2024

Quick summary

When attempting to install widont-part-deux on a site from https://wordpress.com/plugins/widont-part-deux, the installation takes quite a long time then errors out with a notice of:

An error occurred while installing Widon’t Part Deux on ebinnion69.wpcomstaging.com.

CleanShot 2024-07-01 at 12 41 11@2x

This notice is incorrect because the plugin was installed, but it couldn't be activated. This is because the plugin fatals on activation which we see when we activate the plugin from wp-admin.

CleanShot 2024-07-01 at 12 42 14@2x

Some bits of feedback to share here are:

  • The plugin installation should fail much quicker. While I didn't time it, it probably took upwards of a minute.
  • The plugin installation error message is incorrect in this case since the plugin did install but couldn't activate. Ideally, we'd be able to tell the user that the plugin errored on activation so we disabled it as a safety mechanism.
  • Perhaps more ideally, we would just delist the widont-part-deux and other similar plugins if they're guaranteed to fail on sites.

Steps to reproduce

  1. Go to global plugins view at https://wordpress.com/plugins/widont-part-deux
  2. Install the plugin on a WoA site
  3. Observe the slow install and eventual error message
  4. Go to wp-admin and verify that the plugin did install but did not activate.

What you expected to happen

A quicker process with more correct error states.

What actually happened

As described above.

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@ebinnion ebinnion added [Type] Bug Needs triage Ticket needs to be triaged labels Jul 1, 2024
@eoigal eoigal self-assigned this Jul 2, 2024
@eoigal eoigal removed the Needs triage Ticket needs to be triaged label Jul 2, 2024
@eoigal
Copy link
Contributor

eoigal commented Jul 4, 2024

I've created #92284 to update the error message to be more clear on what has happened and also update the modal state so that you can see that the plugin is installed (but not activated).

The slow response appears to be due to how the manage sites on plugins works, where it queues up auto updates each time the plugins endpoint is queried. This coupled by an issue ( p1720076625916529/1719866457.103339-slack-CDLH4C1UZ ) where the plugins that we are trying to update are on Github (not the WP.ORG repo) which means they constantly fail to update as the endpoint doesn't yet support that.

@lupus2k is working on a fix, or at least a bypass of Github plugins, to help avoid the noise from this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants