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

Update endpoint paths #9

Merged
merged 2 commits into from
May 12, 2023
Merged

Conversation

trevorbayless
Copy link
Member

@trevorbayless trevorbayless commented May 11, 2023

Updates Games.export_multi, Studies.export_chapter, and Studies.export to prepend api/ to their paths. Interestingly enough, they worked as it was before but utilizing the path documented in the API will allow for the check-endpoints script to pick them up as implemented.

Related to #6

@trevorbayless trevorbayless changed the title Update endpoint path for export_multi Update endpoint paths May 12, 2023
@kraktus
Copy link
Member

kraktus commented May 12, 2023

We changed it more or less recently as it allows more usage due to CORS rules under the api/ endpoints.
Thanks!

@kraktus kraktus merged commit 91a0e36 into lichess-org:master May 12, 2023
@trevorbayless trevorbayless deleted the multi_game_export branch May 12, 2023 13:32
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

Successfully merging this pull request may close these issues.

2 participants