-
Notifications
You must be signed in to change notification settings - Fork 79
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
Documentation for FA processor #546
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
@@ -394,6 +394,7 @@ impl FungibleAssetBalance { | |||
last_transaction_timestamp: txn_timestamp, | |||
token_standard: TokenStandard::V1.to_string(), | |||
}; | |||
// What is this for? |
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.
V1 events don't have asset type. It's just "0x1::coin::WithdrawEvent" (or something like that) w/ the address and amount of APT.
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.
Ok I see you've commented about it in the main file so I'll leave this.
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.
thanks for adding the comments, should we also summarize this in a doc somewhere? We could also add this to dev docs post migration.
Dev docs post migration makes sense to me. |
92bbf80
to
7c21f9f
Compare
Merge activity
|
No description provided.