Skip to content

v3.0.3 #117

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 29 commits into from
Nov 2, 2020
Merged

v3.0.3 #117

merged 29 commits into from
Nov 2, 2020

Conversation

cgcardona
Copy link
Contributor

@cgcardona cgcardona commented Oct 29, 2020 •
edited
Loading

  • avm.sendMultiple
  • Fix case where passing in a UTXOSet w/ SECPTransferableOutput comes before StakeableLockOutput was causing adding validator to fail
  • Fix info.isBootstrapped typo
  • Add support for the new includeReason property of platformvm.getTxStatus

cgcardona and others added 29 commits September 29, 2020 00:01
@cgcardona
Add optional memo param to avm.send.
cc9f80c
@cgcardona
* Add memo field to avm.send
3add41b 
* Add `changeAddr` field to `avm.send`
* Add `avm.sendMultiple`
@cgcardona
update comment.
71e0b7b
@cgcardona
1. Update memo param comment
a78e6ed 
2. Add Oxford comma
3. Validate changeAddr
4. Pass back asset instead of assetID.
@cgcardona
Update all memo comments.
51a757a
@cgcardona
Fix return type/values.
dd42935
@cgcardona
Fix broken tests.
c346036
@cgcardona
Memo can be string or buffer.
f1985a7
@cgcardona
Test for buffer case.
4a90e92
@cgcardona
Handle changeAddr and memo being passed in as undefined.
afa314d
@cgcardona
Update comments
02fcb37
@cgcardona
Fix comment.
b626ef6
fixing up the code
4cd8bb6
@collincusce
Merge pull request #109 from ava-labs/send_multiple
63af322 
* Add optional `memo` parameter to `avm.send`
* Add optional `changeAddr` parameter to `avm.send`
* Made `from` optional in `avm.send`
* Add `avm.sendMultiple`

The calls `avm.send` and `avm.sendMultiple` should both match the RPCs defined in the docs: https://docs.avax.network/v1.0/en/api/avm/#avmsend
@collincusce
backing up changes to dev for testing
2a29f59
@collincusce
fixes transaction parsing issues
6698083
@collincusce
testing pat
a6658a0
@collincusce
possible fix for addValidatorTx
e5cdc24
@collincusce
AddValidatorTx fixed
fed50f0
@collincusce
back to normal
d41d928
@cgcardona
Add support for the new includeReason property of platformvm.getTxSta…
a8db258 
…tus.
@cgcardona
Pushing bundle.
62c09c8
@cgcardona
3.0.2
f8edc9f
@cgcardona
Merge branch 'master' into development
1e29786
@sponnet
fixed api endpoint typo
96e327b
@cgcardona
Merge pull request #119 from sponnet/master
144ba09 
fixed info api endpoint typo for isBootstrapped
@cgcardona
Merge branch 'development' into add-support-for-include-reason
76bfc7d
@cgcardona
Add details for passed back object
2b00e60
@cgcardona
Merge pull request #114 from ava-labs/add-support-for-include-reason
aa0e948 
Add support for the new `includeReason` property of `platformvm.getTxStatus`.
@cgcardona cgcardona merged commit cf76708 into master Nov 2, 2020
dfenstermaker pushed a commit that referenced this pull request Feb 22, 2023
@dhrubabasu
add string utils to id type (#117)
83dcdd8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cgcardona @collincusce @sponnet