{"action":"opened","issue":{"url":"https://api.github.com/repos/gautema/CQRSlite/issues/2","repository_url":"https://api.github.com/repos/gautema/CQRSlite","labels_url":"https://api.github.com/repos/gautema/CQRSlite/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/gautema/CQRSlite/issues/2/comments","events_url":"https://api.github.com/repos/gautema/CQRSlite/issues/2/events","html_url":"https://github.com/gautema/CQRSlite/issues/2","id":153652796,"number":2,"title":"Nice projet, documentation missing","user":{"login":"Bidou44","id":5153523,"avatar_url":"https://avatars.githubusercontent.com/u/5153523?v=3","gravatar_id":"","url":"https://api.github.com/users/Bidou44","html_url":"https://github.com/Bidou44","followers_url":"https://api.github.com/users/Bidou44/followers","following_url":"https://api.github.com/users/Bidou44/following{/other_user}","gists_url":"https://api.github.com/users/Bidou44/gists{/gist_id}","starred_url":"https://api.github.com/users/Bidou44/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bidou44/subscriptions","organizations_url":"https://api.github.com/users/Bidou44/orgs","repos_url":"https://api.github.com/users/Bidou44/repos","events_url":"https://api.github.com/users/Bidou44/events{/privacy}","received_events_url":"https://api.github.com/users/Bidou44/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-05-08T12:24:26Z","updated_at":"2016-05-08T12:24:26Z","closed_at":null,"body":"Your project looks promising.\r\nThe main problem is that it's not documented at all...\r\n\r\nIt would be really good to have a small documentation that explains the core principle, with the event mechanisms for example...  To example you did is really good, but it's not sufficient.\r\n\r\nAlso, you could explain what is the goal of the Session, how one could interchange Autofact with another DI Framework, how one could bind Entity Framework, if it is possible to use the Window Service Bus instead of the InProcessBus class (or any other similar library), what is the snapshot strategy, and probably a lot more.\r\n\r\nConcerning to code, I didn't go into the details, but I saw 2-3 small things:\r\n- Use TimeSpan.FromXXX instead of new TimeSpan(x, x, x....)\r\n- In the LoadAggreate method of the Repository, you could create the Aggregate just before the LoadFromHistory method\r\n- The IEventHandler interface could be contravariant\r\n"}}