-
Notifications
You must be signed in to change notification settings - Fork 27
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
Update keys api #252
Update keys api #252
Conversation
5f50a3a
to
6cff92e
Compare
…search-swift into update_keys_api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, @bidoubiwa I left some comments, let me know what do you think about them!
Awesome work! 🎉 🌮
345f199
to
b3d7793
Compare
…search-swift into update_keys_api
…swift into update_keys_api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we are good to go! 🎉
Let's get this merged!
6a4dcea
to
026ab54
Compare
bors merge |
Build succeeded: |
Keys API changes!
Breaking
client.getkeys
now returns aResults<Key>
object see specificationNew
client.getKey
returns aKey
object of a specific keyclient.createKey
returns aKey
object created with aKeyParams
objectclient.updateKey
returns aKey
object with the updated keyclient.deleteKey
returns void as the chosen key has been deleted