-
Notifications
You must be signed in to change notification settings - Fork 2
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
👔(dashboard) update consent status logic when update from validated to revoked #341
👔(dashboard) update consent status logic when update from validated to revoked #341
Conversation
f4e57a8
to
ab53cc6
Compare
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.
Current implementation looks a bit too complicated and needs to be more straightforward.
f33a9e4
to
1a118a2
Compare
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.
1a118a2
to
b9d7bf4
Compare
…n checks - Implemented more stringent validation checks during the status change from `VALIDATED` to `REVOKED`. This update ensures that only the necessary fields can be adjusted during this transition. - add validation on the allowed fields - add validation when consent status is updated from validated status - add documentation on business logic of consent in the readme.md file - add custom consent exception - update test - update changelog
b9d7bf4
to
40e0364
Compare
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.
Purpose
Validated consent can be changed to revoked.
Only the
status
andrevoked_at
fields can be modified during this modification.Proposal