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

Discordgo rewrite #129

Merged
merged 7 commits into from
Apr 27, 2024
Merged

Discordgo rewrite #129

merged 7 commits into from
Apr 27, 2024

Conversation

Cyb3r-Jak3
Copy link

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

@Lucky3028
Copy link
Owner

Thanks for your contributions! I’ll check on this weekend. @Cyb3r-Jak3

@Cyb3r-Jak3
Copy link
Author

@Lucky3028 friendly bump on this

@Lucky3028
Copy link
Owner

I'm very sorry for the huge delay in response!!! I will confirm within two weeks. @Cyb3r-Jak3

@Cyb3r-Jak3
Copy link
Author

Cyb3r-Jak3 commented Apr 13, 2024

I am working on account tests for this as well to help make it easier to test.

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
@Lucky3028
Copy link
Owner

@Cyb3r-Jak3
Thank you very much for your patience.
I have confirmed the operation and would like to merge it.
Could you resolve conflicts?

@Cyb3r-Jak3
Copy link
Author

@Lucky3028 Should be all set

@Lucky3028 Lucky3028 enabled auto-merge April 27, 2024 14:19
@Lucky3028
Copy link
Owner

@Cyb3r-Jak3 Thanks!

@Lucky3028 Lucky3028 merged commit eca997a into Lucky3028:main Apr 27, 2024
3 checks passed
@Cyb3r-Jak3 Cyb3r-Jak3 deleted the discordgo-rewrite branch April 27, 2024 15:53
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.

disgord -> discordgo?
2 participants