feat: Implement deletion notification system for expired GDPR reports#60
feat: Implement deletion notification system for expired GDPR reports#60ttak-apphelix wants to merge 8 commits intomasterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Implements a Google Drive comment-based notification step intended to alert external partner contacts before GDPR partner report CSVs are deleted by the existing expiration cleanup script.
Changes:
- Added a deletion-notification message template and logic to post comments on soon-to-be-deleted report files.
- Added partner-permission inspection to derive external email recipients (with denied-domain filtering) for notifications.
- Hooked the notification pass into the
delete_expired_reportscommand prior to deletion.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ttak-apphelix
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description:
Implements a Google Drive comment-based notification step intended to alert external partner contacts before GDPR partner report CSVs are deleted by the existing expiration cleanup script.