gautema / CQRSlite

A lightweight framework to help creating CQRS and Eventsourcing applications in C#
Other
1.1k stars 266 forks source link

Snapshot serialization #76

Closed managerger closed 5 years ago

managerger commented 6 years ago

I've faced with a problem, when I can't serialize SnapshotAggregateRoot. Of course, I can get around the issue by writing SurrogateSerializer, but it seems to be a common task to serialize aggregate when you have a deal with snapshoting (I try to implement memento pattern). So, it would be great to mark SnapshotAggregateRoot and AggregateRoot with [Serializable] attribute.

managerger commented 5 years ago

@gautema Why did you close the thread? It is still useful change, isn't it?

gautema commented 5 years ago

There was no activity for over 6 months, it goes outside the common use case and there was no pull request associated so I closed it.

managerger commented 5 years ago

Thank you, sir I'll create PR shortly