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 source_invite_code to guildmemberdata #39

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LawMixer
Copy link

Drafting this pull request until it is officially added to Discord API, though it is soft released just not announced. Will undraft it when it is timed to merge it.

Link: discord/discord-api-docs#3210 (comment)

@LawMixer
Copy link
Author

I'm ready whenever the maintainers are, everything should be completed! Thank you!

Copy link
Owner

@Bluenix2 Bluenix2 left a comment

Choose a reason for hiding this comment

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

Hi, I've been thinking about this pull request a bit, and waiting to hear about any official news.

Thanks a lot for your first PR! ❤️ Do you know more about the nature of this new field than the discussion you linked?

I am hesitant to add unofficial fields because they can quickly grow out of hand, but that isn't to say that this library follows the docs religiously. It deviates from the docs when doing so improves the typing experience (such as in the case of channels). However, I don't believe this field matches that criteria.

Because it isn't in the official docs, and a quick search on discord-api-types didn't give any results (as of writing), I would like to hold off on merging this unless there is a big need to.

Thanks again!

discord_typings/_resources/_guild.py Outdated Show resolved Hide resolved
@LawMixer
Copy link
Author

LawMixer commented Apr 2, 2024

This change has been slowly coming out for all users, though it seems Discord has not made any pull request for their API documentations for this. Thank you for having me! <3

I just committed a new version with your changes, let me know if there's anything else I can do!

@LawMixer LawMixer requested a review from Bluenix2 April 4, 2024 14:36
@LawMixer
Copy link
Author

LawMixer commented Apr 4, 2024

Thanks a lot for your first PR! ❤️ Do you know more about the nature of this new field than the discussion you linked?

No yet announced from Discord officially, but we've seen ``source_invite_code show up in the Members page on people's discord server, and it is in the member search endpoint.

We've gotten an update from Discord Support on X that pushed things along wayyy quicker, so if you want to wait on them for the official announcement of this change, that's fine by me.

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.

None yet

3 participants