feat: implement check sum reporting and update default target #210
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR implements couple of things:
Checksum support
transfer
'sto
field), we query the provider to see if this account actually belongs to an user (i.e it is not a blob id, contract id etc).I needed to add an unwrap, at a place where I am sure that the result is not going to be failing. After this release I will remove it by rewriting the transfer section of the wallet.
We first check if the checksum is valid while parsing the
To
field. If that is not the case that is an immediate error. If it is indeed with the correct checksum then we proceed to querying the provider to see if it is an actual address.Couple of examples
Here I try to transfer some funds to a contract, which has invalid
Here i try to transfer to my account with valid check sum.