erazerbrecht / erazercqrs Goto Github PK

The current 'Service Layer' is not a 'Service Layer'.

This layer needs to be renamed before creating confusion!

Problem

Ticket.cs needed a public parameterless constructor before it's working with 'Marten'.

public async Task<T> Get(Guid aggregateId)
{
    using (var session = _store.OpenSession())
    {
        return await session.Events.AggregateStreamAsync<T>(aggregateId);
    }
}

Loading of the events requires that the generic has a public parameterless constructor.

Why

The consumer (MVC, WebAPI) shoudn't be able to create a ticket without parameters.
This ctor doesn't fire an event! If the consumer would use this ctor the event store wouldn't contain a created event!

The implementation is a little bit 'hacked' together.

Current situation uses a NodeJS api to actually do the websocket communication with SocketIO.
I did this because SignalR isn't ready yet for dotnet core. (They have a repo but it lacks documentation and I didn't want to rewrite everything in the near future).

Normally it will be finished with ASPNET CORE 2.1 (sheduled for Q4 2017)

Namespace and folder name are not in sync!

server.js and server.js.map need to be removed from source control

Problem

Marten requires having a public Apply method for every event that is needed to load the aggregate from it's previous events.

Documentation (link)

Why

Leaving this method public makes it possible to consume this methods in the controllers.
If you do this you can change the 'state' of the ticket without firing an event!