-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fides ints 121 Marigold Engage by Sailthru Access and Erasure #4826
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Passing run #7548 ↗︎Details:
Review all test suite changes for PR #4826 ↗︎ |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4826 +/- ##
===========================================
+ Coverage 66.31% 86.77% +20.46%
===========================================
Files 345 346 +1
Lines 20855 20909 +54
Branches 2727 2732 +5
===========================================
+ Hits 13829 18144 +4315
+ Misses 6525 2289 -4236
+ Partials 501 476 -25 ☔ View full report in Codecov by Sentry. |
...fides/api/service/saas_request/override_implementations/marigold_engage_request_overrides.py
Dismissed
Show dismissed
Hide dismissed
Co-authored-by: Adrian Galvan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Adrian Galvan <[email protected]>
Closes #
Description Of Changes
Write some things here about the changes and any potential caveats
Code Changes
Steps to Confirm
Pre-Merge Checklist
https://github.com/ethyca/fidesdocs/pull/355
https://github.com/ethyca/fidesdocs/issues/354
CHANGELOG.md
Pre-Merge Checklist
connector_params
identity_email
oridentity_phone_number