You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
To anyone reading this, do you think we should add more helper methods to request discord api? Currently people only get an http client and do whatever they want but should we provide functions abstracting away the URL and stuff?
I just feel this project is way too light and would love to make it a bit more useful than this.
I'll leave this open until someone steps in and gives me some feedback
Additionally if someone could give me an idea on how should the interface be exposed? I'd imagine putting it in a subpackage so people can still use the basic provider if they don't need the advanced helpers.
The text was updated successfully, but these errors were encountered:
I think you should have the API calls be done in functions and handle all the errors, then have them return the json parsed for easier use.
also to make it so that you need fewer lines of code when implementing it, have everything be handled on this package side.
I think you should have the API calls be done in functions and handle all the errors, then have them return the json parsed for easier use.
also to make it so that you need fewer lines of code when implementing it, have everything be handled on this package side.
Thank you for the reply, I'll try to design a better interface in a subpackage when I get the time.
To anyone reading this, do you think we should add more helper methods to request discord api? Currently people only get an http client and do whatever they want but should we provide functions abstracting away the URL and stuff?
I just feel this project is way too light and would love to make it a bit more useful than this.
I'll leave this open until someone steps in and gives me some feedback
Additionally if someone could give me an idea on how should the interface be exposed? I'd imagine putting it in a subpackage so people can still use the basic provider if they don't need the advanced helpers.
The text was updated successfully, but these errors were encountered: