Skip to content

Fix ServerSideRender link point to 404 page#21409

Closed
widoz wants to merge 4 commits intoWordPress:wp/5.4from
widoz:patch-2
Closed

Fix ServerSideRender link point to 404 page#21409
widoz wants to merge 4 commits intoWordPress:wp/5.4from
widoz:patch-2

Conversation

@widoz
Copy link
Copy Markdown
Contributor

@widoz widoz commented Apr 5, 2020

The component page ServerSideRender seems no longer exists infact, looking at pages with ServerSideRender a possible user will get two results, the dynamic page which is the current one and the packages page which contains the documentation for the server side render package.

Therefore I suppose a papable page could be https://developer.wordpress.org/block-editor/packages/packages-server-side-render/

The component page `ServerSideRender` seems no longer exists infact, looking at pages with [ServerSideRender](https://developer.wordpress.org/block-editor/?s=ServerSideRender) a possible user will get two results, the dynamic page which is the current one and the packages page which contains the documentation for the server side render package.

Therefore I suppose a papable page could be https://developer.wordpress.org/block-editor/packages/packages-server-side-render/
@Soean Soean added the [Type] Developer Documentation Documentation for developers label Apr 5, 2020
Copy link
Copy Markdown
Member

@mkaz mkaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good, thanks for the fix

@mkaz
Copy link
Copy Markdown
Member

mkaz commented Apr 13, 2020

Thanks for this fix, it looks like it was opened against the WordPress 5.4 branch. You will need to create a new PR against master for this to be able to be merged.

@mkaz
Copy link
Copy Markdown
Member

mkaz commented Apr 15, 2020

@widoz It looks like you don't need to close and create a new one.

You should be able to edit this PR and switch the branch it is against to master - see this documentation to illustrate changing the base branch.

@gziolo gziolo changed the base branch from wp/5.4 to master April 30, 2020 04:05
@gziolo gziolo changed the base branch from master to wp/5.4 April 30, 2020 04:06
@mkaz mkaz changed the base branch from wp/5.4 to master June 5, 2020 20:02
@mkaz mkaz changed the base branch from master to wp/5.4 June 5, 2020 20:04
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Jun 5, 2020
@mkaz mkaz closed this in #22945 Jun 6, 2020
@widoz widoz deleted the patch-2 branch June 7, 2020 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Status] In Progress Tracking issues with work in progress [Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants