Skip to content

feat: add has_pool typed boolean filter to property search endpoints …#97

Merged
jlealtru merged 3 commits intomainfrom
has-pool-filter
Apr 3, 2026
Merged

feat: add has_pool typed boolean filter to property search endpoints …#97
jlealtru merged 3 commits intomainfrom
has-pool-filter

Conversation

@jlealtru
Copy link
Copy Markdown
Collaborator

@jlealtru jlealtru commented Apr 3, 2026

…(DAT-15)

Adds has_pool as a first-class parameter to both property.search.retrieve (v1) and property_v2.search.retrieve (v2), replacing the need to pass it via the params dict catch-all. Bumps version to 1.17.0.


Note

Low Risk
Low risk: adds an optional boolean query filter (has_pool) to request-building logic and schemas without changing existing defaults or core retrieval flow.

Overview
Adds a first-class has_pool: bool | None filter to both property.search.retrieve (v1) and property_v2.search.retrieve (v2), wiring it through parameter preparation / schema validation so it is sent as the API-expected "true"/"false" value when provided.

Bumps the client version to 1.17.0, updates the changelog, and extends unit tests to cover has_pool behavior in both v1 and v2 filter builders.

Written by Cursor Bugbot for commit 51f1947. Configure here.

jlealtru and others added 3 commits April 3, 2026 15:36
…(DAT-15)

Adds has_pool as a first-class parameter to both property.search.retrieve (v1)
and property_v2.search.retrieve (v2), replacing the need to pass it via the
params dict catch-all. Bumps version to 1.17.0.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jlealtru jlealtru requested a review from zhibindai26 April 3, 2026 19:42
@jlealtru jlealtru merged commit 9c8edb1 into main Apr 3, 2026
1 check passed
@jlealtru jlealtru deleted the has-pool-filter branch April 3, 2026 19:45
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