Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP #6: First working version after some cleanup #8

Closed
wants to merge 6 commits into from
Closed

WIP #6: First working version after some cleanup #8

wants to merge 6 commits into from

Conversation

renatomefi
Copy link
Member

@renatomefi renatomefi commented Sep 8, 2016

What has been done

  • Some major cleanup in the events
  • Added event type as enum and constants in event class

How to test

  • Checkout the code and use the system normally

Todo

  • Migrate the current events to the new one in order to keep production working
  • Remove the Collections which are not used anymore
  • Possibility to filter by event type
  • What to do with the command handler? Keep it or not since this is a synchronous operation. (Talked to a friend and he usually has 2 queues, one is sync and the other isn't, this could be a solution in our case)

Notes

@renatomefi renatomefi added this to the 1.0.0 milestone Sep 8, 2016
@renatomefi renatomefi closed this Jul 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant