Deprecate the old request manager

[lucanovera]

Ticket ENG-2133

Description Of Changes

Delete old request manager page. Replace with new list based request manager.

Code Changes

• Replaced old request manager page with new one
• Deleted components only referenced by the old request manager
• Removed outdated cypress tests (new ones were added in a different file)
• Removed the feature flag for the new request manager

Steps to Confirm

1. Open preview link
2. Check that Privacy requests > Request manager now shows the new request manager
3. Try navigating to a detail page and back to the list page

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
  • Updates unreleased work already in Changelog, no new entry necessary
• UX feedback:
  • All UX related changes have been reviewed by a designer
  • No UX review needed
• Followup issues:
  • Followup issues created
  • No followup issues
• Database migrations:
  • Ensure that your downrev is up to date with the latest revision on main
  • Ensure that your downgrade() migration is correct and works
    • If a downgrade migration is not possible for this change, please call this out in the PR description!
  • No migrations
• Documentation:
  • Documentation complete, PR opened in fidesdocs
  • Documentation issue created in fidesdocs
  • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
  • No documentation updates required

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
fides-plus-nightly	Ready	Preview, Comment	Dec 19, 2025 8:53pm

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
fides-privacy-center	Ignored		Dec 19, 2025 8:53pm

[greptile-apps]

Greptile Summary

This PR successfully deprecates the old table-based request manager and replaces it with the new list-based implementation. The change removes ~900 lines of code by deleting obsolete components, tests, and the privacyRequestV2 feature flag.

Key Changes:

• Moved new request manager from /new-privacy-requests to /privacy-requests route
• Removed feature flag gating - new implementation is now the default
• Deleted old components: PrivacyRequestsContainer, RequestTable, RequestTableColumns, RequestTableFilterModal, and related hooks
• Removed outdated Cypress tests for table-based UI (detail page tests remain)
• Updated navigation config to remove duplicate nav item

Minor Issue:

• The PRIVACY_REQUESTS_ROUTE_NEW constant in routes.ts should be removed as it's no longer used

Confidence Score: 4/5

• This PR is safe to merge with minimal risk - it's primarily a code deletion PR that removes deprecated code
• Score reflects thorough cleanup of old code and proper feature flag removal. One unused route constant remains but doesn't affect functionality. The new implementation was already tested in previous PRs.
• Pay attention to routes.ts for the unused constant cleanup

Important Files Changed

Filename	Overview
clients/admin-ui/src/pages/privacy-requests/index.tsx	Replaced old implementation with new list-based dashboard, imports moved from old container to new components
clients/admin-ui/src/flags.json	Removed privacyRequestV2 feature flag as new implementation is now default
clients/admin-ui/src/features/common/nav/nav-config.tsx	Removed "Request manager (new)" nav item that used feature flag and separate route
clients/admin-ui/cypress/e2e/privacy-requests.cy.ts	Removed old table-based tests (approve, deny, delete from list); detail page tests remain intact

[greptile-apps]

Additional Comments (1)

1. clients/admin-ui/src/features/common/nav/routes.ts, line 45 (link)

   style: Unused route constant - should be deleted since /new-privacy-requests page no longer exists

_{13 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[speaker-ender]

Looks good!
Always nice to see a bunch of old code removal