-
Notifications
You must be signed in to change notification settings - Fork 143
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.
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
Tokens #273
Merged
Merged
Tokens #273
Changes from 42 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
f9ae535
feat: Add transfer of custom tokens naively
MartinMinkov 980c1d8
Mess around with Parties
MartinMinkov 7ddbaa2
feat: Add token class
MartinMinkov c9ebc35
this.token.transfer successfully changes balances with public key input
MartinMinkov c5a34d5
feat: Brute force custom token transfer
MartinMinkov c87de4c
Change createUnsigned to use optional object parameter for tokens
MartinMinkov 4d44977
Remove old transfer
MartinMinkov 25b06b7
Use Bool() for account creation fee for token accounts
MartinMinkov 74bbd4f
Add basic mint/burn methods
MartinMinkov 432abc1
wip: burn and transfer pass transaction logic
MartinMinkov a94b68b
Set lazy-signature for required parties in token transfers
MartinMinkov f5cb144
Fix spelling in sign function
MartinMinkov 1b4b990
Add token id in Ledger.getAccount
MartinMinkov 0bd67fd
Remove newTokenAccount in favor of using Party.fundNewAccount()
MartinMinkov 9717a51
Change this.token to a function to take optional token id
MartinMinkov 4dde4ff
Add token in getAccount
MartinMinkov b1f5867
Refactor this.token().transfer to this.token().send()
MartinMinkov 753ce86
Use getBalance for fetching balance from local ledger
MartinMinkov b353132
Remove comment code
MartinMinkov c8aa6cc
Add tokenId to getAccount
MartinMinkov 728a132
Refactor createUnsigned to take string instead of Field for caller an…
MartinMinkov 4383060
Add proper typing for snarkyjs binding token functions
MartinMinkov 5a1dbe6
Add tokenSymbol to Party API
MartinMinkov d78353c
Fix getCachedAccount by passing in tokenId
MartinMinkov 66432c4
Fix usage of fetchAccount
MartinMinkov ef8267c
Fix tokenSymbol setter
MartinMinkov 335b551
Scaffold test file for tokens
MartinMinkov ad3e967
Add first section of token tests, refactor existing Party token code
MartinMinkov 6f7191e
Add minting with precondition tests
MartinMinkov f0c2d72
Refactor getAccount and getBalance parameters
MartinMinkov ece0c82
Implement burn and send tests
MartinMinkov 6ff6785
Fix fetchAccount type parameters
MartinMinkov f5178ce
Use init method for zkApp state
MartinMinkov d94b75f
Add support for checked tokenId, convert tokenId type to Field, addit…
MartinMinkov 48c9101
Refactor checked computation check and error messages
MartinMinkov 4802f95
Update bindings
MartinMinkov cc3d690
PR feedback
MartinMinkov 92be6a4
Fix parentTokenId parameter for new Token()
MartinMinkov 3f78945
Add getter for tokenId on Party structure
MartinMinkov 1225452
Update bindings
MartinMinkov fe55f6e
Fix CircuitValue.isConstant
MartinMinkov c359962
Update for isNew
MartinMinkov d870ae1
Fix intg tests
MartinMinkov 39281e7
Merge branch 'main' into test/tokens-playground
MartinMinkov e73a4f6
Use createChildParty instead of createUnsigned
MartinMinkov 903a860
Merge branch 'main' into test/tokens-playground
MartinMinkov 8203973
Add formatted getAccount error
MartinMinkov dc19579
Fix checked and unchecked condition in token constructor
MartinMinkov 4788a9a
Fix GraphQL query parameter
MartinMinkov 997eecd
integrate `deploy()` with qanet
mitschabaude c0035b6
fix integration test
mitschabaude File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
The mina commit used to generate the backends for node and chrome is | ||
97ff72a1d43cda02225d972031cdb0639f0abf84 | ||
2df63cc94756f94a8daaea4cc6659c3ffd51447e |
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
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
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
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
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
Oops, something went wrong.
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.
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.
don't you need a comma between the publicKey and tokenId in the query?
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.
Good catch!