Skip to content

Conversation

briangow
Copy link
Contributor

In 63c987f updates to HeaderMixin were made to properly format the header date and time fields. This fix did not make it into MultiHeaderMixin (for multi-segment records). Therefore, writing a date using the wrheader method under MultiHeaderMixin currently results in writing the base_date as YYYY-MM-DD which isn't compatible with the DD/MM/YYYY format requirement.

This PR updated MultiHeaderMixin by using the code from HeaderMixin to properly set formats.

@briangow briangow requested a review from bemoody June 23, 2025 19:42
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.

1 participant