-
Notifications
You must be signed in to change notification settings - Fork 118
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
temp - add wallet integration guide #732
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
import { Aptos, AptosConfig, Network } from "@aptos-labs/ts-sdk"; | ||
const aptos = new Aptos(new AptosConfig(Network.Devnet)); | ||
const sender = Account.fromPrivateKey("private key"); | ||
const rawTxn = await aptos.transferDigitalAssetTransactions({sender, digitalAssetAddress: "0x1213456", recipient: "0x111111111"}); // List of tokens owned by 0x1 |
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.
Two issues with this line:
- The function name should be
transferDigitalAssetTransaction
(singular, not plural) - The trailing comment appears to be copied from a different code example - this function transfers a digital asset rather than listing tokens
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
Digital Asset transfers are simple. In order to transfer a digital asset, you need to know the address | ||
of the asset (also known as the `token_id`), and the address of the receiver. | ||
|
||
To transfer a digital, call `0x1::object::transfer` with the `token_id` and the receiver address. |
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.
Missing the word "asset" in this sentence - should read "To transfer a digital asset
, call..."
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
This issue is stale because it has been open 45 days with no activity. Remove the |
Description
Checklist
pnpm fmt
?pnpm lint
?