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

support for top volume on last 24h endpoint #34

Merged
merged 1 commit into from
Dec 14, 2018

Conversation

monsieurleberre
Copy link
Contributor

↪️ Pull Request

Add support for the endpoint at https://min-api.cryptocompare.com/data/top/totalvolfull

💻 Examples

Just one more endpoint supported, nothing else :)

🚨 Test instructions

Debug the integration test provided to check that it works

💥 Does this PR introduce a breaking change?

  • No

✔️ PR Todo

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@joancaron joancaron merged commit 1738e82 into joancaron:master Dec 14, 2018
@joancaron
Copy link
Owner

Hello @monsieurleberre
Thank you again for this contribution !

@monsieurleberre monsieurleberre deleted the topby24hvolumefull branch December 14, 2018 11:13
@monsieurleberre
Copy link
Contributor Author

monsieurleberre commented Dec 14, 2018

My pleasure :) 👍
@joancaron do you think you could publish an updated nuget package ? Thanks a lot!

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