-
Notifications
You must be signed in to change notification settings - Fork 14
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
Base class for handlers #174
Comments
#161 and #162 both tangential. This feature should do one critical thing.
As a side effect it should/probably will make it easier to unit test the logic of a handler. For a follow up we will be able to add yammer to this base class for performance tracking. |
So I coded up a simple base class called AbstractHandler. https://github.com/zznate/intravert-ug/tree/abstract_handler . Please supply some comments. |
This looks fine - I did something similar in for http - see o.u.v.handler.IntraHandlerBase. I think we need to have better naming scheme for http vs. msg bus handlers and push that into the packages. Having good/predictable package and class names will make it easy for people hacking around to get started. |
This is done. We can convert invidual handlers as we touch them |
Handlers need a base class that can deal with runtime exceptions.
The text was updated successfully, but these errors were encountered: