Skip to content

support NaiveDateTime as well - #1

Open
mayel wants to merge 1 commit into
tanguilp:masterfrom
bonfire-networks:pr-naive-datetime
Open

support NaiveDateTime as well#1
mayel wants to merge 1 commit into
tanguilp:masterfrom
bonfire-networks:pr-naive-datetime

Conversation

@mayel

@mayel mayel commented Apr 3, 2023

Copy link
Copy Markdown

thanks for this library! hope this addition is useful to others :)

@tanguilp

Copy link
Copy Markdown
Owner

It's not clear what happens if there are mixed DateTime and NaiveDateTime, and what to do with timezone in such situation.

I believe it might be more relevant to convert from NaiveDateTime to DateTime before using this lib. I don;t think using NaiveDateTime is a good practice in any case (but I understand there might be some legacy dates with no TZ info here and there).

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