feat: Token Airdrop Transaction #2492
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ivaylonikolov7
changed the title
0xivanov
reviewed
Aug 30, 2024
ivaylonikolov7
changed the title
0xivanov
reviewed
Aug 30, 2024
0xivanov
reviewed
Aug 30, 2024
0xivanov
reviewed
Sep 2, 2024
0xivanov
reviewed
Sep 2, 2024
This was
linked to
issues
Sep 3, 2024
0xivanov
reviewed
Sep 4, 2024
agadzhalov
requested changes
Sep 9, 2024
ivaylonikolov7
dismissed stale reviews from 0xivanov and ivaylogarnev-limechain
via
dda20d3
agadzhalov
previously approved these changes
Sep 10, 2024
ivaylonikolov7
dismissed stale reviews from 0xivanov and agadzhalov
via
7a36f44
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
* feat: add airdrop claim and cancel transactions Signed-off-by: Ivaylo Nikolov <[email protected]> * chore: update protobufs Signed-off-by: Ivaylo Nikolov <[email protected]> * feat: update commit messages Signed-off-by: Ivaylo Nikolov <[email protected]> * test: add unit tests for cancel and claim Signed-off-by: Ivaylo Nikolov <[email protected]> * wip(test): add integration tests for cancel and claim transactions Signed-off-by: Ivaylo Nikolov <[email protected]> * fix: claim used the wrong channel function Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: rename transaction name Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: remove claim references in airdropcancel integration test Signed-off-by: Ivaylo Nikolov <[email protected]> * test(fix): fix not working test for airdrop cancel and claim Signed-off-by: Ivaylo Nikolov <[email protected]> * fix: airdropcancel transaction should work on sender not receiver Signed-off-by: Ivaylo Nikolov <[email protected]> * feat: add token airdrop example Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: rename transactions to have the same name as java Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: remove redundant imports and empty lines Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: rename transaction tests to have the same name as java sdk Signed-off-by: Ivaylo Nikolov <[email protected]> * docs: add licenses Signed-off-by: Ivaylo Nikolov <[email protected]> * fix: remove dead code from token cancel Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: consistency in constructor Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: change airdrop_supply_per_person naming Signed-off-by: Ivaylo Nikolov <[email protected]> * test: fix setting sender id Signed-off-by: Ivaylo Nikolov <[email protected]> * refactor: specify what kind of tokens are airdropped Signed-off-by: Ivaylo Nikolov <[email protected]> --------- Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
ivaylonikolov7
force-pushed
the
feat/tokenairdroptransaction
branch
from
27cada3
to
0e6079f
Compare
|
Merged
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description:
HIP 904 - Token Airdrop Transaction
This PR introduces AirdropTokenTransaction transaction. Adds AccountAmount and AirdropTokenTransfer helper classes.
Related issue(s):
Fixes #2465
Notes for reviewer:
Needs to have integration tests to be ready.
Checklist