Skip to content

add delete-rooms admin command#339

Draft
donjuanplatinum wants to merge 18 commits intomatrix-construct:mainfrom
donjuanplatinum:delete-rooms
Draft

add delete-rooms admin command#339
donjuanplatinum wants to merge 18 commits intomatrix-construct:mainfrom
donjuanplatinum:delete-rooms

Conversation

@donjuanplatinum
Copy link
Copy Markdown
Contributor

@donjuanplatinum donjuanplatinum commented Feb 25, 2026

This subcommand delete-rooms can delete rooms by rooms index like
!admin rooms delete-rooms 1 2 3 --page 1

@jevolk jevolk added the feature New feature or functionality that didn't exist. label Feb 27, 2026
@donjuanplatinum
Copy link
Copy Markdown
Contributor Author

It looks like an unnecessary trailing comma in src/core/utils/two_phase_counter.rs at the line 142
debug_assert!(self.check_pending(id), "sequence number must be currently pending",);
call triggered a CI failure in Clippy. I've removed it in the latest commit.

@jevolk
Copy link
Copy Markdown
Member

jevolk commented Feb 27, 2026

It's probably a new nightly lint and you don't have to worry about it, thank you though 🙏🏻

@jevolk jevolk force-pushed the delete-rooms branch 2 times, most recently from 7eaf1f6 to 345b2ed Compare February 27, 2026 23:34
@dasha-uwu dasha-uwu marked this pull request as draft March 6, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or functionality that didn't exist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants