feat: enhance jetton page #403
Conversation
pages/cookbook/jettons.mdx
Outdated
|
|
||
| Calculation of contract's jetton wallet is done via function. | ||
| To obtain jetton wallet state init we need jetton wallets data and code. Common initial data layout is shown below but in may differ in some case, like in . | ||
| Since notifications come from your contract's jetton wallet as shown in diagram, the function should be used in state init the owner address field. |
|
|
||
| init(jettonWalletCode: Cell, jettonMasterAddress: Address) { | ||
| self.jettonWalletCode = jettonWalletCode; | ||
| self.jettonMasterAddress = jettonMasterAddress; | ||
| self.myJettonWalletAddress = calculateJettonWalletAddress(myAddress(), jettonMasterAddress, jettonWalletCode); |
There was a problem hiding this comment.
as I already said before in previous reviews, in most cases the solution for verification is to calculate that address offchain and then just add it into the stateinit of the contract itself.
calculating the address on-chain will only work if you're 100% sure in the jetton code and data structure, or if you take them as variable parameters in the stateinit
| require(sender() == myJettonWalletAddress, "Notification not from your jetton wallet!"); | ||
| require(sender() == self.myJettonWalletAddress, "Notification not from your jetton wallet!"); | ||
|
|
||
| self.myJettonAmount += msg.amount; |
There was a problem hiding this comment.
We should also highlight that these notifications are not guaranteed to be sent. The default implementation doesn't send notification if forwardAmount is zero, so in such cases you can't really rely on them.
|
Hey, should i move all my requests to tact-lang/tact repository? |
|
@a-bahdanau if that's convenient to you. If not, it's ok to continue working on existing ones here. Once they're done, I'll simply port them to the main docs :) |
|
This PR is transferred to tact-lang/tact#944 Massive thanks to @pixelplex and @a-bahdanau for opening and working on this and subsequent PRs in Tact compiler repo, and thanks to @Gusarich for the review! |
Closes #399