Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: omit the warn log message about getting slash commands for all guilds #477

Merged
merged 10 commits into from
Jul 18, 2022
Merged

Conversation

sawa-ko
Copy link
Contributor

@sawa-ko sawa-ko commented Jul 9, 2022

For example, if we have a bot with 5K servers and 30% of them invited the bot when it didn't have slash commands yet, the console will be filled with Spam about the warning message of not being able to get the guild commands.

By having it as an array type, it is impossible to avoid that, and this way you can make it work globally without having to put each guild id in the option.

@sawa-ko sawa-ko changed the title feat: omit the error message about getting slash commands for all guilds feat: omit the warn log message in logger about getting slash commands for all guilds Jul 9, 2022
@sawa-ko sawa-ko changed the title feat: omit the warn log message in logger about getting slash commands for all guilds feat: omit the warn log message about getting slash commands for all guilds Jul 9, 2022
Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
@sawa-ko sawa-ko requested a review from vladfrangu July 10, 2022 06:24
src/lib/SapphireClient.ts Outdated Show resolved Hide resolved
src/lib/utils/application-commands/getNeededParameters.ts Outdated Show resolved Hide resolved
src/lib/SapphireClient.ts Outdated Show resolved Hide resolved
sawa-ko and others added 4 commits July 10, 2022 11:38
Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
@favna favna merged commit 70b4861 into sapphiredev:main Jul 18, 2022
@sawa-ko sawa-ko deleted the fawnauwu branch July 18, 2022 20:19
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