Allow OT migration to auto write to fides preferences api#6099
Merged
eastandwestwind merged 7 commits intomainfrom May 2, 2025
Merged
Allow OT migration to auto write to fides preferences api#6099eastandwestwind merged 7 commits intomainfrom
eastandwestwind merged 7 commits intomainfrom
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
gilluminate
reviewed
Apr 30, 2025
gilluminate
approved these changes
May 1, 2025
Contributor
gilluminate
left a comment
There was a problem hiding this comment.
Looks good and works in testing. Thanks for updating to make it more maintainable through better abstraction! reminder to add a CHANGELOG entry :)
fides
|
||||||||||||||||||||||||||||
| Project |
fides
|
| Branch Review |
main
|
| Run status |
|
| Run duration | 00m 56s |
| Commit |
|
| Committer | Catherine Smith |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
5
|
| View all changes introduced in this branch ↗︎ | |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes https://ethyca.atlassian.net/browse/LJ-722
Description Of Changes
Allow OT migration to auto write to fides preferences api
Code Changes
Steps to Confirm
window.fides_overridesdeclaration. Add a new field,ot_fides_mappingthat will hold your OT -> fides mappings.Example core data:
To prep this, you'll need to JSON.stringify, then URL-encode it.
Here's my resulting fides_overrides obj:
Look at the
groupsparam. Take that value and url-decode it. These groups map to either 0 (opted out) or 1 (opted in).4. Based on our onetrust consent cookie and our mappings, we should see the essential / analytics / advertising / marketing notices opted-in by default, if they exist
5. We should see the
fides_consentcookie written to the browser and consent vals should reflect thewindow.Fides.consentvalues. Note that the consent method in the cookie is "ot_migration"6. We should see new entries in Consent Reporting in Admin UI for the "ot_migration" consent
Pre-Merge Checklist
CHANGELOG.mdupdatedmaindowngrade()migration is correct and works