search

jensenkd / plex-api

.NET Core SDK for Plex Media Server
MIT License
86 stars 27 forks source link

Added CreatedAt to model MediaContainer #14

Closed JasonLandbridge closed 4 years ago

JasonLandbridge commented 4 years ago

The MediaContainer model was missing the CreatedAt which I have added.

Also, I have noticed that all DateTime object in the models are integers, such as UpdatedAt or CreatedAt. I believe it would be safer to have them each as a long since I have come across some misconfigured servers which return values such as "2247395385730901353".

What are your thoughts on that?

jensenkd commented 4 years ago

Yes, looks good. I agree on the int->long as well.