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.

Sign up for GitHub

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

Jump to bottom

api: Ticker Historical prices route #5

Merged
merged 4 commits into from
Dec 15, 2023
Merged

api: Ticker Historical prices route #5

merged 4 commits into from
Dec 15, 2023

Conversation

pajicf
Copy link
Owner

@pajicf pajicf commented Dec 15, 2023

What does this PR resolve? 🚀

  • API route for fetching token historical prices

Details 📝

This PR adds a new route to the v1 api: /tickers/:ticker/history which fetches historical prices in a given unix timestamp range.
The data is consumed through Coingecko, formatted and relayed back to the caller.

Checklist ✅

  • I've read Contributions section in the README.md document
  • I have merged the main branch into my branch and resolved all conflicts
  • I have documented the new code
  • I have created automated tests for the new code
  • I have smoke tested the new code locally
  • I have assigned the PR to myself
  • I have named the PR correctly

@pajicf pajicf added the enhancement New feature or request label Dec 15, 2023
@pajicf pajicf self-assigned this Dec 15, 2023
@pajicf pajicf merged commit fdbdd0a into main Dec 15, 2023
2 checks passed
@pajicf pajicf deleted the historical-prices branch December 15, 2023 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pajicf pajicf

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pajicf