Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixbug emptydatetime #113

Closed
wants to merge 7 commits into from
Closed

Fixbug emptydatetime #113

wants to merge 7 commits into from

Conversation

maplye
Copy link
Contributor

@maplye maplye commented Dec 3, 2012

fix set default date to today when date field value is empty

@jfinkels
Copy link
Owner

jfinkels commented Dec 3, 2012

Please only make pull requests which fix a single issue at a time. I will review this change once you have removed the code changes which are irrelevant to the issue described in the issue title.

@maplye
Copy link
Contributor Author

maplye commented Dec 3, 2012

sorry,I do that, because I don't make my fork clearly.

jfinkels added a commit that referenced this pull request Feb 4, 2013
@jfinkels
Copy link
Owner

jfinkels commented Feb 4, 2013

I'm closing this because I have committed code that interprets empty date objects as None. See c857c91.

If you have any other issues, please make a pull request which fixes a single issue at a time.

@jfinkels jfinkels closed this Feb 4, 2013
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.

None yet

2 participants