Skip to content

AutoMap/Infer TimeSpan properties by default #1705

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
russcam opened this issue Jan 7, 2016 · 1 comment
Closed

AutoMap/Infer TimeSpan properties by default #1705

russcam opened this issue Jan 7, 2016 · 1 comment
Assignees

Comments

@russcam
Copy link
Contributor

russcam commented Jan 7, 2016

Reference to #1643

I think we should consider handling/inferring System.TimeSpan properties by default, storing it as a double representing the TotalMilliseconds of the TimeSpan. Storing it as a string makes less sense as you may want to query on the field and this would be unituitive on a string field.

Doing this would require a custom JsonConverter for TimeSpan.

What are your thoughts, @Mpdreamz, @gmarz?

@russcam russcam added the Vote label Jan 20, 2016
@RobinRieger
Copy link
Contributor

👍

Manually adding attribute for 'double' here atm, because string mapping doesn't make any sense....

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

No branches or pull requests

2 participants