search

russ- / pychallonge

Python bindings for the CHALLONGE! API
http://challonge.com/api
BSD 2-Clause "Simplified" License
49 stars 19 forks source link

Updated base URL to api.challonge.com/v1 #2

Closed corneldm closed 10 years ago

corneldm commented 10 years ago

Requests to the deprecated API route (challonge.com/api) are commonly blocked by Challonge's bot filters. The newer route with the api subdomain bypasses these filters.

corneldm commented 10 years ago

Hey @russ- , will you merge this pull request in when you have a chance? It just switches the base URL from challonge.com/api to api.challonge.com/v1. Bot/spam filtering on our end is much more forgiving on the api subdomain. Thanks!

russ- commented 10 years ago

I don't have a Python dev environment now, so I am trusting you that your change works. ;) Also made you a collaborator on the repo, since I (demonstrably) do not maintain it. I am not sure what dance needs to happen for your fork to be the "primary", but feel free to start that process too, if it makes your life easier.

corneldm commented 10 years ago

Thanks, I appreciate your work on this. We'll be adding oauth and access to more resources in v2 of our API, so I'll likely build onto this project to add compatibility.