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: add use_clyde_ai permission and role subscription purchase notifications to SystemChannelFlags #2224

Closed
wants to merge 3 commits into from

Conversation

MCausc78
Copy link

@MCausc78 MCausc78 commented Sep 3, 2023

discord/discord-api-docs#6354

Summary

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.
  • I have updated the changelog to include these changes.

@Lulalaby
Copy link
Member

Lulalaby commented Sep 3, 2023

Well while ur on it, please add SetVoiceChannelStatus

/// <summary>
/// Allows members to create and edit voice channel status.
/// </summary>
[PermissionString("Set Voice Channel Status"), DiscordInExperiment]
SetVoiceChannelStatus = 1L << 48

And look for other possible missing permissions (i think the create_xy are missing)

@@ -75,6 +75,9 @@ These changes are available on the `master` branch, but have not yet been releas
([#2206](https://github.com/Pycord-Development/pycord/pull/2206))
- Added function `Guild.delete_auto_moderation_rule`.
([#2153](https://github.com/Pycord-Development/pycord/pull/2153))
- Added
[`USE_CLYDE_AI` permission](https://github.com/discord/discord-api-docs/pull/6354).
Copy link
Contributor

Choose a reason for hiding this comment

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

What about the role subscription purchase notifications flag?

Copy link
Member

Choose a reason for hiding this comment

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

^ + the changelog entry is entirely wrong

@plun1331
Copy link
Member

@MCausc78 any updates?

@Dorukyum
Copy link
Member

@MCausc78 Clyde has been deactivated, please update the pull request accordingly.

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.

5 participants