Skip to content

Conversation

@ryangallen
Copy link

Let me know if these tests and fixes aren't appropriate.

@lovelydinosaur lovelydinosaur added this to the 3.1.2 Release milestone May 1, 2015
lovelydinosaur added a commit that referenced this pull request May 1, 2015
DateField to_representation can handle str and empty values. Fixes #2656, #2687.
@lovelydinosaur lovelydinosaur merged commit abcd6f2 into encode:master May 1, 2015
@lovelydinosaur
Copy link
Contributor

Nice work!

@ryangallen ryangallen deleted the datefield branch May 3, 2015 17:25
@mdentremont
Copy link
Contributor

@tomchristie Do you know if a similar change should be made for the DateTimeField and the TimeField? I started looking into this but quickly ran into some trickyness with timezones in relation to strptime.

@lovelydinosaur
Copy link
Contributor

@mdentremont Don't know. May be worth looking at.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants