Skip to content

Fix typo in DateTimeType #5430

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

Closed
wants to merge 1 commit into from
Closed

Conversation

frost-nzcr4
Copy link
Contributor

After quick review found this one. Is this a typo or not?

After quick review found this one. Is this a typo or not?
@stof
Copy link
Member

stof commented Sep 4, 2012

no it is not. In case of a input being a string for the model data, it needs to be converted to a DateTime object (as the norm data should be a DateTime). But this DateTime object should still use the model_timezone. Otherwise, you will change the timezone twice: once from the model data to the norm data and another one from the norm data to the view data

@frost-nzcr4 frost-nzcr4 closed this Sep 4, 2012
@travisbot
Copy link

This pull request fails (merged a6e1256 into 41ffd00).

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