Jan Jansen
Jan Jansen
@FlorianHockmann Would you like to review it again?
@porunov Would you like to review this PR?
@FlorianHockmann I've fixed both comments.
I think it would be a good idea to look into the examples. I don't had run them over some period of time.
It would be good if you add the stacktrace
Currently, I don't think to move it into ruma-events, because this repos are used to define structures these could use by client or servers.
I think, if we use Elasticsearch could reduce the complexity of sync massively. It could increase performance.
Could be better to switch `NaiveDateTime` form `chrono`.
Why you want to separate these parts?
We need the same storages for some, than we need to think of spliting storage/models in different repo to integrated, it easily without writing all functions multiple time.