Automattic / Automattic/jetpack
Redirect Everything project
- 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
Assessment
This issue has not been assessed yet.