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

fix(docs): update privileged intent information #1542

Merged
merged 2 commits into from
Aug 2, 2022
Merged

fix(docs): update privileged intent information #1542

merged 2 commits into from
Aug 2, 2022

Conversation

JustaSqu1d
Copy link
Member

Summary

This pull request makes Privileged Intents more informative and updates privileged intents information.

Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting, examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.

@Dorukyum
Copy link
Member

Dorukyum commented Aug 1, 2022

I don't think the changes except the April - September fix are needed.

@JustaSqu1d
Copy link
Member Author

JustaSqu1d commented Aug 1, 2022

Currently, this requires opting in explicitly via the developer portal as well.

makes it sound like it's temporary, while it doesn't seem that way. On the other hand, I think stating why it is required to be enabled on the Developer Portal also helps with understanding.

Lulalaby
Lulalaby previously approved these changes Aug 2, 2022
@Lulalaby Lulalaby enabled auto-merge (rebase) August 2, 2022 00:09
@Lulalaby Lulalaby requested a review from Dorukyum August 2, 2022 00:09
plun1331
plun1331 previously approved these changes Aug 2, 2022
@Dorukyum Dorukyum dismissed stale reviews from plun1331 and Lulalaby via f1f8e3b August 2, 2022 09:36
@Dorukyum Dorukyum disabled auto-merge August 2, 2022 09:37
@Dorukyum Dorukyum merged commit 27c088b into Pycord-Development:master Aug 2, 2022
@JustaSqu1d JustaSqu1d deleted the patch-1 branch August 2, 2022 15:22
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