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

Guild embed was renamed to widget #1536

Merged
merged 3 commits into from
May 13, 2020
Merged

Conversation

advaith1
Copy link
Contributor

Updates widget/embed stuff in the Guild page according to my discussion with night in Discord Developers

@@ -19,8 +19,8 @@ Guilds in Discord represent an isolated collection of users and channels, and ar
| region | string | [voice region](#DOCS_RESOURCES_VOICE/voice-region-object) id for the guild |
| afk_channel_id | ?snowflake | id of afk channel |
| afk_timeout | integer | afk timeout in seconds |
| embed_enabled? | boolean | whether this guild is embeddable (e.g. widget) |
| embed_channel_id? | ?snowflake | if not null, the channel id that the widget will generate an invite to |
| embed_enabled? | boolean | whether or not the server widget is enabled (deprecated, replaced with widget_enabled) |
Copy link
Contributor

Choose a reason for hiding this comment

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

I know it was server before as well. But shouldn't guild be preferred? Considering this is a guild object structure and not a server object structure. Maybe a nitpick.
Otherwise looks ok.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They're referred to as servers in many other parts of this page too

Copy link
Contributor

Choose a reason for hiding this comment

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

I guess it should be fixed in a separate PR. It's just annoying with all these inconsistency

@msciotti msciotti merged commit 1feb353 into discord:master May 13, 2020
Erk- added a commit to Erk-/twilight that referenced this pull request May 27, 2020
discord/discord-api-docs#1536

Signed-off-by: Valdemar Erk <valdemar@erk.io>
Erk- added a commit to Erk-/twilight that referenced this pull request May 27, 2020
discord/discord-api-docs#1536

Signed-off-by: Valdemar Erk <valdemar@erk.io>
Erk- added a commit to twilight-rs/twilight that referenced this pull request May 28, 2020
* Update the model with the newest changes.

- Document optional Connection object fields.
discord/discord-api-docs#1466

- Update invite create event fields and example invite object.
discord/discord-api-docs#1438

- New member count fields on get guilds
discord/discord-api-docs#1528

- Add support for `include_roles` when pruning.
discord/discord-api-docs#1563

- Add `max_video_channel_users` to guilds.
discord/discord-api-docs#1512


- Rename guild embed to guild widget.
discord/discord-api-docs#1536

Signed-off-by: Valdemar Erk <valdemar@erk.io>
@advaith1 advaith1 deleted the widget-update branch October 9, 2020 02:24
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.

3 participants