fix: added skydropx slug back to shipping partners list

[rjchow]

• fix: added skydropx slug back to shipping partners list

All Submissions:

• Have you followed the WooCommerce Contributing guideline?
• Does your code follow the WordPress' coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Previously the skydropx plugin was excluded from the shipping partners list as it was doing a redirect on activation, which then causes an error in the REST API call.

This has been remedied by them in the next version, which should be 1.0.7

Not too sure when they will release it but it will be out before 7.6 is released to public, so it should be safe to make this change before skydropx publishes it. In the unlikely event they do not release it, we will have to make a code freeze violation request to revert this commit.

How to test the changes in this Pull Request:

• Have you followed the Writing high-quality testing instructions guide?

1. Set up a new site, go through OBW without installing any plugins. Set your location to Mexico
2. If skydropx has published 1.0.7, ignore this step. Otherwise, only install this custom build of skydropx plugin without activating it. skydropx-cotizador-y-envios.zip
3. Go to shipping task on the home screen and set up skydropx via the shipping task
4. Should succeed without any errors.

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you created a changelog file for each project being changed, ie pnpm --filter=<project> changelog add?
• Have you included testing instructions?

FOR PR REVIEWER ONLY:

• I have reviewed that everything is sanitized/escaped appropriately for any SQL or XSS injection possibilities. I made sure Linting is not ignored or disabled.

[github-actions]

Test Results Summary

Commit SHA: ad292e5

Test 🧪	Passed ✅	Failed 🚨	Broken 🚧	Skipped ⏭️	Unknown ❔	Total 📊	Duration ⏱️
API Tests	259	0	0	2	0	261	0m 50s
E2E Tests	191	0	0	10	0	201	14m 24s

---

To view the full API test report, click here.
To view the full E2E test report, click here.
To view all test reports, visit the WooCommerce Test Reports Dashboard.

[codecov]

Codecov Report

Merging #37286 (ad292e5) into trunk (0cf5677) will decrease coverage by 0.0%.
The diff coverage is 19.6%.

Additional details and impacted files

@@            Coverage Diff             @@
##             trunk   #37286     +/-   ##
==========================================
- Coverage     46.7%    46.7%   -0.0%     
- Complexity   17191    17197      +6     
==========================================
  Files          429      429             
  Lines        64845    64883     +38     
==========================================
+ Hits         30275    30282      +7     
- Misses       34570    34601     +31     

Impacted Files	Coverage Δ
plugins/woocommerce/includes/class-wc-ajax.php	4.3% <0.0%> (-<0.1%)	⬇️
...ugins/woocommerce/includes/class-wc-post-types.php	2.7% <0.0%> (-0.1%)	⬇️
...ludes/tracks/events/class-wc-settings-tracking.php	0.0% <0.0%> (ø)
...ocommerce/includes/abstracts/abstract-wc-order.php	86.6% <50.0%> (-0.1%)	⬇️
...gins/woocommerce/includes/wc-product-functions.php	44.7% <100.0%> (+1.0%)	⬆️

... and 1 file with indirect coverage changes

[chihsuan]

Looks good and tested well! 👍

[ilyasfoo]

Tested well in a JN with this plugin instead:

skydropx-cotizador-y-envios.zip

Details: p1679290917992459/1679042446.165109-slack-C01SFMVEYAK

[ilyasfoo]

Merging this now to make into the code freeze