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

Add a release page #407

Open
wants to merge 89 commits into
base: trunk
Choose a base branch
from

Conversation

StevenDufresne
Copy link
Contributor

Related https://meta.trac.wordpress.org/ticket/7783.

This PR adds the "Releases" tab behind a feature flag show_release_beta so we can work together on the site.

@tellyworth
Copy link
Contributor

@StevenDufresne I think the CPT code is kinda-sorta functional, thought not really hooked up to anything yet.

I've added a bin script you can use to (re)generate CPT entries for a specific plugin for testing purposes:

php bin/update-release-cpt.php --plugin=debug-bar

Next step is to properly hook this up to the main Plugin Directory class, and add some internal and REST APIs.

StevenDufresne and others added 12 commits November 25, 2024 05:12
Not sure this logic is quite right, since I'm only creating the draft when the version number doesn't already exist as a release.

I think this probably needs to be a revision # check, but the revision isn't yet stored for all releases; I'll look at that tomorrow.
… release

Needs more testing. Not sure if having the 'date' part is a good idea.
The tag and version values are both stored in postmeta if needed
…ges.

It might be ok to run it synchronously like this; otherwise I'll move it to an async job.

The Plugin_Check class isn't quite complete yet. It's copied from the code in the upload handler shortcode; we should replace that with a call to the class when it's complete.
bazza pushed a commit that referenced this pull request Dec 18, 2024
bazza pushed a commit that referenced this pull request Dec 18, 2024
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

Successfully merging this pull request may close these issues.

2 participants