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

OfficialAPI: add river race and river race log endpoints #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gogaz
Copy link
Contributor

@gogaz gogaz commented Sep 16, 2020

Changes Description

Update war logs URL to match the latest version of Clash Royale
Given that the endpoint data format for river races slightly differs from previous clan wars, I advocate for new methods over modificating get_clan_war* methods.

Type of PR

  • Bug Fix
  • Feature Addition

Checklist

  • Docstrings added (NumpyDoc)
  • If necessary, add to the documentation files
  • Add to the CHANGELOG file
  • Tox checked

@gogaz gogaz force-pushed the patch-2 branch 7 times, most recently from 16217fe to 9eea1d4 Compare September 18, 2020 17:56
@gogaz gogaz changed the title Fix: Update clan war URL to match CR September update OfficialAPI: add river race and river race log endpoints Sep 18, 2020
@gogaz
Copy link
Contributor Author

gogaz commented Sep 19, 2020

@fourjr any chance to have this reviewed any time soon?

@igonro
Copy link

igonro commented Oct 1, 2020

How is this going @fourjr ? I'm interested in using it in my project.

@fourjr
Copy link
Contributor

fourjr commented Oct 4, 2020

Hey, I've been busy over the past week. Will review and push to PyPi by end of next week.

@fourjr
Copy link
Contributor

fourjr commented Oct 17, 2020

Hey, sorry for the late reply, why isn't the get_clan_war method removed? From the api documentation, /currentwar is no longer in use.
image

@gogaz
Copy link
Contributor Author

gogaz commented Nov 1, 2020

@fourjr good catch, I hadn't tested it!
What do you think we should do with get_clan_warlog? It's been showing possibilities to bring it back are being investigated for almost 3 monthes now so it sounds like it's not going to be back again...

@PaKostya
Copy link

What status with this branch?

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.

4 participants