feat: add multi-ID delete support with per-ID results #79

shoko · 2026-04-04T15:07:18+02:00

shoko commented

2026-04-04 15:07:18 +02:00

Summary

Implements multi-ID delete support for issue #47:

Multi-ID support: /delete 1 2 3 deletes multiple bounties at once
Per-ID results: Shows success/failure for each ID
Admin-only: Already enforced via existing service layer
Soft delete: Already implemented via deleted_at timestamp

Changes

core/services.py

Add delete_bounties() method that returns individual results per ID

apps/telegram-bot/commands.py

Update cmd_delete() to accept multiple IDs and show per-ID messages

tests/test_services.py

Add 3 tests for delete_bounties() method

Fixes #47

## Summary Implements multi-ID delete support for issue #47: - **Multi-ID support**: /delete 1 2 3 deletes multiple bounties at once - **Per-ID results**: Shows success/failure for each ID - **Admin-only**: Already enforced via existing service layer - **Soft delete**: Already implemented via deleted_at timestamp ## Changes ### core/services.py - Add delete_bounties() method that returns individual results per ID ### apps/telegram-bot/commands.py - Update cmd_delete() to accept multiple IDs and show per-ID messages ### tests/test_services.py - Add 3 tests for delete_bounties() method Fixes #47

shoko added 1 commit 2026-04-04 15:07:18 +02:00

feat: add multi-ID delete support with per-ID results 90b0b564c2

- Add delete_bounties() method to BountyService that returns individual
  results per bounty ID (deleted, not_found, permission_denied)
- Update cmd_delete to accept multiple IDs and show per-ID messages
- Add 3 tests for delete_bounties method

Example output:
/delete 1 2 3
✅ Bounty #1 deleted.
✅ Bounty #2 deleted.
⛔ Bounty #3 not found.

Fixes #47

shoko merged commit a727751978 into main

2026-04-04 15:28:37 +02:00

shoko referenced this issue from a commit

2026-04-04 15:28:38 +02:00

Merge pull request 'feat: add multi-ID delete support with per-ID results' (#79) from fix/issue-47 into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/jigaido#79