Skip to content

Add default None to optional fields - #204

Merged
0e4ef622 merged 2 commits into
mainfrom
fix-pydantic-v2-reports
May 14, 2026
Merged

Add default None to optional fields#204
0e4ef622 merged 2 commits into
mainfrom
fix-pydantic-v2-reports

Conversation

@0e4ef622

@0e4ef622 0e4ef622 commented May 13, 2026

Copy link
Copy Markdown
Contributor

for pydantic v2


Note

Medium Risk
Touches many Pydantic response/request models; while changes are largely default-value semantics, they can affect validation/requiredness and serialized output in client integrations.

Overview
Bumps the SDK version to 5.4.0.

Updates multiple Pydantic models (e.g., bank statements, entities, reports, recurring payments, income check, webhooks, recurrence groups) to set explicit = None defaults for Optional[...] fields, aligning model required/optional behavior with Pydantic v2 expectations. A v2 bank statement test fixture ID is also updated.

Reviewed by Cursor Bugbot for commit edb677b. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit ed5a062. Configure here.

Comment thread ntropy_sdk/v2/recurring_payments.py
@0e4ef622
0e4ef622 force-pushed the fix-pydantic-v2-reports branch from ed5a062 to 9b24f81 Compare May 13, 2026 17:48
@0e4ef622
0e4ef622 added this pull request to the merge queue May 14, 2026
Merged via the queue into main with commit 38a9ab5 May 14, 2026
3 checks passed
@0e4ef622
0e4ef622 deleted the fix-pydantic-v2-reports branch May 14, 2026 23:33
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