Directory

Update Welcome Guide article links to avoid redirect by SavPhill · Pull Request #48582 · WordPress/gutenberg · GitHub
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

Update Welcome Guide article links to avoid redirect #48582

Merged
merged 5 commits into from
Mar 3, 2023

Conversation

SavPhill
Copy link
Contributor

@SavPhill SavPhill commented Feb 28, 2023

What?

In the Welcome Guide, the final step titled 'Learn how to use the block editor' links to a support article under 'Here's a detailed guide' which opens a new tab for the url: https://wordpress.org/support/article/wordpress-editor/.

This url has since been redirected, and the final url the user will visit will be: https://wordpress.org/documentation/article/wordpress-block-editor/

Why?

There is no valid reason to point to an old url for this support article. By doing so, it slows the help experience down as the user is redirected but more importantly could lead to a broken page if the redirect is removed in error at a later date.

How?

My PR removes the need for the redirection, by pointing the user directly to the correct help article.

Testing Instructions

Visit the Welcome Guide modal
Follow the Welcome Guide steps until the 4th and final one.
Hover over the link titled Here's a detailed guide and view the url.
Click the link and view the final url in the address bar.

Fixes #48583.

Testing Instructions for Keyboard

Screenshots or screencast

redirection-welcome-guide

The link title Here's a detailed guide opens in a new tab to a help article which is a redirected link. We should point directly to the correct article. My PR addresses this small change and points to https://wordpress.org/documentation/article/wordpress-block-editor/.
@@ -119,7 +119,7 @@ export default function WelcomeGuideStyles() {
) }
<ExternalLink
href={ __(
'https://wordpress.org/support/article/styles-overview/'
'https://wordpress.org/documentation/article/wordpress-block-editor/'
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for working on this.

This doesn't seem to be the right link that has been updated. There are a few welcome guides, this one is for globals styles and has a different URL.

I see it still has a redirect though. The URL now resolves to https://wordpress.org/documentation/article/styles-overview/, so it might still be worth updating it, but also updating the other ones too.

There are a few that I found that have the link you mentioned:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your comment. Yes you are correct, I have edited the wrong link in this case. I will recheck and edit the PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for working on this.

This doesn't seem to be the right link that has been updated. There are a few welcome guides, this one is for globals styles and has a different URL.

I see it still has a redirect though. The URL now resolves to https://wordpress.org/documentation/article/styles-overview/, so it might still be worth updating it, but also updating the other ones too.

There are a few that I found that have the link you mentioned:

Thank you @talldan. I have adjusted the links for all suggested, as there is no reason to display the redirected url. This now fixes the problem I was originally addressing.

For the initial url which I edited incorrectly, I have reverted this change for now, and will address it in a different PR to avoid confusion.

Reverting this incorrect link edit. This does require changing, but will be addressed in a different PR
@SavPhill SavPhill requested review from talldan and removed request for draganescu, adamziel, noisysocks, kevin940726 and tellthemachines March 1, 2023 05:40
@talldan talldan added [Type] Task Issues or PRs that have been broken down into an individual action to take General Interface Parts of the UI which don't fall neatly under other labels. labels Mar 3, 2023
Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

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

Thank you for updating these 👍

@talldan talldan merged commit 6be1e86 into WordPress:trunk Mar 3, 2023
@talldan talldan changed the title Welcome Guide link points to a redirected help article Update Welcome Guide article links to avoid redirect Mar 3, 2023
@github-actions github-actions bot added this to the Gutenberg 15.4 milestone Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels. [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Welcome Guide support link points to a redirected url
2 participants