shoko/jigaido
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

feat(/admin): add /admin command for admin management #66

Closed
shoko wants to merge 1 commits from fix/issue-52 into main
pull from: fix/issue-52
merge into: shoko:main
Conversation 0 Commits 1 Files Changed 2 +98
shoko commented 2026-04-04 09:01:58 +02:00
Owner
Copy Link

Summary

Add /admin command to manage room admins.

Features

  • /admin remove @username - Remove user from admin list
  • /admin add @username - Add user to admin list
  • /admin list - List all admins

Behavior

/admin remove @username

  • Success: ✅ @username is no longer an admin.
  • Error (not admin): ⛔ @username is not an admin.
  • Error (not authorized): ⛔ Only admins can remove admins.

/admin add @username

  • Success: ✅ @username is now an admin.
  • Error (already admin): ⛔ @username is already an admin.
  • Error (not authorized): ⛔ Only admins can add admins.

/admin list

  • Shows: Admins: admin_id:123 admin_id:456

Implementation

  • Uses existing BOUNTY_SERVICE.add_admin(), remove_admin(), and list_admins() methods
  • Resolves @username to user ID via Telegram bot API
  • Admin-only operations check requesting user is admin

Fixes #52

## Summary Add `/admin` command to manage room admins. ## Features - `/admin remove @username` - Remove user from admin list - `/admin add @username` - Add user to admin list - `/admin list` - List all admins ## Behavior ### /admin remove @username - Success: `✅ @username is no longer an admin.` - Error (not admin): `⛔ @username is not an admin.` - Error (not authorized): `⛔ Only admins can remove admins.` ### /admin add @username - Success: `✅ @username is now an admin.` - Error (already admin): `⛔ @username is already an admin.` - Error (not authorized): `⛔ Only admins can add admins.` ### /admin list - Shows: `Admins: admin_id:123 admin_id:456` ## Implementation - Uses existing `BOUNTY_SERVICE.add_admin()`, `remove_admin()`, and `list_admins()` methods - Resolves @username to user ID via Telegram bot API - Admin-only operations check requesting user is admin Fixes #52
shoko added 1 commit 2026-04-04 09:01:59 +02:00
feat(/admin): add /admin command for admin management dfaf633707 
- Add /admin remove @username - removes user from admin list
- Add /admin add @username - adds user to admin list (bonus)
- Add /admin list - lists all admins

Resolves #52
shoko closed this pull request 2026-04-04 09:20:11 +02:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
adapter
Storage adapter
 architecture
Architecture/design
 cleanup
Cleanup task
 cli
CLI application
 core
Core domain layer
 enhancement
need-user-approval
phase-1
Phase 1 task
 phase-2
Phase 2 implementation task
 telegram-bot
Telegram bot app
 tests
Tests
 v2
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
han shoko
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shoko/jigaido#66
Delete Branch "fix/issue-52"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?