Command handlers should be optional by default

[commanded]

Command handlers should be optional by default

[Papipo]

What about using pattern matching?

def execute(state, %OpenAccount{} = command) do
[...]
end

This forces commands to be defined. There might be a mismatch between the CamelCase command and the snake_case function name.

[slashdotdash]

@Papipo I've updated the issue to use execute/2 as the default function name.

[slashdotdash]

Fixed by #35.

[brendanzab]

What is the purpose of command handlers if they can just be implemented with pattern matching and exectute/2?

[slashdotdash]

@brendanzab By default you can route commands directly to the aggregate and use pattern matching on its execute function.

I like to keep my aggregate functions pure. Therefore, if I need to pull in external data (e.g. third party system, read model projection) I do this in a command handler. Then pass this additional data to the aggregate.

You don't need to use command handlers if you prefer not.