-
Notifications
You must be signed in to change notification settings - Fork 13
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
No documentation for Tournaments by Videogame? #52
Comments
So not every example on their site has a 1-to-1 implementation in the SDK; this is mostly because the use cases vary in size from situation to situation. Since this is an SDK, I intended for it to help developers tackle the small units of work that would come with using their API (ie pulling a tournament then searching the sets for some data are two calls instead of one more complex function). The more broad a use case, the more of a reason I see it belonging in the SDK. If it's specific, then there's less of a reason since it affects a far smaller amount of developers. However that being said, I've done literally nothing on this project for months, and honestly I wouldn't have too much issue adding this in to help you out. |
Hi! Wow, that was a fast reply. |
That's a good point I wasn't considering.. I believe that the functions in the NetworkInterface.js actually allow for a query to be entered by a developer for use in the queuing backend; however there could be an easier way to access this so I'll consider adding a convenience function to the root of the SDK. For now I'll plan on adding your function in soon. I'll keep this open until it's done. |
That's extremely smart! |
So I forgot to update on this. It's taken a while to get the above implemented due to my daily life. I will jump back into this with the other bugs I need to look at. |
(Question, not issue!)
There's a page on the smash.gg developer docs that outlines a Tournaments by Videogame endpoint. I don't see that anywhere on smashgg.js's docs; is that an intentional omission? Or is there something available to get that data?
Thanks! :)
The text was updated successfully, but these errors were encountered: