-
Notifications
You must be signed in to change notification settings - Fork 9
feat: use protocol version interceptor in clients #63
Conversation
clients/nodejs/CorePromiseClient.js
Outdated
* @param {?Object} credentials | ||
* @param {?Object} options | ||
*/ | ||
constructor(hostname, credentials = grpc.credentials.createInsecure(), options = {}) { | ||
constructor(hostname, version, credentials = grpc.credentials.createInsecure(), options = {}) { |
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.
set version via setter
Move protocolVersion from constructor to setter
* @param {?Object} credentials | ||
* @param {?Object} options | ||
*/ | ||
constructor(hostname, credentials = grpc.credentials.createInsecure(), options = {}) { | ||
constructor(hostname, version, credentials = grpc.credentials.createInsecure(), options = {}) { |
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.
Still version here
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.
We should update grpc-common
here after we merge all PRs there.
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "@dashevo/dapi-grpc", | |||
"version": "0.12.1", | |||
"version": "0.13.0-dev.1", |
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.
We need to bump the version in a separate PR (chore) due to conventional changelog flow.
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.
👍
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.
Cool
BREAKING_CHANGE: you have to set up protocol version via
setProtocolVersion
method when using promise clients