Skip to content

v0.3.1: port Zapier review fixes (connection label, 402/404, safer billed defaults) - #4

Merged
ivodraginov-cloud merged 1 commit into
mainfrom
fix/0.3.1-review-fixes
Nov 11, 2026
Merged

ivodraginov-cloud merged 1 commit into
mainfrom
fix/0.3.1-review-fixes

Conversation

@ivodraginov-cloud

Copy link
Copy Markdown
Contributor

Ports the Zapier 0.4.1/0.4.2 review fixes that apply to this Make app, ahead of Make's public review.

  • Connection label: metadata is "Token ending XXXX", with fallbacks "API token" / "API token missing". Make shows the app name beside every connection, so the label never includes "BatchData".
  • HTTP 402 maps to a clear insufficient-funds DataError. HTTP 404 says the ID or path was not found.
  • Safer billed defaults: Search Properties, Get Investor Properties, Find Investors (Buy Box) and Get Property Permits now default Limit to 1, because later modules run once per result. Watch New Properties in a Search Session (Billed) defaults to 5 properties per check.
  • README notes the new defaults. Tests cover the label, error mapping and defaults (78 tests pass).

The shared catalog.json is unchanged. The Zapier "Your Reference ID" drop is already handled here: batch modules pass row requestId through to the API.

…er billed defaults)

- Connection metadata label is "Token ending XXXX", with fallbacks "API token" / "API token missing". Make shows the app name beside the connection, so the label never includes "BatchData".
- HTTP 402 maps to a clear insufficient-funds DataError; 404 says the ID or path was not found.
- Billed searches (Search Properties, Get Investor Properties, Find Investors (Buy Box), Get Property Permits) default Limit to 1, because later modules run once per result.
- Watch New Properties in a Search Session (Billed) defaults to 5 properties per check.
- README notes the new defaults. Tests cover the label, error mapping and defaults.

The shared catalog.json is unchanged. Batch row "Your Reference ID" already reaches the API in this app, so that Zapier fix did not apply.
@ivodraginov-cloud
ivodraginov-cloud merged commit d0c6af6 into main Nov 11, 2026
1 check passed
@ivodraginov-cloud
ivodraginov-cloud deleted the fix/0.3.1-review-fixes branch November 11, 2026 16:53
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