-
Notifications
You must be signed in to change notification settings - Fork 32
Allow for custom transaction signers to be provided #155
Conversation
🦋 Changeset detectedLatest commit: e3d32f7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
790f3e4 to
5193e69
Compare
3d8e24c to
e3d32f7
Compare
gregsantos
left a comment
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 shall be approved
|
Thanks for the super fast turnaround! Unfortunately, there seems to be some confusion around the cryptographic terminology:
That is, what this PR has called |
Apologies, this was my mistake and I will try to make a revised PR ASAP. Thank you for the clarification. |
|
Thanks @jribbink — any chance you could also do an |
Will do - the package has been moved from https://www.npmjs.com/package/flow-js-testing to https://www.npmjs.com/package/@onflow/flow-js-testing, so the published package will appear in this new registry. |
|
Ah, didn't catch the move — thanks @jribbink! |
Closes #154
Description
For contributor use:
masterbranchFiles changedin the Github PR explorer