flow-cli
flow-cli copied to clipboard
Use Flow Account API for account creation
Closes #597
Description
Replaces current implementation of manual account creation with the Flow Account API stated in https://github.com/Outblock/flow-account-api
For contributor use:
- [x] Targeted PR against
master
branch - [x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
- [x] Code follows the standards mentioned here
- [x] Updated relevant documentation
- [x] Re-reviewed
Files changed
in the Github PR explorer - [x] Added appropriate labels
We shouldn't change APIs just to facilitate an issue with transaction iD. As soon as I finish some work on the playground I will help you solve the transaction ID issue.
Shouldn't we also have an API that can get tx object just by txID though? @sideninja The previous function GetTransactionResult still exists as there are other functions that require it.
@sukantoraymond I don't think we need both. In any way that's a different topic, we can discuss changing the API as part of the https://github.com/onflow/flow-cli/issues/493 but again I don't think we should be doing this now just to make the API work. We need to find the problem there. Until then this should wait.