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

Bring plugin back online #40

Closed
acelaya opened this issue Jan 16, 2024 · 8 comments
Closed

Bring plugin back online #40

acelaya opened this issue Jan 16, 2024 · 8 comments
Assignees

Comments

@acelaya
Copy link
Contributor

acelaya commented Jan 16, 2024

Relates to hypothesis/product-backlog#1527

This is the reason the plugin was closed, and the steps to follow to bring it back:

Reason

The email associated with the plugins owner's user account bounced. As such, all plugins associated with the account were closed, because we do require your email to be functional.

What to do next

  • Make sure the email on the user account is valid
  • If the email is a group mail or mailing list, make sure it can receive email from external domains or non-members (Google changed their defaults in 2019 due to GDPR laws)
  • If the email forwards, check all addresses to make sure they're valid and do not forward bounces
  • If the ownership of the plugin is in doubt, let us know what accounts are supposed to have access and be the official owners so we can transfer them appropriately
  • You must update the plugin readme to confirm it is compatible with the current release of WordPress. This is to ensure people can actually find your plugin.
  • Perform a full security and guideline check of your own work. Look for sanitization, remote loading of content, and any other minor bug.
  • Update all the code and upload it to SVN.
  • Reply to this email that you are ready for a review
@acelaya acelaya changed the title Bring plugin back to life Bring plugin back online Jan 16, 2024
@acelaya acelaya self-assigned this Jan 16, 2024
@acelaya
Copy link
Contributor Author

acelaya commented Jan 23, 2024

Steps to follow:

Nice to have:

@acelaya
Copy link
Contributor Author

acelaya commented Jan 25, 2024

I have already published a new version using the new automated GitHub workflow.

I requested a new review to the WordPress plugins team, and will follow-up with more details as soon as they come back to us.

@SteelWagstaff
Copy link

SteelWagstaff commented Jan 25, 2024

Thanks for your work in reviving this @acelaya. We require the hypothesis WordPress plugin in our bedrocks at Pressbooks. Because the plugin was deactivated at WordPress at org, the mirror at wpackagist has been closed and it's not picking up your latest releases: https://wpackagist.org/search?q=hypothesis&type=any&search=
Screenshot from 2024-01-25 10-21-42

I'm hopeful that reactivating the plugin on WordPress.org will be enough to get the mirror back up and working.

In the meantime, however, would you have any interest in registering the plugin directly with packagist.org? The process is pretty simple and the GitHub repo already has the required composer.json file. See "Publishing Packages" instructions here: https://packagist.org/. That would allow us to simply run composer require hypothesis/hypothesis to manage the plugin as a dependency, indepedently of whether or not the wordpress.org listing is active and the wpackagist service is mirroring all tagged releases.

I could do this myself as it is a public repo, but it would then list me in packagist as a maintainer of the project, which is a bit misleading.

@acelaya
Copy link
Contributor Author

acelaya commented Jan 26, 2024

In the meantime, however, would you have any interest in registering the plugin directly with packagist.org

This definitely makes sense. People managing their WordPress installations/deployments with composer would definitely benefit from it. I'll try to get it published there today.

Although, now that I look at the package name, it probably needs to be renamed to hypothesis/hypothesis-wp-plugin or something in those lines. The hypothesis/hypothesis name is too "top level", so to speak, and might be needed for other packages eventually.

@acelaya
Copy link
Contributor Author

acelaya commented Jan 26, 2024

@SteelWagstaff you can find the plugin in packagist now with the package name hypothesis/wp-hypothesis https://packagist.org/packages/hypothesis/wp-hypothesis

@SteelWagstaff
Copy link

Amazing -- thanks!

@acelaya
Copy link
Contributor Author

acelaya commented Feb 2, 2024

The plugin has been enabled again by the plugins team. I have also requested the previous owner to transfer ownership to us.

@acelaya acelaya closed this as completed Feb 2, 2024
@acelaya
Copy link
Contributor Author

acelaya commented Feb 5, 2024

Confirming wpackagist has started to pick up the new updates now that the plugin is listed again
image

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

No branches or pull requests

2 participants