Automattic / Automattic/jetpack

Redirect Everything project

Open
#15,403 1 comment 1 reaction 1 assignee Claimed by @leogermani View on GitHub
[Feature] Redirects Epic New Feature
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

This issue tracks what is still to be done in the Redirect Everything project after its [first PR](https://github.com/Automattic/jetpack/pull/15140) is merged.

More context on: 1151641796826042-as-1162696453656729

## Finish Linters

- [ ] [JS Linter](https://github.com/Automattic/jetpack/pull/15555)
- [ ] [PHP Linter](https://github.com/Automattic/jetpack/pull/15524)

## Finish redirect on js side

- [x] Finish and deploy the [npm package](https://github.com/Automattic/wp-calypso/pull/41497)
- [x] Make the function always send the current site url by default
- [x] Use npm package in blocks -> finish [this PR](https://github.com/Automattic/jetpack/pull/15350)
- [x] Make all JS links use the npm package and delete the helper function from client libs

These files need special attention:

- [ ] /_inc/client/plans/feture-item.jsx
- [ ] /_inc/client/traffic/related-posts.jsx
- [ ] /extensions/shared/components/upgrade-nudge/index.jsx
- [ ] /extensions/shared/plan-upgrade-notification.js
- [ ] /extensions/shared/test/get-validated-attributes.js
- [ ] /modules/plugin-search/plugin-search.js
- [ ] /modules/search/instant-search/components/jetpack-colophon.jsx
- [ ] /tools/jetpack-live-branches/jetpack-live-branches.user.js

## Handle edge cases

- [ ] handle localized URLs on the server side, by getting blog settings in the cache site
- [ ] do the same to handle currency

- [ ] Handle checkout URLs

- [ ] Remove URLs from i18n strings (remember to add `rel="noopener noreferrer"` in links inside i18n strings when needed)

**List of PHP files with links to either `https://jetpack.com` or `https://wordpress.com` that need to be redirected**:

- [ ] /_inc/class.jetpack-provision.php
- [ ] /_inc/lib/components.php
- [ ] /3rd-party/class.jetpack-amp-support.php
- [ ] /3rd-party/vaultpress.php
- [ ] /class.jetpack.php
- [ ] /functions.photon.php
- [ ] /jetpack.php
- [ ] /modules/calypsoify/class.jetpack-calypsoify.php
- [ ] /modules/carousel/jetpack-carousel.php
- [ ] /modules/minileven.php
- [ ] /modules/plugin-search.php
- [ ] /modules/protect.php
- [ ] /modules/sitemaps/sitemap-builder.php
- [ ] /modules/sitemaps/sitemap-stylist.php
- [ ] /modules/subscriptions/views.php
- [ ] /modules/theme-tools/site-breadcrumbs.php
- [ ] /modules/theme-tools/site-logo.php
- [ ] /modules/videopress/class.jetpack-videopress.php
- [ ] /modules/widgets/simple-payments/admin-warning.php
- [ ] /packages/backup/src/class-helper-script-manager.php
- [ ] /packages/jitm/src/class-jitm.php
- [ ] /tests/php/general/test_class.jetpack.php
- [ ] /tests/php/general/test-class.jetpack-gutenberg.php
- [ ] /tests/php/modules/shortcodes/test-class.gist.php

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.