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

DiscordServerStatus heads up settings #306

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Buff-oG
Copy link

@Buff-oG Buff-oG commented Sep 15, 2023

DiscordServerStatus heads up settings

README.md and the DiscordServerStatus plugin have a extra text lenght to inform everyone that you must have "Message Content Intent" enable on your discord Bot. Otherwise it won't be able to read the request inside the discord channel.

DiscordServerStatus heads up settings

README.md and the DiscordServerStatus plugin have a extra text lenght to inform everyone that you must have "Message Content Intent" enable on your discord Bot. Otherwise it won't be able to read the request inside the discord channel.
Copy link
Contributor

@fantinodavide fantinodavide left a comment

Choose a reason for hiding this comment

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

Looks good in my opinion, no changes to the plugin code that could break its functionalities, but I think this should also be specified in the main README.md file, in the specific part about discord connector configuration.

@Buff-oG
Copy link
Author

Buff-oG commented Dec 26, 2023

Still awaiting merge.

@Buff-oG
Copy link
Author

Buff-oG commented Jan 31, 2024

Awaiting merge?

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.

4 participants