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

Use DateTimeStyles.AdjustToUniversal.#24

Merged
bgrainger merged 1 commit intoFaithlife:masterfrom
ejball:master
May 4, 2018
Merged

Use DateTimeStyles.AdjustToUniversal.#24
bgrainger merged 1 commit intoFaithlife:masterfrom
ejball:master

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