Skip to content

[Team Deletions] Act upon scheduled deletions (remove team's dashboards) - #6627

Open
aerosol wants to merge 2 commits into
team-sites-deletions-emailsfrom
team-sites-deletions-deletion-executor
Open

[Team Deletions] Act upon scheduled deletions (remove team's dashboards)#6627
aerosol wants to merge 2 commits into
team-sites-deletions-emailsfrom
team-sites-deletions-deletion-executor

Conversation

@aerosol

@aerosol aerosol commented Aug 31, 2026

Copy link
Copy Markdown
Member

On top of #6624

@aerosol
aerosol changed the base branch from master to team-sites-deletions-emails August 31, 2026 06:37
@aerosol
aerosol force-pushed the team-sites-deletions-deletion-executor branch from e466999 to 44a7c49 Compare August 31, 2026 06:42
@aerosol
aerosol requested a review from ukutaht August 31, 2026 06:44
@aerosol
aerosol requested a review from zoldar September 1, 2026 08:41
from(sch in TeamDeletionSchedule,
where: sch.status == :reminder_sent,
where: sch.deletion_date <= ^today,
preload: [:team]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Preload team from a join perhaps?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aerosol
aerosol force-pushed the team-sites-deletions-deletion-executor branch from 6c43cd5 to bfd42b0 Compare September 3, 2026 07:05
@aerosol
aerosol force-pushed the team-sites-deletions-deletion-executor branch from bfd42b0 to a17c40a Compare September 3, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants