Skip to content

Intents Application Process#74

Open
StephenDaDev wants to merge 14 commits into
mainfrom
intents-application
Open

Intents Application Process#74
StephenDaDev wants to merge 14 commits into
mainfrom
intents-application

Conversation

@StephenDaDev

Copy link
Copy Markdown
Member

No description provided.

StephenDaDev and others added 13 commits June 13, 2026 00:34
For now, I am placing this in the usage guide to get started, we can move this later if we feel this is not the most appropriate place for it
* Updated FAQ

* Fix grammatical issues

Co-authored-by: Martin <55140357+martinbndr@users.noreply.github.com>
* Updated FAQ

* Fix grammatical issues

Co-authored-by: Martin <55140357+martinbndr@users.noreply.github.com>
@StephenDaDev StephenDaDev requested a review from a team June 28, 2026 13:50
@StephenDaDev StephenDaDev marked this pull request as ready for review June 28, 2026 13:50

@sebkuip sebkuip 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.

Whilst it's a good start, I feel like there's some major points that need to be improved. Please see my detailed explanation below

Comment thread faq/intents-review.md
<figure><img src="../.gitbook/assets/Intents-Review-2.png"><figcaption><p>Questions for Prescence Intent <strong>Not Required</strong> for Modmail.</p></figcaption></figure>
<figure><img src="../.gitbook/assets/Intents-Review-3.png"><figcaption><p>Message Content Intent Questions and Certification of Answers</p></figcaption></figure>

## Suggested Responses

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 just feel like there's gotta be a better way to format all of this. The expand boxes feel kinda needless, as those would normally be used for extra/optional info, but here it's required info.

Also, there's a copy button with code blocks, but because the question and the separator line are also inside this code block, those get copied too.
There's no text wrapping either (which is normal for a code block), so you have no idea what you're copying until you paste it in the application. See image. This really is bad UX. Whilst this is mostly made for people who would be more familiar with bots due to running them on large servers, I feel like it's still justified that they get a good UX.

Image

Comment thread faq/privacy-policy.md
When you send a direct message to the Bot to initiate a support thread or interact with an active thread, we collect and store the following information:

1. **User Data:** Your Discord User ID, username, and avatar.
2. **Message Content:** The text content, attachments, and timestamps of any messages sent to the Bot.

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.

Shouldn't this be attachment links as we don't store attachments directly (except for plugins in some cases)?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yes

Comment thread faq/privacy-policy.md
{% endhint %}

{% hint style="info" %}
**Server Owners**

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 believe that explaining about how the privacy policy is legally binding and that owners should review the policy's statements before submitting it should come BEFORE the policy template first. That would be reasonable effort to ensure we tried to inform users, instead of providing at a later stage that might be overlooked

Comment thread SUMMARY.md
* [Thread Menu](usage-guide/threadmenu.md)
* [Frequently Asked Questions](frequently-asked-questions.md)
* [Intents Review Process](faq/intents-review.md)
* [Instance Privacy Policy](faq/)

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.

Link to the correct document is missing here causing the page to be blank

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