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

When SOAP Body is returned blank <soap:body/>, there were failures in… #891

Merged
merged 2 commits into from
Oct 19, 2016

Conversation

vidishdatta
Copy link
Contributor

When SOAP Body is returned blank soap:body/, there were failures in processing it.

@coveralls
Copy link

coveralls commented Oct 18, 2016

Coverage Status

Coverage increased (+0.007%) to 93.13% when pulling 95769fd on vidishdatta:add-check-for-body into 7841b64 on vpulim:master.

@vidishdatta
Copy link
Contributor Author

Hi @jsdevel , @herom - Requesting your review. Please let me know if I can provide any further info.

Also, how do we usually publish this back to NPM and at what intervals? The current version on NPM is 16.0

Thanks!

@jsdevel
Copy link
Collaborator

jsdevel commented Oct 18, 2016

Looks good @vidishdatta. One minor comment then I think we're good to go.

});
}, baseUrl);
});

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove this empty line.

@jsdevel jsdevel merged commit 1d576ea into vpulim:master Oct 19, 2016
@jsdevel
Copy link
Collaborator

jsdevel commented Oct 19, 2016

Thanks @vidishdatta !

@coveralls
Copy link

coveralls commented Oct 19, 2016

Coverage Status

Coverage increased (+0.007%) to 93.13% when pulling f5c4ef8 on vidishdatta:add-check-for-body into 7841b64 on vpulim:master.

@vidishdatta vidishdatta deleted the add-check-for-body branch October 19, 2016 00:15
@vidishdatta
Copy link
Contributor Author

Hi @jsdevel , Thanks for accepting changes. This is my first contribution to open source.

Just wanted to understand - When do we usually release it to NPM registry and when do we usually create a release for this repository?

Thanks Again!

@jsdevel
Copy link
Collaborator

jsdevel commented Oct 19, 2016

We usually get to releases every 2 weeks or so.

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.

3 participants