ThembisileNGQ / Akkatecture

a cqrs and event sourcing framework for dotnet core using akka.net
https://akkatecture.net
MIT License
524 stars 54 forks source link

Investigate using Nodatime for any date/time functionality #445

Open SeanFarrow opened 3 years ago

SeanFarrow commented 3 years ago

Hi,

Having looked at this project (and really liking it), I noticed that the datetimeoffset type is used when representing the time passed since the Unix epoch.

As someone who is pretty fanatical about correct time representation and testing, I wondered whether you would consider switching to Nodatime. This would give you a type (Instant) that represents the time in UTC and therefore devoid of any timezone information as well as a clock representation that can be either mocked or have a fake representation injected that allows the manipulation of time during tests.

If this would be something you'd consider, I'm happy to put a proof of concept/PR together.

Thanks for the good work, it is really a time saver.

Sean.

MarkusGeigerDev commented 3 years ago

😞 https://github.com/Lutando/Akkatecture/issues/446