-
Notifications
You must be signed in to change notification settings - Fork 80
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
Fix LJ-447 #5829
Fix LJ-447 #5829
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
61a42a0
to
26029d6
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5829 +/- ##
==========================================
- Coverage 86.97% 86.94% -0.03%
==========================================
Files 407 406 -1
Lines 25021 25007 -14
Branches 2696 2695 -1
==========================================
- Hits 21762 21743 -19
- Misses 2676 2682 +6
+ Partials 583 582 -1 ☔ View full report in Codecov by Sentry. |
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.
✨
fides
|
Project |
fides
|
Branch Review |
main
|
Run status |
|
Run duration | 00m 50s |
Commit |
|
Committer | erosselli |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
0
|
|
0
|
|
5
|
View all changes introduced in this branch ↗︎ |
Description
This undos some changes made in commit 1cf38cd, which are breaking consent requests on fidesplus. I leave only the changes for the
poll_for_exited_privacy_request_tasks
which aren't breaking any flows, and rolled back all the changes related to theGraphTask
code.Steps to Confirm
Pre-Merge Checklist
CHANGELOG.md
updatedmain
downgrade()
migration is correct and works