search

mycroes / Sally7

C# implementation of Siemens S7 connections with a focus on performance
MIT License
56 stars 22 forks source link

DateTime handling #46

Open remcob-engineero opened 9 months ago

remcob-engineero commented 9 months ago

I'm currently working on a project in which we need to read DTL (DateTimeLong) from the PLC. For now, we've created a small wrapper around Sally7. Would you like me to create a pull request in which I add the conversion from DateTime to DateTimeLong?

mycroes commented 9 months ago

Hi @remcob-engineero, Please do. I assume your preferred solution would be to have DataBlockDataItem<DateTime>, I'm definitely open to add that support.

Keep in mind the current codebase has not yet been released; I actually broke compatibility for DataBlockDataItem to remove the mutability of the configuration.

Also nice to see another user chiming in, especially from the Netherlands! Feel free to open as many issues and PR's as you like. 🚀