Skip to content

Format datetime fields with time and timezone information when present - #367

Merged
kedder merged 1 commit into
kedder:masterfrom
Wigny:format-datetime
Oct 28, 2025
Merged

Format datetime fields with time and timezone information when present#367
kedder merged 1 commit into
kedder:masterfrom
Wigny:format-datetime

Conversation

@Wigny

@Wigny Wigny commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Ensures the time and timezone information for datetime fields is included in the OFX when present, accordingly to the spec.

image

It hides time and timezone when they are missing for fields that were previously rendered as dates (e.g., DTPOSTED), but keeps rendering time for fields that were already rendered with it (e.g., DTSERVER) to avoid breaking changes.

@coveralls

coveralls commented Oct 27, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 95.22% (+0.1%) from 95.115%
when pulling b2a36c3 on Wigny:format-datetime
into 64073d5 on kedder:master.

@Wigny
Wigny marked this pull request as draft October 27, 2025 22:22
@Wigny
Wigny marked this pull request as ready for review October 27, 2025 22:57
Comment thread src/ofxstatement/tests/test_ofx.py Outdated
@kedder

kedder commented Oct 28, 2025

Copy link
Copy Markdown
Owner

Looks good, thanks!

@kedder
kedder merged commit 6a27468 into kedder:master Oct 28, 2025
10 checks passed
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.

3 participants