Skip to content

Conversation

@tvandort
Copy link
Contributor

@tvandort tvandort commented Jan 7, 2026

Ticket ENG-2192

Description Of Changes

Missed changelog update.

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
Copy link
Contributor

vercel bot commented Jan 7, 2026

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

2 Skipped Deployments
Project Deployment Review Updated (UTC)
fides-plus-nightly Ignored Ignored Jan 7, 2026 2:55pm
fides-privacy-center Ignored Ignored Jan 7, 2026 2:55pm

@tvandort tvandort marked this pull request as ready for review January 7, 2026 14:55
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 7, 2026

Greptile Summary

This PR adds a missing changelog entry for the security headers feature that was previously merged in PR #7134. A single line is added to the "Added" section of the unreleased changes, documenting the addition of recommended security headers to the Admin UI and Fides API.

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it's a documentation-only changelog update.
  • Perfect score: minimal, focused change that only adds a single line to document an already-merged feature. No code changes, no dependencies affected, no risk of regression. The entry follows existing changelog format and correctly references the associated PR.
  • No files require special attention

Important Files Changed

Filename Overview
CHANGELOG.md Added changelog entry for security headers feature merged in PR #7134. Single line addition to document the new feature correctly.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 7, 2026

Greptile's behavior is changing!

From now on, if a review finishes with no comments, we will not post an additional "statistics" comment to confirm that our review found nothing to comment on. However, you can confirm that we reviewed your changes in the status check section.

This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR".

@tvandort tvandort enabled auto-merge January 7, 2026 14:59
@tvandort tvandort added this pull request to the merge queue Jan 7, 2026
Merged via the queue into main with commit 14643c0 Jan 7, 2026
42 checks passed
@tvandort tvandort deleted the ENG-2192-changelog branch January 7, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants