Skip to content

Migrate to API v2 and add OHLC endpoint#2

Open
oligirling wants to merge 1 commit intomainfrom
feature/v2-api-ohlc-endpoint
Open

Migrate to API v2 and add OHLC endpoint#2
oligirling wants to merge 1 commit intomainfrom
feature/v2-api-ohlc-endpoint

Conversation

@oligirling
Copy link
Contributor

Summary

  • Breaking: Migrate all requests from API v1 to v2 (https://currencyapi.net/api/v2/)
  • Add new ohlc() endpoint with currency, date, base, and interval parameters (Tier 3+)
  • Bump version to 2.0.0

Changes

  • endpoint.pyAPI_VERSION updated from v1 to v2
  • ohlc.py — New OHLC endpoint class with fluent interface
  • currency.py — Added ohlc() factory method
  • tests/endpoints/test_ohlc.py — Full test suite for OHLC endpoint
  • tests/test_currency.py — Added test_ohlc_method
  • README.md — OHLC endpoint docs with parameter table, examples, and sample response
  • CHANGELOG.md[2.0.0] entry documenting breaking changes and new endpoint
  • setup.py — Version bumped to 2.0.0

Test plan

  • All 45 tests pass in Docker (python -m coverage run -m unittest discover)
  • 100% test coverage maintained across all modules

🤖 Generated with Claude Code

- Update base URL from v1 to v2 (https://currencyapi.net/api/v2/)
- Add new OHLC endpoint with currency, date, base, and interval params
- Add full test coverage for OHLC endpoint (100% coverage maintained)
- Bump version to 2.0.0 and update README/CHANGELOG accordingly

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 22175049469

Details

  • 71 of 71 (100.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 21981060819: 0.0%
Covered Lines: 380
Relevant Lines: 380

💛 - Coveralls

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

Comments