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

Fix GuildTemplate struct and documentation #1191

Merged
merged 5 commits into from Jun 19, 2022

Conversation

QPixel
Copy link
Contributor

@QPixel QPixel commented Jun 1, 2022

This is a fix to PR #1091. It seems that the UsageCount field was incorrectly set to a string instead of a int. This PR has been made to fix this. I've tested this fix on my own machine.

Relevant Documentation: https://discord.com/developers/docs/resources/guild-template

@FedorLap2006 FedorLap2006 self-requested a review June 1, 2022 11:52
@FedorLap2006 FedorLap2006 added the breaking changes Contains breaking changes. Should be reflected in the changelog label Jun 1, 2022
structs.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@FedorLap2006 FedorLap2006 left a comment

Choose a reason for hiding this comment

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

You probably commited settings.json and components binary by mistake, so please remove them.

structs.go Outdated Show resolved Hide resolved
@QPixel
Copy link
Contributor Author

QPixel commented Jun 15, 2022

You probably commited settings.json and components binary by mistake, so please remove them.

Whoops, I thought I had deleted those.

@QPixel
Copy link
Contributor Author

QPixel commented Jun 15, 2022

@FedorLap2006 Anything else you need changed?

@FedorLap2006 FedorLap2006 merged commit 5da28ed into bwmarrin:master Jun 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking changes Contains breaking changes. Should be reflected in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants