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

Fix for Self issue37 #39

Merged
merged 6 commits into from
Oct 31, 2017
Merged

Fix for Self issue37 #39

merged 6 commits into from
Oct 31, 2017

Conversation

takaomoriyama
Copy link
Contributor

@takaomoriyama takaomoriyama commented Oct 20, 2017

Fix for solving issues discussed in Issue #37.
Should be tested with PR watson-intu/cpp-sdk#3.

Signed-off-by: Takao Moriyama moriyama@jp.ibm.com

@takaomoriyama
Copy link
Contributor Author

@rlyle I found an issue with the latest code. So please hold on for a while.

@takaomoriyama
Copy link
Contributor Author

takaomoriyama commented Oct 27, 2017

I think I've resolved Issue #41 with the latest commit 2076a04.
Someone, please review again.

@takaomoriyama takaomoriyama merged commit eaaa741 into watson-intu:develop Oct 31, 2017
@takaomoriyama takaomoriyama changed the title Self issue37 Fix for Self issue37 Oct 31, 2017
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.

None yet

2 participants