Skip to content

Conversation

6543
Copy link
Member

@6543 6543 commented Jan 22, 2025

... filter dismissed via SQL more

for tests just create this database entry:

id type dissmissed
1 4 0
2 3 1

currently I discover this edgecases in the wild :/

with this patch the review request can still be deleted


Sponsored by Kithara Software GmbH

@6543 6543 added the type/bug label Jan 22, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 22, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Jan 22, 2025
@lunny
Copy link
Member

lunny commented Jan 22, 2025

Could you add some tests?

@6543 6543 changed the title Make it possible to delete stale review requests that incorrectly remain in DB WIP: Make it possible to delete stale review requests that incorrectly remain in DB Jan 22, 2025
Signed-off-by: 6543 <6543@obermui.de>
@6543 6543 changed the title WIP: Make it possible to delete stale review requests that incorrectly remain in DB Make it possible to delete stale review requests that incorrectly remain in DB Sep 1, 2025
@6543 6543 requested a review from lunny September 1, 2025 15:01
@6543 6543 mentioned this pull request Sep 8, 2025
6 tasks
@6543 6543 requested a review from lunny September 16, 2025 14:24
@6543 6543 requested a review from techknowlogick September 18, 2025 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/go Pull requests that update Go code type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants