-
Notifications
You must be signed in to change notification settings - Fork 301
feat(target_chains/ton): add ton deploy script #2039
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
LGTM but I'm confused about similar codes interacting with the contract getting merged.
* init ton js sdk * fix docs * fix * address comments
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.
LGTM.
I am wondering whether having a "test" contract that exposes the same methods has any value at the moment (because you have more complexity on the TS side with BaseWrapper and two extensions of it) but I don't have full clarity to make a comment about it.
deployed to testnet at
EQDwGkJmcj7MMmWAHmhldnY-lAKI6hcTQ2tAEcapmwCnztQUwill create sdk for ton in a following PR and will update docs for ton testnet address