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

Client update #46

Merged
merged 2 commits into from
Dec 5, 2016
Merged

Client update #46

merged 2 commits into from
Dec 5, 2016

Conversation

srl295
Copy link
Contributor

@srl295 srl295 commented Dec 3, 2016

Note: branch contains #19 right now. Will rebase.

Update client to match current REST api, exept for /config MT endpoint.

* change getInfo() function
* update docs

Fixes: IBM-Cloud#19
* sequence number
* external services (but NOT the /config endpoints)
* others
@srl295 srl295 added this to the v1.3.0 milestone Dec 3, 2016
@srl295 srl295 self-assigned this Dec 3, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 88.116% when pulling 75be717 on srl295:client-update into 597c1df on IBM-Bluemix:master.

Copy link

@yumaoka yumaoka left a comment

Choose a reason for hiding this comment

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

This change set looks fine.
Do you have an issue for config endpoints?

@srl295
Copy link
Contributor Author

srl295 commented Dec 5, 2016

@yumaoka now I do, #48 is for the /config endpoints

@srl295 srl295 merged commit c492284 into IBM-Cloud:master Dec 5, 2016
@srl295 srl295 deleted the client-update branch December 5, 2016 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants