-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
1604c24
to
411b348
Compare
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.
Good improvements! Could also say that DU methods returns their values in Promise
s if that is not 100% clear.
README.md
Outdated
const client = new StreamrClient({ | ||
auth: { privateKey }, | ||
}) | ||
|
||
const dataUnion = await client.deployDataUnion() | ||
await client.setAdminFee(0.3, { dataUnion }) | ||
const receipt = await dataUnion.setAdminFee(0.3, { dataUnion }) |
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 second parameter (({ dataUnion })
has been removed
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.
it has in fact been removed ;) #224
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.
or about to be
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.
In #224 you remove option for another method (addMembers
, removeMembers
), I removed setAdminFee
parameter earlier (but forgot to update this README).
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.
Fixed here
Co-authored-by: Matthew <fonty1@users.noreply.github.com>
Updated the DU function docs to reflect the fact that we now have the DataUnion object (examples rewritten)
Also general cleanup. Please add freely to it where you see shortcomings.