Skip to content

Replace hardcoded list of allowed server invites - #175

Merged
jeremy-rifkin merged 5 commits into
TCCPP:mainfrom
michael-kenzel:allowed-invite-management
Oct 2, 2025
Merged

Replace hardcoded list of allowed server invites#175
jeremy-rifkin merged 5 commits into
TCCPP:mainfrom
michael-kenzel:allowed-invite-management

Conversation

@michael-kenzel

Copy link
Copy Markdown
Contributor

No description provided.

@jeremy-rifkin jeremy-rifkin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this! Pretty much looks good just a couple quick comments

Comment thread src/components/anti-invite-links.ts Outdated
Comment thread src/components/anti-invite-links.ts Outdated
Comment thread src/components/anti-invite-links.ts Outdated
@michael-kenzel
michael-kenzel force-pushed the allowed-invite-management branch from 9512600 to 6cf8a6b Compare October 2, 2025 14:05
@michael-kenzel
michael-kenzel force-pushed the allowed-invite-management branch from 6cf8a6b to edd052a Compare October 2, 2025 14:07
@sonarqubecloud

sonarqubecloud Bot commented Oct 2, 2025

Copy link
Copy Markdown

@jeremy-rifkin jeremy-rifkin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

.set_description("manage allowed server invites")
.add_subcommand(
new TextBasedCommandBuilder("add", EarlyReplyMode.ephemeral)
.set_permissions(Discord.PermissionFlagsBits.Administrator)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll have to ensure subcommands properly override permissions

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup

@jeremy-rifkin
jeremy-rifkin merged commit 8d31999 into TCCPP:main Oct 2, 2025
6 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