wordpress-seo icon indicating copy to clipboard operation
wordpress-seo copied to clipboard

Feature/get redirects

Open kyrylo-polozenko-newfold opened this issue 6 months ago • 1 comments

Context

This PR introduces full CRUD (Create, Read, Update, Delete) support for redirects, along with UI improvements and internal refactoring for better maintainability and performance.

Summary

changelog: enhancement This PR can be summarized in the following changelog entry:

Adds functionality for creating, retrieving, updating, and deleting redirects.

Includes design and UI improvements to the redirects interface.

Relevant technical choices:

Implemented useRedirectFilters and useGetRedirects hooks to manage state and data fetching.

Refactored logic for sorting, filtering, and bulk actions (e.g., bulk delete).

Added loading indicators and success/error notifications.

Improved table and form UI for a better user experience.

UI changes

  • [ ] This PR changes the UI in the plugin. I have added the 'UI change' label to this PR.

Documentation

  • [ ] I have written documentation for this change.

Pull Request Test Coverage Report for Build 30c6b14cf72bf2df4c2ac8a578b09d5053a0b516

Details

  • 1 of 23 (4.35%) changed or added relevant lines in 9 files are covered.
  • 105 unchanged lines in 8 files lost coverage.
  • Overall coverage increased (+0.1%) to 53.548%

Changes Missing Coverage Covered Lines Changed/Added Lines %
admin/class-config.php 0 1 0.0%
admin/views/redirects.php 0 1 0.0%
packages/js/src/redirects/components/route-error-fallback.js 0 1 0.0%
packages/js/src/redirects/components/route-layout.js 0 1 0.0%
packages/js/src/redirects/routes/redirects.js 0 1 0.0%
packages/js/src/redirects/initialize.js 0 2 0.0%
packages/js/src/redirects/appProvider.js 0 5 0.0%
packages/js/src/redirects/store/preferences.js 0 10 0.0%
<!-- Total: 1 23
Files with Coverage Reduction New Missed Lines %
packages/js/src/shared-admin/hooks/use-formik-error.js 1 0.0%
packages/js/src/shared-admin/hooks/use-parsed-user-agent.js 1 0.0%
packages/js/src/redirects/constants/index.js 1 0.0%
packages/js/src/redirects/components/route-layout.js 1 0.0%
src/integrations/admin/redirects-page-integration.php 2 89.47%
packages/js/src/redirects/routes/redirects.js 2 0.0%
admin/class-config.php 23 0.0%
admin/views/redirects.php 74 0.0%
<!-- Total: 105
Totals Coverage Status
Change from base Build 9eca112a43d47f6670b05e1adc82fe6694c5958e: 0.1%
Covered Lines: 30345
Relevant Lines: 57698

💛 - Coveralls

coveralls avatar Jun 16 '25 11:06 coveralls

Since we are moving some of the functionality to Premium, I moved this PR to needs action

vraja-pro avatar Jun 25 '25 12:06 vraja-pro