Making WordPress.org

Opened 3 years ago

Closed 4 months ago

#5504 closed defect (bug) (fixed)

Allow uploading block-based themes

Reported by: aristath's profile aristath Owned by:
Milestone: Priority: normal
Component: Theme Directory Keywords:
Cc:

Description

Right now uploading a block-based theme in the .org repository is blocked by the theme-check that runs on upload. Authors are forced to add hacks and useless code in their themes to bypass some of these checks. However, themes are not allowed to contain unused code so the process currently followed is this:

  • Author uploads a theme that passes theme-check
  • Author contacts the themes-team reps.
  • The team reps do a preliminary review of the theme, and set the theme to be a special-case theme. This allows theme updates to bypass theme-check when uploaded
  • Author uploads their "real" theme.

The above process is unsustainable and if someone tries to upload a valid FSE theme the upload will fail. It's OK right now since block-based themes are still experimental, but we have started getting requests and questions about how theme-authors can upload their themes, so we're expecting more submissions to start coming in soon.

Suggestions: Either theme-check should be disabled for block-based themes, OR some checks should be skipped - see https://github.com/WordPress/theme-check/pull/280

Change History (6)

#1 @williampatton
3 years ago

For this ticket we need to update the Theme Check plugin. First we have to define a definite way to identify FSE themes and use that to determine which checks to perform in the plugin.

#2 @dd32
3 years ago

Just noting this Theme Check PR to adapt it to allow FSE themes to pass https://github.com/WordPress/theme-check/pull/280

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


3 years ago

#4 @poena
3 years ago

The Theme Check updates are ready for testing.

Summary of the checks that are skipped
for block themes and why:
https://github.com/WordPress/theme-check/issues/279#issuecomment-851774058

#5 @dd32
3 years ago

As of today & [11035], the new Theme Check is deployed on WordPress.org.

I think that should fix this ticket?

#6 @dd32
4 months ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.