ENG-1992: Fix pre-approval webhook documentation

[tvandort]

Ticket ENG-1992

Description Of Changes

Open api docs say pre-execution when they mean pre-approval.

Code Changes

• Rename some functions.

Steps to Confirm

1. Check that pre-approval web hooks say pre-approval and not pre-execution on swagger docs.

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.

2 Skipped Deployments

Project	Deployment	Preview	Comments	Updated (UTC)
fides-plus-nightly	Ignored	Preview		Nov 17, 2025 7:55pm
fides-privacy-center	Ignored			Nov 17, 2025 7:55pm

[greptile-apps]

Greptile Summary

• Renamed three endpoint functions from *_pre_execution_* to *_pre_approval_* for consistent terminology
• Updated log messages and inline comments to use "Pre-Approval" instead of "Pre-Execution"

Confidence Score: 5/5

• This PR is safe to merge with no risk - it only contains documentation and naming fixes.
• The changes are purely cosmetic, renaming functions and updating documentation strings for consistency. No logic changes, no behavioral changes, and no dependencies affected. All references have been consistently updated.
• No files require special attention

Important Files Changed

Filename	Overview
src/fides/api/api/v1/endpoints/pre_approval_webhook_endpoints.py	Renamed function names and updated documentation strings from "pre-execution" to "pre-approval" for terminology consistency.

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format}

[erosselli]

that was not at all confusing.. 😆

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.33%. Comparing base (cb7c97d) to head (b29c022).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6985   +/-   ##
=======================================
  Coverage   87.33%   87.33%           
=======================================
  Files         525      525           
  Lines       34458    34458           
  Branches     3967     3967           
=======================================
  Hits        30093    30093           
  Misses       3501     3501           
  Partials      864      864           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.