nyxx-discord / nyxx

Wrapper around Discord API for Dart

Apache License 2.0

326 stars 49 forks source link

Fix OAuth2 requiring `identify` scope, and add ability to quickly list guilds with `Nyxx.connectRest` #634

Closed MCausc78 closed 7 months ago

MCausc78 commented 7 months ago

Add ability to quickly list guilds with Nyxx.connectRest/Nyxx.connectOAuth2 (fix https://github.com/nyxx-discord/nyxx/issues/630) (UserGuild structure)
Make Nyxx.connectOAuth2 no longer throw if identify scope is missing

I could make OAuth2Manager class and put fetchCurrentOAuth2Information to it, but its seems overkill

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected) (Nyxx.connectOAuth2/Nyxx.connectOAuth2WithOptions no longer throws if there no identify scope)

Checklist:

[x] Ran dart analyze or make analyze and fixed all issues
[x] Ran dart format --set-exit-if-changed -l 160 ./lib or make format and fixed all issues
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works