-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fully implement stakeholder features #3
Conversation
package.json
Outdated
@@ -1,5 +1,5 @@ | |||
{ | |||
"name": "codechain-stakeholder-sdk", |
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.
You shouldn't change this field because -js
postfix is redundant when it comes to the NPM package.
749cb63
to
db57eb4
Compare
f0ffc4e
to
b58ff42
Compare
src/index.ts
Outdated
|
||
export function createDelegateCCSTransaction( | ||
sdk: SDK, | ||
delegator: PlatformAddressValue, |
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.
Doesn't it have to be a delegatee
?
src/index.ts
Outdated
|
||
export function createRequestRevokeTransaction( | ||
sdk: SDK, | ||
delegator: PlatformAddressValue, |
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.
The same here.
b58ff42
to
5521508
Compare
5521508
to
c267c0f
Compare
README.md
Outdated
}); | ||
``` | ||
|
||
### Request to revoke a delegated stake tokens |
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.
Is the article a
necessary? I'm not sure about it
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'll fix them and merge this PR.
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.
LGTM
c267c0f
to
efda55b
Compare
Resolves: #1