-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add wrapper for /partidos #7
Comments
Hey, can I work on this issue? |
@rguzg Sure! Let us know if you need any further info! |
Is there a reason why the imports don't have the .js extension? Because I couldn't get sample.js to work unless I added the extension to the imports |
@rguzg Yeah, primarily the
I need to highlight that the project is still a WIP and that the A next step, to fully support node, would be using a bundler properly configured. Sorry about that, but currently I think you won't be able to run the file directly using |
Yeah, it may be a good idea to also support node. For now, I left the imports as they were before I forked the repository |
Thanks for bring this up, I'll open an issue about it soon. |
🇧🇷 Adicionar wrapper para o endpoint
/partidos
Problem / Problema
🇬🇧 The SDK doesn't support
/partidos
endpoint🇧🇷 O sdk ainda não possui suporte para o endpoint de partidos
/partidos
.Solution / Solução
🇬🇧 Add the methods respecting the patterns, validations and available options for each endpoint:
🇧🇷 Adicione os métodos respeitando os padrões, validações e opções disponíveis para cada endpoint:
/partidos
/partidos/{id}
/partidos/{id}/membros
The text was updated successfully, but these errors were encountered: