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

Option to force DateTimes to deserialize as Utc #28

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

michaelbartnett
Copy link
Contributor

Things got a little messy in the last version of this PR (#20), so it seemed easier to just rebase from upstream/master and try again.

Here's the notes copied from there:

The default DateTime deserialization will set the DateTimeKind to Local if there's timezone information in the string. This makes sense, but timezones are a >PITA to manage, and being able to guarantee everything comes out as UTC is pretty important (to me, at least).

Adds JsonMapperOptions flags enum, and a static Options member in the JsonMapper class.

The behavior is enabled by a global flag which is a little gross, so maybe some sort of JsonMapper context could be useful in the future.

Conflicts:
	test/JsonMapperTest.cs
@devlead devlead changed the base branch from master to develop December 23, 2017 23:43
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

1 participant