Skip to content
This repository has been archived by the owner on Jun 7, 2022. It is now read-only.

Replace all docs.woocommerce.com links with woocommerce.com's #8105

Merged
merged 3 commits into from Jan 4, 2022

Conversation

chihsuan
Copy link
Member

@chihsuan chihsuan commented Jan 3, 2022

Fixes #8096

This PR fixes the broken "WooCommerce Docs" link and replaces all old docs.woocommerce.com links with woocommerce.com's.

Detailed test instructions:

  1. Go to /wp-admin/admin.php
  2. Click "Help menu" on the top-right corner and make sure all links work
  3. Confirm the other links still work (You could use an editor like vscode to help you open the links without copy-and-paste)

…tion

Update links

Fix doc links

Fix doc links

Fix doc links
Update changelogs
@@ -275,7 +275,7 @@ protected static function get_amazon_ebay_extension_data() {
}

$data['settingsUrl'] = admin_url( 'admin.php?page=codisto-settings' );
$data['docsUrl'] = 'https://docs.woocommerce.com/document/getting-started-with-woocommerce-amazon-ebay-integration/?utm_medium=product';
$data['docsUrl'] = 'https://woocommerce.com/documentation/getting-started-with-woocommerce-amazon-ebay-integration/?utm_medium=product';
Copy link
Member Author

Choose a reason for hiding this comment

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

Though it's not in the allowed plugin list, this plugin seems no longer available and the document link is 404.

I think this is the new plugin by Codisto.
https://woocommerce.com/products/amazon-ebay-integration/

Copy link
Member Author

@chihsuan chihsuan Jan 3, 2022

Choose a reason for hiding this comment

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

If we're no longer need this. I think we can just remove it. Please help confirm this, thanks.

Copy link
Contributor

Choose a reason for hiding this comment

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

@chihsuan I think you're correct on it. I installed https://woocommerce.com/products/amazon-ebay-integration/ and confirm the plugin has the same setting page path (page=codisto-settings).

Copy link
Contributor

Choose a reason for hiding this comment

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

@chihsuan Actually I've found the new documentation page in https://woocommerce.com/document/multichannel-for-woocommerce-google-amazon-ebay-walmart-integration/ from the product page. I think we can replace it with this instead

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated. Thanks for confirming! 😄

@chihsuan chihsuan self-assigned this Jan 3, 2022
@chihsuan chihsuan requested a review from a team January 3, 2022 08:41
Copy link
Contributor

@moon0326 moon0326 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 working on this so quickly @chihsuan

It LGTM and tested well 🚢 🚀

@chihsuan chihsuan merged commit 37312b1 into main Jan 4, 2022
@chihsuan chihsuan deleted the fix/8096-doc-links branch January 4, 2022 09:04
@ilyasfoo ilyasfoo added this to the 3.1.0 milestone Jan 6, 2022
ilyasfoo pushed a commit that referenced this pull request Jan 6, 2022
* Replace all docs.woocommerce.com links with woocommerce.com/documentation
* Add changelogs

Update changelogs

* Update woocommerce-amazon-ebay-integration doc url
ObliviousHarmony pushed a commit to woocommerce/woocommerce that referenced this pull request Mar 18, 2022
…merce/woocommerce-admin#8105)

* Replace all docs.woocommerce.com links with woocommerce.com/documentation
* Add changelogs

Update changelogs

* Update woocommerce-amazon-ebay-integration doc url
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WooCommerce Docs link on the help menu does not work
3 participants