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

Enhance web service request with additional request processor component #293

Merged
merged 4 commits into from
Apr 26, 2014
Merged

Enhance web service request with additional request processor component #293

merged 4 commits into from
Apr 26, 2014

Conversation

levend
Copy link
Contributor

@levend levend commented Apr 23, 2014

Giving the possibility to include an additional request processor in the dynamic method based web service invocation chain.

…the dynamic method based web service invocation chain.
@nikita-leonov
Copy link
Contributor

Thanks for submitting both code and unit tests. Could you update documentation as well? Thanks.

@yuriy-tolstoguzov
Copy link
Contributor

It seems travis-ci removed ant from test environment so our ant simian does not work.

@yuriy-tolstoguzov
Copy link
Contributor

I found similar issue with running tests: facebookarchive/xctool#342
That's related to recent environment update on travis-ci. Our master branch starts to fail too.

@nikita-leonov Anyway, the code looks good, you should consider to merge it despite of travis warning

@levend
Copy link
Contributor Author

levend commented Apr 23, 2014

@nikita-leonov Added some words about usage into ROADWebService.md file with sample code. Should I do it somewhere else?

@yuriy-tolstoguzov
Copy link
Contributor

nikita-leonov added a commit that referenced this pull request Apr 26, 2014
Enhance web service request with additional request processor component
@nikita-leonov nikita-leonov merged commit 4bf9f47 into epam:master Apr 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants