Skip to content

feat: allow removing members from groups#1110

Open
amycatgirl wants to merge 9 commits intostoatchat:mainfrom
amycatgirl:feat/remove-from-group
Open

feat: allow removing members from groups#1110
amycatgirl wants to merge 9 commits intostoatchat:mainfrom
amycatgirl:feat/remove-from-group

Conversation

@amycatgirl
Copy link
Copy Markdown
Contributor

Apparently group owners could not remove members from groups because there wasn't an option to do so.

Fixes #1026

How was this PR tested?

Tried removing my alt account from a group I had created on the main stoat instance.

  • Was able to remove members from a group
    • Could not remove myself from the group
    • Could not remove other members without ManageChannel permission
    • Username/Display Name displays properly inside modal
    • Group name displays properly inside modal

Screenshots & Screencasts (if appropriate)

image image

Checklist:

  • I have carefully read the contributing guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR
  • I have confirmed that any new dependencies are strictly necessary
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code

Please declare, if any, LLM usage involved in creating this PR

Jetbrains WebStorm has an inline completion feature enabled by default that I wasn't aware at the time of writing the pull request's code.

Signed-off-by: Amy <amy+git@amogus.cloud>
closes stoatchat#1026

Signed-off-by: Amy <amy+git@amogus.cloud>
@amycatgirl amycatgirl requested review from a team as code owners April 11, 2026 19:50
Signed-off-by: Amy <amy+git@amogus.cloud>
Signed-off-by: Amy <amy+git@amogus.cloud>
im on windows, which prefers crlf
but prettier expects lf by default

this makes prettier shut the fuck up about those since they add visual noise (webstorm and prettier lint warn about those) and git converts those to crlf after committing anyway (see core.autocrlf)

Signed-off-by: Amy <amy+git@amogus.cloud>
Signed-off-by: Amy <amy+git@amogus.cloud>
@amycatgirl
Copy link
Copy Markdown
Contributor Author

amycatgirl commented Apr 11, 2026

I had to change the .prettierrc file because WebStorm and prettier wouldn't shut up about CRLF and was preventing me from running the formatter correctly on windows.

Comment thread packages/client/components/app/menus/UserContextMenu.tsx Outdated
Signed-off-by: Amy <amy+git@amogus.cloud>
@amycatgirl amycatgirl requested a review from insertish April 12, 2026 13:48
@gabrielfordevelopment gabrielfordevelopment added enhancement New feature or request ux review This needs UX review labels Apr 22, 2026
Signed-off-by: Amy <138383945+amycatgirl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ux review This needs UX review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Removing users from groups

4 participants