Skip to content

fix(autodelete): don't filter out entries with count=0#5635

Merged
TBonnin merged 1 commit into
masterfrom
tbonnin/nan-2545/candidate-zero
Mar 17, 2026
Merged

fix(autodelete): don't filter out entries with count=0#5635
TBonnin merged 1 commit into
masterfrom
tbonnin/nan-2545/candidate-zero

Conversation

@TBonnin
Copy link
Copy Markdown
Collaborator

@TBonnin TBonnin commented Mar 13, 2026

Take into account entries in the records_count table that have count = 0 when selecting candidates for records autodelete so the entry is cleaned up. Right now those entries are kept in the db.


This PR removes the filter that excluded record_counts rows where count equals 0 when selecting auto-delete candidates. This ensures stale count entries with zero records are eligible for cleanup.


This summary was automatically generated by @propel-code-bot

Take into account entries in the records_count table that have count = 0
when selecting candidates for records autodelete so the entry is cleaned up
@linear
Copy link
Copy Markdown

linear Bot commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

@propel-code-bot propel-code-bot Bot left a comment

Choose a reason for hiding this comment

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

Review completed with no issues identified.

Status: No Issues Found | Risk: Low

Review Details

📁 1 files reviewed | 💬 0 comments

Instruction Files
├── .claude/
│   ├── agents/
│   │   └── nango-docs-migrator.md
│   └── skills/
│       ├── agent-builder-skill/
│       │   ├── EXAMPLES.md
│       │   └── SKILL.md
│       ├── creating-integration-docs/
│       │   └── SKILL.md
│       └── creating-skills-skill/
│           └── SKILL.md
├── AGENTS.md
└── GEMINI.md

@TBonnin TBonnin requested a review from a team March 13, 2026 13:14
@TBonnin TBonnin added this pull request to the merge queue Mar 17, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 17, 2026
@TBonnin TBonnin added this pull request to the merge queue Mar 17, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 17, 2026
@TBonnin TBonnin added this pull request to the merge queue Mar 17, 2026
Merged via the queue into master with commit df662dd Mar 17, 2026
25 checks passed
@TBonnin TBonnin deleted the tbonnin/nan-2545/candidate-zero branch March 17, 2026 18:20
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.

3 participants