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

feat: add /admin command to list room admins #81

Closed
shoko wants to merge 1 commits from fix/issue-50 into main
pull from: fix/issue-50
merge into: shoko:main
Conversation 0 Commits 1 Files Changed 1 +35 -1
shoko commented 2026-04-04 15:08:05 +02:00
Owner
Copy Link

Summary

Implement /admin command as specified in issue #50:

  • Lists all admin user IDs for the current room
  • Output format: Room Admins:\n- @user1\n- @user2
  • Shows No admins configured for this room. if none exist
  • Available to everyone (no permission check needed)

Changes

  • Add _find_username_by_user_id() helper function to look up usernames from user IDs
  • Update cmd_admin() to support listing when called with no args or with list subcommand
  • Update /help to include /admin command

Testing

/admin
Room Admins:
- @123456789
- @987654321

Closes #50

## Summary Implement /admin command as specified in issue #50: - Lists all admin user IDs for the current room - Output format: `Room Admins:\n- @user1\n- @user2` - Shows `No admins configured for this room.` if none exist - Available to everyone (no permission check needed) ## Changes - Add `_find_username_by_user_id()` helper function to look up usernames from user IDs - Update `cmd_admin()` to support listing when called with no args or with `list` subcommand - Update `/help` to include /admin command ## Testing ``` /admin Room Admins: - @123456789 - @987654321 ``` Closes #50
shoko added 1 commit 2026-04-04 15:08:05 +02:00
feat: add /admin command to list room admins 5e9ec3a5ec 
Implement /admin command as specified in issue #50:
- Lists all admin user IDs for the current room
- Output format: 'Room Admins:\n- @user1\n- @user2'
- Shows 'No admins configured for this room.' if none exist
- Available to everyone (no permission check needed)

Changes:
- Add _find_username_by_user_id helper function
- Update cmd_admin to support listing when called with no args
- Update /help to include /admin command

Closes #50
shoko closed this pull request 2026-04-04 15:29:04 +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#81
Delete Branch "fix/issue-50"

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?