Skip to content

ENG-2641: Update changelog for pw encoding#7383

Merged
tvandort merged 2 commits intomainfrom
ENG-2641-update-changelog
Feb 12, 2026
Merged

ENG-2641: Update changelog for pw encoding#7383
tvandort merged 2 commits intomainfrom
ENG-2641-update-changelog

Conversation

@tvandort
Copy link
Contributor

@tvandort tvandort commented Feb 12, 2026

Ticket ENG-2641

Description Of Changes

Update changelog for release cherry-pick.

Code Changes

Changelog.

Steps to Confirm

None

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 Feb 12, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Preview Feb 12, 2026 9:05pm
fides-privacy-center Ignored Ignored Feb 12, 2026 9:05pm

Request Review

@tvandort tvandort enabled auto-merge February 12, 2026 21:02
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 12, 2026

Greptile Overview

Greptile Summary

This PR updates the changelog for version 2.79.0 by consolidating the entry for PR #7375 (database password encoding fix) from its YAML file into CHANGELOG.md and deleting the YAML file.

Issues found:

  • The changelog entry was added to the ### Added section (line 34), but the original YAML file specifies type: Fixed, indicating this should be in the ### Fixed section instead

Confidence Score: 2/5

  • This PR has a categorization error that should be corrected before merging
  • The changelog entry is placed in the wrong section (Added instead of Fixed), which violates the changelog format conventions and could mislead readers about the nature of the change
  • Pay close attention to CHANGELOG.md - the entry needs to be moved to the correct section

Important Files Changed

Filename Overview
CHANGELOG.md Added changelog entry for PR #7375 under wrong section - entry is in Added but should be in Fixed based on the YAML file type

Last reviewed commit: a0ff05f

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

CHANGELOG.md Outdated
- Celery workers now have an HTTP healthcheck endpoint that can be used to check if the workers are running for environments that do not support running a command to check if the workers are running. [#7091](https://github.com/ethyca/fides/pull/7091)
- Added distance column to StagedResourceAncestor table [#7326](https://github.com/ethyca/fides/pull/7326) https://github.com/ethyca/fides/labels/db-migration
- Added database schema for privacy assessment answer history tracking [#7330](https://github.com/ethyca/fides/pull/7330) https://github.com/ethyca/fides/labels/db-migration
- Encode database password when reused for readonly connection [#7375](https://github.com/ethyca/fides/pull/7375)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this entry is in the ### Added section but the YAML file shows type: Fixed - should be moved to the ### Fixed section

Suggested change
- Encode database password when reused for readonly connection [#7375](https://github.com/ethyca/fides/pull/7375)

@tvandort tvandort added this pull request to the merge queue Feb 12, 2026
Merged via the queue into main with commit bbe5a02 Feb 12, 2026
38 of 39 checks passed
@tvandort tvandort deleted the ENG-2641-update-changelog branch February 12, 2026 21:45
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.

2 participants