Skip to content

Conversation

@hmd-ali
Copy link
Contributor

@hmd-ali hmd-ali commented Nov 25, 2025

No description provided.

@hmd-ali hmd-ali requested a review from wiktoriavh November 25, 2025 09:23
Comment on lines +10 to +15
if (config.clearGlobalCommands) {
console.log('Clearing global commands...');
await rest.put(Routes.applicationCommands(config.discord.clientId), { body: [] });
console.log('Global commands cleared.');
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is temporary to fix the duplicate commands.
We were always registering global commands that triggered every time the bot started.
When this is live, we could then remove it and only rely on the deploy commands script to deploy commands.

@wiktoriavh wiktoriavh changed the title Feat/command registration feat: command registration Nov 26, 2025
Copy link
Member

@wiktoriavh wiktoriavh left a comment

Choose a reason for hiding this comment

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

After this, we need to run the other pipeline to add the commands, and then make a PR that removes this clearing here?

@hmd-ali
Copy link
Contributor Author

hmd-ali commented Nov 26, 2025

After this, we need to run the other pipeline to add the commands, and then make a PR that removes this clearing here?

This is what the pipeline calls, we can need to run the pipeline once then we remove the "clear global commands" part

@hmd-ali hmd-ali merged commit 61b530e into main Nov 26, 2025
1 check passed
@hmd-ali hmd-ali deleted the feat/command-registration branch November 26, 2025 17:58
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.

3 participants