ObserverManager.Notify calls multiple places uses non-task notification delegate

Cratis / Chronicle

Event sourcing made easy—no complexity, just powerful tools and flexibility for everyone, from beginners to pros.

https://cratis.io

MIT License

31 stars 6 forks source link

ObserverManager.Notify calls multiple places uses non-task notification delegate #1418

Open woksin opened 2 months ago

woksin commented 2 months ago

Notify has an overload for a FUnc<TObserver, Task> norification delegate, why are we not using this? Because we are using the non-async version we have task.Wait() calls hanging around

woksin commented 2 months ago

Fixed by #1421