Skip to content

TCF Experience option for selecting a restriction configuration#6033

Merged
gilluminate merged 4 commits intomainfrom
gill/LJ-623/tcf-config-option-in-experience
Apr 14, 2025
Merged

TCF Experience option for selecting a restriction configuration#6033
gilluminate merged 4 commits intomainfrom
gill/LJ-623/tcf-config-option-in-experience

Conversation

@gilluminate
Copy link
Copy Markdown
Contributor

@gilluminate gilluminate commented Apr 10, 2025

Closes LJ-623

Description Of Changes

This PR adds support for selecting TCF (Transparency & Consent Framework) Publisher Override configurations when configuring Privacy Experiences. It includes UI changes to allow users to select and manage TCF configurations within the privacy experience settings.

Code Changes

  • Added TCF configuration selection dropdown in Privacy Experience form
  • Added handling for clearing TCF configuration (sets to null when cleared)
  • Added new Cypress test for TCF configuration editing functionality

Steps to Confirm

  1. When running locally, make sure you have latest from fidesplus repo main branch and that TCF is enabled.
  2. Navigate to the consent settings page in the Admin UI (/settings/consent)
  3. Enable the Override vendor purposes option.
  4. Create a new configuration
    • Click the "Create configuration +" button
    • Give your config a name and save it. (you should see the table appear below)
    • Add another
  5. Navigate to a TCF privacy experience configuration
  6. Verify the TCF Configuration dropdown is visible and populated with available configurations
  7. Select a different TCF configuration and save - verify it updates successfully
  8. Clear the TCF configuration and save - verify it clears successfully
  9. Verify the dropdown is disabled if either:
    • Override vendor purposes setting is disabled in consent settings
    • No TCF configurations exist

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
  • 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
Copy Markdown
Contributor

vercel Bot commented Apr 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fides-plus-nightly ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 14, 2025 9:16pm
fides-privacy-center ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 14, 2025 9:16pm

Comment thread CHANGELOG.md
Comment on lines +27 to +30
### Changed
- Changed how TCF Publisher Overrides gets configured in consent settings [#6013](https://github.com/ethyca/fides/pull/6013)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I accidentally added this to the wrong release this morning, so just moving it to the correct spot here

Comment on lines +80 to +84
label: "Reject all",
value: RejectAllMechanism.REJECT_ALL,
},
{
label: "Reject Consent Only",
label: "Reject consent only",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updates just for you @jpople ;)

Copy link
Copy Markdown
Contributor

@jpople jpople left a comment

Choose a reason for hiding this comment

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

Tested locally, everything working as expected for me.

@gilluminate gilluminate force-pushed the gill/LJ-623/tcf-config-option-in-experience branch from 4b6796c to de60ce6 Compare April 14, 2025 21:12
@gilluminate gilluminate merged commit 1716101 into main Apr 14, 2025
17 checks passed
@gilluminate gilluminate deleted the gill/LJ-623/tcf-config-option-in-experience branch April 14, 2025 21:21
@cypress
Copy link
Copy Markdown

cypress Bot commented Apr 14, 2025

fides    Run #12816

Run Properties:  status check passed Passed #12816  •  git commit 1716101ff6: TCF Experience option for selecting a restriction configuration (#6033)
Project fides
Branch Review main
Run status status check passed Passed #12816
Run duration 00m 52s
Commit git commit 1716101ff6: TCF Experience option for selecting a restriction configuration (#6033)
Committer Jason Gill
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 5
View all changes introduced in this branch ↗︎

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