Skip to content

Add docs for privileged intents#1554

Merged
ajax146 merged 18 commits into
mainfrom
PrivilegedInentsDocs
Jun 12, 2026
Merged

Add docs for privileged intents#1554
ajax146 merged 18 commits into
mainfrom
PrivilegedInentsDocs

Conversation

@ajax146

@ajax146 ajax146 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Mr-KayZ

Mr-KayZ commented Jun 11, 2026

Copy link
Copy Markdown
Member

Couple of issues:

Guild Presences intent has zero documentation

  • main.py uses discord.Intents.all() which does implictly request Guild Presences privileged intent.
  • This doc is not present anywhere from what I searched, so you can do one of the following:
    • Remove discord.Intents.all() entirely and replace it with minimal intents objects that only enable members and message_content, and remove the word Presence from the README instruction.
    • Alternatively, create the doc for documentation/Privileged-Intents/Guild Presences.md.
  • Either way, README.md needs to be updated as well after this.

Couple of modules are documented under the wrong module path

Some of the modules in Message Content.md are listed under the wrong path:

  • moderation.listen - Actual path is in modules/administration/listen.py - Rename it to administration.listen in Message Content.md
  • moderation.paste - Actual path is in modules/operation/paste.py - Rename it to operation.paste in Message Content.md

Minor issues

  • Spelling mistake for moderation.event, the path is called modules/moderation/events.py - with the trailing s, so just rename moderation.event in Message Content.md to moderation.events.

@ajax146

ajax146 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

So as far as the first point goes, intent and privileged intent are not the same thing. There are 18 total intents on discord, only 3 of which are considered to be privileged. I have every intention of changing the code to remove the presences intent, since its not really used by this bot (see #1556)

My plan is to swap to default intents and then explicitly enable members and message_content for TS.
With refactoring (see #1555) I do believe the modmail bot will need nothing more than the default intents

As far as all the typos in here, I'll get to fixing them

@ajax146

ajax146 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

@Mr-KayZ I have re-written basically every word in the docs now, if you want to re-review it I am sure there are more typos to be found.

@dkay0670

Copy link
Copy Markdown
Contributor

Is this the documentation that is getting submitted to Discord for the application?

@ajax146

ajax146 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

I am not sure exactly what discord will be asking for, as I am still not allowed to apply. Discord is batching opening applications to developers so I will have to wait my turn to see what is asked for.

The intention is to use this as documentation for what privileged intents are being used and why we need them, as I am confident that will be a question asked.

@ajax146 ajax146 merged commit 061f626 into main Jun 12, 2026
5 checks passed
@ajax146 ajax146 deleted the PrivilegedInentsDocs branch June 13, 2026 02:41
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