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

Forum channels #1246

Merged
merged 9 commits into from
Sep 29, 2022
Merged

Forum channels #1246

merged 9 commits into from
Sep 29, 2022

Conversation

FedorLap2006
Copy link
Collaborator

Implementation of newly added forum channels.

Useful links

Initial implementation of forum channels REST API
@FedorLap2006 FedorLap2006 added feature Feature implementation high priority Issue or PR with high priority of merge labels Sep 17, 2022
@FedorLap2006 FedorLap2006 self-assigned this Sep 17, 2022
Copy link

@fsandov fsandov left a comment

Choose a reason for hiding this comment

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

LGTM! 🔥 🚀

restapi.go Outdated Show resolved Hide resolved
restapi.go Outdated Show resolved Hide resolved
restapi.go Outdated Show resolved Hide resolved
restapi.go Outdated Show resolved Hide resolved
structs.go Outdated Show resolved Hide resolved
structs.go Outdated Show resolved Hide resolved
structs.go Outdated Show resolved Hide resolved
structs.go Outdated Show resolved Hide resolved
Added periods in the end of all documentation comments
Fix incorrect value of ChannelFlagRequireTag.
Rename DefaultReaction to ForumDefaultReaction.
Use ForumDefaultReaction instead of DefaultReaction in DefaultReactionEmoji field.
restapi.go Outdated Show resolved Hide resolved
structs.go Outdated Show resolved Hide resolved
* Change "GUILD_FORUM" to "forum" in comment to ForumTag
* Fix spacing for documentation of embeds parameter in ForumThreadStartEmbeds
Align documentation for parameters of ForumThreadStartComplex.
@FedorLap2006 FedorLap2006 merged commit e570648 into master Sep 29, 2022
LWCoder pushed a commit to AtTheTavern/discordgo that referenced this pull request Apr 11, 2023
* feat: forums

Initial implementation of forum channels REST API

* fix: linter

* feat: cosmetic changes

Added periods in the end of all documentation comments

* fix(ChannelFlagRequireTag): incorrect value

Fix incorrect value of ChannelFlagRequireTag.

* refactor(DefaultReaction): rename to ForumDefaultReaction

Rename DefaultReaction to ForumDefaultReaction.

* fix(Channel): use ForumDefaultReaction

Use ForumDefaultReaction instead of DefaultReaction in DefaultReactionEmoji field.

* fix: gofmt

* feat: cosmetic changes

* Change "GUILD_FORUM" to "forum" in comment to ForumTag
* Fix spacing for documentation of embeds parameter in ForumThreadStartEmbeds

* docs(ForumThreadStartComplex): align parameters

Align documentation for parameters of ForumThreadStartComplex.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature implementation high priority Issue or PR with high priority of merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants