Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

Use DateTimeStyles.AdjustToUniversal. #24

Merged
merged 1 commit into from
May 4, 2018
Merged

Conversation

ejball
Copy link
Member

@ejball ejball commented May 3, 2018

Using DateTimeStyles.AdjustToUniversal instead of DateTimeStyles.None returns a correct but converted-to-Local time (not Unspecified) if a Utc time was saved (which would have caused a Z to be appended). (Behavior if a Local or Unspecified time was saved is not changed by DateTimeStyles.AdjustToUniversal, i.e. it still returns Unspecified in both cases.)

@ejball ejball requested a review from bgrainger May 3, 2018 15:58
Copy link
Member

@bgrainger bgrainger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would like to get the build failure fixed before merging.

Using DateTimeStyles.AdjustToUniversal instead of DateTimeStyles.None returns a correct but converted-to-Local time (not Unspecified) if a Utc time was saved (which would have caused a Z to be appended). (Behavior if a Local or Unspecified time was saved is not changed by DateTimeStyles.AdjustToUniversal, i.e. it still returns Unspecified in both cases.)
@bgrainger bgrainger merged commit 71892ab into Faithlife:master May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants