Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Consider using NotificationFactory and DocumentEventFactory a bit differently #43

Open
alechenninger opened this issue Apr 22, 2016 · 1 comment

Comments

@alechenninger
Copy link
Contributor

Right now they essentially just parse out a single serialized entity to a java implementation with business logic, and it is hard coded that this happens per entity name or canonical type. This is a natural pattern, however it would be more flexible to just pass a factory and let this decide how it gets you a notification or document event back, which allows for more parsing options other than just by entity name. This is occasionally useful, and either way probably results in cleaner code.

@alechenninger
Copy link
Contributor Author

Also, MessageFactory current works as is being suggested here

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant