WordPress.org

Making WordPress.org

Opened 7 months ago

Closed 5 months ago

Last modified 5 months ago

#4244 closed defect (fixed)

Change the base branch used to synchronize the Gutenberg handbook

Reported by: youknowriad Owned by: coffee2code
Milestone: Priority: normal
Component: Handbooks Keywords:
Cc:

Description

The idea is that the handbook should follow the stable version of WordPress. Ideally, we'd have at least two versions of the handbook, one for the stable version and another one for the Gutenberg plugin. We could also imagine a version selector but for now, let's fix the most urgent.

I'm new to the meta repo and trac, I'm not sure how to test this patch. Anyway, let me know if I did anything wrong.

Attachments (1)

change-gutenberg-handbook-branch.diff (871 bytes) - added by youknowriad 7 months ago.

Download all attachments as: .zip

Change History (8)

#1 @dd32
7 months ago

change-gutenberg-handbook-branch.diff looks like the first steps, however is going to get old having to update that branch reference..

Additionally, the manifest itself links to master still, so updating this won't actually change anything AFAIK.

#2 follow-up: @youknowriad
7 months ago

Mmm right, I guess the manifest should be updated to use relative links instead. I created an issue about it https://github.com/WordPress/gutenberg/issues/14303. I'll circle back here once it's done.

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

Replying to youknowriad:

Mmm right, I guess the manifest should be updated to use relative links instead. I created an issue about it https://github.com/WordPress/gutenberg/issues/14303. I'll circle back here once it's done.

Sure, that'll work, make sure you circle back and get support for that added to the parser before it's in master though :)
Bit hard to test until there's at least a PR to test against.

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


5 months ago

#5 @tellyworth
5 months ago

  • Owner set to coffee2code
  • Status changed from new to assigned

As per Slack, it seems like @coffee2code might handle this as part of migrating the Blocks handbook to developer.w.org.

#6 @coffee2code
5 months ago

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

In 8776:

Markdown Importer: Support relative paths for per-page markdown source.

The markdown_source values defined in manifests can continue to be fully qualified URLs, but now they can alternatively also be paths relative to the URL path for the manifest file. Parent path notation can be used (e.g. "../file.md"), but attempts to refer to higher level paths (e.g. "../../file.md") will not be honored beyond the parent.

Fixes #4244.
See #4388.
See https://github.com/WordPress/gutenberg/pull/14739.
See https://github.com/WordPress/gutenberg/pull/15254.

#7 @coffee2code
5 months ago

  • Component changed from General to Handbooks

As per #4388 and https://github.com/WordPress/gutenberg/pull/15254 (and some recent discussions in #meta-devhub on Slack), the Gutenberg handbook is being migrated to DevHub (it's just being finalized now).

The handbook importer as used on DevHub now supports relative paths, and the manifest used for the Block Editor handbook there makes use of relative paths.

Since the legacy handbook at https://wordpress.org/gutenberg/handbook/ is being deprecated and eventually redirected, the importer for the site (which is a customized implementation) was not updated to support relative paths (and, in fact, has been disabled in [8775]).

See https://github.com/WordPress/gutenberg/pull/15254#issuecomment-491340461.

Note: See TracTickets for help on using tickets.