-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat(stellar): its example scripts #490
base: main
Are you sure you want to change the base?
Conversation
stellar/its-example.js
Outdated
minter, | ||
); | ||
|
||
await broadcast(operation, wallet, chain, 'Deploy Token', options); |
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.
We should change the behaviour to not authorize all invocations by default, and authorize it explicitly.
stellar/its-example.js
Outdated
.command('deploy-token <symbol> <name> <decimals> <salt> <initialSupply> ') | ||
.action((symbol, name, decimal, salt, initialSupply, options) => { | ||
mainProcessor(deployToken, [symbol, name, decimal, salt, initialSupply], options); | ||
}); |
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.
also add canonical registration and deployment?
You should do an e2e test for that as well. To register and deploy XLM to Avalanche.
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.
I'm facing an authorization issue with deployRemoteCanonicalToken. Work in progress.
726eb06
to
6c65f8a
Compare
#### Execute | ||
|
||
```bash | ||
node stellar/its.js execute [source-chain] [message-id] [source-address] [payload] |
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.
node stellar/its.js execute [source-chain] [message-id] [source-address] [payload] | |
node stellar/gateway.js execute [source-chain] [message-id] [source-address] [payload] |
Move this under gateway section
@@ -77,6 +78,9 @@ async function buildTransaction(operation, server, wallet, networkType, options | |||
const prepareTransaction = async (operation, server, wallet, networkType, options = {}) => { | |||
const builtTransaction = await buildTransaction(operation, server, wallet, networkType, options); | |||
|
|||
// ahram test |
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.
debug code
async function setTrustedChain(wallet, _, chain, contractConfig, arg, options) { | ||
const contract = new Contract(contractConfig.address); | ||
async function setTrustedChain(wallet, _, chain, arg, options) { | ||
const contract = new Contract(chain.contracts.interchain_token_service?.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.
provide contract
as an arg to reduce duplication
const itsAddressScVal = nativeToScVal(Address.fromString(itsAddress), { type: 'address' }); | ||
const axlearScVal = nativeToScVal('axelar', { type: 'string' }); | ||
const axelarAddressScVal = nativeToScVal('axelar13ehuhysn5mqjeaheeuew2gjs785f6k7jm8vfsqg3jhtpkwppcmzqtedxty', { type: 'string' }); | ||
const payloadHex = |
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.
debug payload?
await broadcast(operation, wallet, chain, 'Remote Canonical Token Deployed', options); | ||
} | ||
|
||
// async function deployRemoteCanonicalTokenOrg(wallet, _, chain, args, options) { |
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.
remove
AXE-7043