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.

Sign up for GitHub

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

Jump to bottom

Demonstrating the use of an Interface for this #1

Open

choult wants to merge 1 commit into adear11:JOINDIN-475 from choult:JOINDIN-475

choult commented Nov 21, 2014

Check it out @akrabat!

Encapsulating like this allows us to:

Cut down on method parameters
Extend in the future without changing the method signature
Ensure we're not passing Request to the SpamCheck service as a concept

(I hasten to add I've not tested this; it's more of a demonstration. Feel free to ignore, @adear11 )


          JOINDIN-475 - demonstrating the use of an Interface to pass client de…

e5359b5

…tails to the SpamCheckService

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet