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

No documentation for Tournaments by Videogame? #52

Open
midblue opened this issue Nov 19, 2019 · 5 comments
Open

No documentation for Tournaments by Videogame? #52

midblue opened this issue Nov 19, 2019 · 5 comments

Comments

@midblue
Copy link

midblue commented Nov 19, 2019

(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! :)

@BrandonCookeDev
Copy link
Owner

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.

@midblue
Copy link
Author

midblue commented Nov 19, 2019

Hi! Wow, that was a fast reply.
First off I just want to say thanks for making such a solid SDK and for being so active in the issues even though you're not developing for it directly at the moment! Please let me know if you're really going to follow through on that offer to add it in, or if I should start trying to figure out my own hacked-together queries that would fall outside of the rate limiting etc offered by your SDK.

@BrandonCookeDev
Copy link
Owner

BrandonCookeDev commented Nov 19, 2019

That's a good point I wasn't considering..
My backend setup to handle their rate limit is quite convenient, and it's probably hard to setup projects mincing my sdk with raw queries.

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.

@midblue
Copy link
Author

midblue commented Nov 19, 2019

That's extremely smart!
I would actually say that letting users add their own queries into the queue (with well-documented formatting for how to make those queries) is actually a much better capability than what I asked for, and would allow for anyone who needs one of these edge cases — like me — to make it happen within your already excellent framework without any additional work from you to add a helper function.
(Emphasis on well-documented! Because to be honest, not knowing how to make a graphql query properly via a POST request is what brought me to your library in the first place.)

@BrandonCookeDev
Copy link
Owner

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.

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

No branches or pull requests

2 participants