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 docs domain name to Woo.com #2144

Merged
merged 10 commits into from
Nov 7, 2023
Merged

Update docs domain name to Woo.com #2144

merged 10 commits into from
Nov 7, 2023

Conversation

ianlin
Copy link
Member

@ianlin ianlin commented Nov 6, 2023

Changes proposed in this Pull Request:

Update the old WooCommerce.com → the new Woo.com in several places, especially for documentation links.

Project: peeuvX-13n-p2

Detailed test instructions:

  1. Confirm all new URLs actually go to a page.

Additional details:

  • Did not change the developer.woocommerce.com and api-vipgo.woocommerce.com URLs
  • Added a TODO (87e5872) for updating developer.woocommerce.com when the migration is done
  • Also changed the anchor #get-500-in-free-ad-credits to #google-performance-max-campaigns because there is no anchor for the former, and the latter one is the parent section of it.

Changelog entry

Doc - Use new Woo.com domain.

@ianlin ianlin requested a review from a team November 6, 2023 10:06
@ianlin ianlin self-assigned this Nov 6, 2023
@github-actions github-actions bot added the changelog: update Big changes to something that wasn't broken. label Nov 6, 2023
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #2144 (4467e80) into develop (4cb36d5) will not change coverage.
Report is 6 commits behind head on develop.
The diff coverage is 20.0%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             develop   #2144   +/-   ##
=========================================
  Coverage       58.3%   58.3%           
  Complexity      4119    4119           
=========================================
  Files            452     452           
  Lines          17678   17678           
=========================================
  Hits           10300   10300           
  Misses          7378    7378           
Flag Coverage Δ
php-unit-tests 58.3% <20.0%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
google-listings-and-ads.php 6.2% <ø> (ø)
src/Integration/WooCommercePreOrders.php 0.0% <ø> (ø)
src/Menu/MenuFixesTrait.php 0.0% <ø> (ø)
src/MultichannelMarketing/GLAChannel.php 98.5% <100.0%> (ø)
src/MerchantCenter/MerchantStatuses.php 60.5% <0.0%> (ø)
src/Notes/LeaveReviewActionTrait.php 0.0% <0.0%> (ø)
src/Notes/SetupCampaign.php 0.0% <0.0%> (ø)
src/PluginHelper.php 73.2% <0.0%> (ø)

Copy link
Contributor

@mikkamp mikkamp left a comment

Choose a reason for hiding this comment

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

Thanks for the changes, they look good, I confirmed the links are working and not redirecting to other locations. I don't find any other instances which should be changed.

@ianlin ianlin merged commit 60e1d4f into develop Nov 7, 2023
13 of 14 checks passed
@ianlin ianlin deleted the update/new-woo-domain branch November 7, 2023 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: update Big changes to something that wasn't broken.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants