-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Implement missing API endpoints #6
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Hi, I opened a PR #30 addressing some of these. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Is this issue still open ? If its open, how much api's are left |
Hey, yes the endpoints not ticked are left to grab . |
Thanks, I will work on it if anyone else is not working |
Cool, if you want to implement external-engine endpoints, make sure to check feedback given in #66 before |
Hey, I think these api's are already completed in the code but not ticked here.
Also what is |
The list below shows API endpoints that are currently still missing from berserk (generated by the
check-endpoints.py
script).Feel free to choose any one and implement it.
See https://github.com/lichess-org/berserk/tree/master/berserk/clients for existing endpoint implementations.
New endpoints should have their responses typed. See https://github.com/lichess-org/berserk/blob/master/berserk/clients/opening_explorer.py/#L35 for an example of how to type dictionary responses.
For more info on the details of any specific endpoint, see https://lichess.org/api.
Also see the Get Started contributing section.
/api/tokenDo not implement for the moment Implement/oauth
endpoints #55/oauthDo not implement for the moment Implement/oauth
endpoints #55external-engine
endpoints #66 beforeexternal-engine
endpoints #66 beforeexternal-engine
endpoints #66 beforeThe text was updated successfully, but these errors were encountered: