Skip to content

feat: add Invite.type#1142

Merged
shiftinv merged 11 commits intomasterfrom
feature/invite-type
Aug 9, 2024
Merged

feat: add Invite.type#1142
shiftinv merged 11 commits intomasterfrom
feature/invite-type

Conversation

@shiftinv
Copy link
Member

@shiftinv shiftinv commented Jan 2, 2024

Summary

Adds support for invite types. This also resolves an old TODO regarding null channels in invites, since friend invites do not link to a specific resource.
I don't think friend invites ever shipped, but the type is included regardless to match the linked PR.

Checklist

  • If code changes were made, then they have been tested
    • I have updated the documentation to reflect the changes
    • I have formatted the code properly by running pdm lint
    • I have type-checked the code by running pdm pyright
  • 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, ...)

This is technically not required since both of these fields have fallbacks,
but still good practice nonetheless.
@shiftinv shiftinv added t: enhancement New feature s: waiting for api/docs Issue/PR is waiting for API support/documentation labels Jan 2, 2024
@shiftinv shiftinv marked this pull request as ready for review January 2, 2024 15:27
Co-authored-by: Victor <[email protected]>
Signed-off-by: shiftinv <[email protected]>
@Victorsitou Victorsitou added the t: api support Support of Discord API features label Mar 22, 2024
@shiftinv shiftinv removed the s: waiting for api/docs Issue/PR is waiting for API support/documentation label May 18, 2024
@shiftinv shiftinv merged commit caf8b92 into master Aug 9, 2024
@shiftinv shiftinv deleted the feature/invite-type branch August 9, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t: api support Support of Discord API features t: enhancement New feature

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants