Skip to content

Feat/price feed v2 - #103

Open
jlealtru wants to merge 3 commits into
mainfrom
feat/price-feed-v2
Open

jlealtru wants to merge 3 commits into
mainfrom
feat/price-feed-v2

Conversation

@jlealtru

Copy link
Copy Markdown
Collaborator

No description provided.

Wraps POST /v2/price_feed/price_feed and POST /v2/price_feed/price_feed_smoothed
as client.price_feed_v2.price_feed and client.price_feed_v2.price_feed_smoothed,
reusing PropertyTypeService so property_type (ALL, SINGLE_FAMILY, NEW_CONSTRUCTION)
is sent in the POST body. ParclLabsService.url is now optional for POST-only
endpoints. Bumps version to 1.19.0.
- POST requests send limit/offset as query params only; offset passed via
  params was previously misrouted into the JSON body and limit duplicated there
- auto-pagination no longer re-applies the caller's initial offset on next pages
- restore ServiceGroup.add_service positional order (url stays nullable and is
  passed explicitly as None for the POST-only v2 services)
- PropertyTypeService / PortfolioSizeService copy the caller's params instead
  of mutating them
- tests: positional registration, offset routing, param immutability, two-page
  pagination through requests.request, first-page-only default; live forced
  pagination (limit=2) and all three property_type values on both v2 routes
- docs: page-size / auto_paginate behavior and API-side property_type validation
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