-
Notifications
You must be signed in to change notification settings - Fork 21
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
Discordgo rewrite #129
Discordgo rewrite #129
Conversation
Thanks for your contributions! I’ll check on this weekend. @Cyb3r-Jak3 |
@Lucky3028 friendly bump on this |
I'm very sorry for the huge delay in response!!! I will confirm within two weeks. @Cyb3r-Jak3 |
Unit tests have been added. |
* Start adding acceptance tests * Add test for permission datasource * Add test for role data source * Add test for system channel data source * Add test for catergory channel resource * Add test for text channel resource * Add test for invite resource * Simplfy checking for string * Add test for channel permission resource * Add test for webhook resource * Add test for message resource * Add test for role resource * Finish unit tests
@Cyb3r-Jak3 |
@Lucky3028 Should be all set |
@Cyb3r-Jak3 Thanks! |
Rewrite the provider to use https://github.com/bwmarrin/discordgo over https://github.com/andersfylling/disgord it is more up to date and offers more features. It should be backwards compatible, but probably worth releasing as a breaking change.
It does currently use the master branch and will need to until bwmarrin/discordgo#1497 makes it into a release
Closes #91