Skip to content

feat: allow leaving of server#129

Merged
chojs23 merged 4 commits into
chojs23:mainfrom
nexxai:leave-guild
May 27, 2026
Merged

feat: allow leaving of server#129
chojs23 merged 4 commits into
chojs23:mainfrom
nexxai:leave-guild

Conversation

@nexxai
Copy link
Copy Markdown
Contributor

@nexxai nexxai commented May 26, 2026

Summary

Allows a user to leave a guild they are currently a member of

Why

There are guilds that people join that they want to leave for one reason or another

How

I implemented a new section under the server's leader-key menu called "Server management". Currently we only support leaving servers, but if this PR is accepted, there will be a fast-follow PR that supports joining new servers.

Testing

  • cargo fmt --all --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test --all-features
  • Manual test in a real terminal (describe below)

OS: macOS Tahoe 26.5
Terminal: Ghostty 1.3.1

Screenshots or recordings

[LEADER] key is pressed:
Screenshot 2026-05-25 at 6 59 16 PM

s is pressed:
Screenshot 2026-05-25 at 6 59 26 PM

l (lowercase "L") is pressed:
Screenshot 2026-05-25 at 6 59 37 PM

Checklist

  • One logical change per PR.
  • Link a related issue.
  • No tokens, passwords, MFA codes, or raw auth bodies in code, tests, or logs.
  • Change does not add self-bot automation, mass actions, or scraping.
  • Updated README.md, if behavior or workflows changed.

@nexxai
Copy link
Copy Markdown
Contributor Author

nexxai commented May 26, 2026

Fixed merge conflicts. @chojs23 this is ready for review

@chojs23
Copy link
Copy Markdown
Owner

chojs23 commented May 27, 2026

Why don't you put it in the server action?

@nexxai
Copy link
Copy Markdown
Contributor Author

nexxai commented May 27, 2026

@chojs23 Originally I was thinking about adding other tools that would live in the "management" sphere, but for now, I've moved it to the server actions.

Screenshot 2026-05-26 at 6 45 53 PM

Comment thread src/tui/ui/popups/confirmation.rs Outdated
Comment thread src/tui/ui/popups/confirmation.rs Outdated
Comment thread src/tui/ui/popups/confirmation.rs Outdated
@chojs23
Copy link
Copy Markdown
Owner

chojs23 commented May 27, 2026

One possible future direction is to keep “Server Management” as a separate action group, where we can add broader server-level settings later.

@chojs23 chojs23 merged commit 59ec291 into chojs23:main May 27, 2026
2 checks passed
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.

2 participants