-
Notifications
You must be signed in to change notification settings - Fork 104
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
name-service: Migrate to cosmwasm v0.11.0 #97
Conversation
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
73675e2
to
868afda
Compare
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.
…-examples into cw-0.11.0-name-service
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.
} | ||
|
||
const coralnetOptions: Options = { | ||
httpUrl: 'https://lcd.coralnet.cosmwasm.com', |
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.
httpUrl: 'https://lcd.coralnet.cosmwasm.com', | |
httpUrl: 'https://rest.cosmwasm.hub.hackatom.dev', |
|
||
const coralnetOptions: Options = { | ||
httpUrl: 'https://lcd.coralnet.cosmwasm.com', | ||
networkId: 'cosmwasm-coral', |
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.
networkId: 'cosmwasm-coral', | |
networkId: 'hackatom-wasm', |
source: "https://github.com/CosmWasm/cosmwasm-examples/tree/nameservice-0.7.0/nameservice", | ||
builder: "cosmwasm/rust-optimizer:0.10.4" | ||
}; | ||
const sourceUrl = "https://github.com/CosmWasm/cosmwasm-examples/releases/download/nameservice-0.7.0/contract.wasm"; |
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.
const sourceUrl = "https://github.com/CosmWasm/cosmwasm-examples/releases/download/nameservice-0.7.0/contract.wasm"; | |
const sourceUrl = "https://raw.githubusercontent.com/CosmWasm/cosmwasm-examples/master/mask/artifacts/cw_nameservice.wasm"; |
* Usage: npx @cosmjs/cli --init https://raw.githubusercontent.com/CosmWasm/cosmwasm-examples/master/nameservice/helpers.ts | ||
* | ||
* Create a client: | ||
* const client = await useOptions(coralnetOptions).setup(password); |
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.
* const client = await useOptions(coralnetOptions).setup(password); | |
* const client = await useOptions(heldernetOptions).setup(password); |
* await client.getAccount() | ||
* | ||
* Get the mnemonic: | ||
* await useOptions(coralnetOptions).recoverMnemonic(password) |
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.
* await useOptions(coralnetOptions).recoverMnemonic(password) | |
* await useOptions(heldernetOptions).recoverMnemonic(password) |
readonly gasLimits: Partial<GasLimits<CosmWasmFeeTable>> // only set the ones you want to override | ||
} | ||
|
||
const coralnetOptions: 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.
const coralnetOptions: Options = { | |
const heldernetOptions: Options = { |
coralnetOptions.gasPrice, | ||
coralnetOptions.gasLimits, |
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.
coralnetOptions.gasPrice, | |
coralnetOptions.gasLimits, | |
heldernetOptions.gasPrice, | |
heldernetOptions.gasLimits, |
const wallet = await loadOrCreateWallet(coralnetOptions, keyfile, password); | ||
const client = await connect(wallet, coralnetOptions); |
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.
const wallet = await loadOrCreateWallet(coralnetOptions, keyfile, password); | |
const client = await connect(wallet, coralnetOptions); | |
const wallet = await loadOrCreateWallet(heldernetOptions, keyfile, password); | |
const client = await connect(wallet, heldernetOptions); |
|
||
const recoverMnemonic = async (password: string, filename?: string): Promise<string> => { | ||
const keyfile = filename || options.defaultKeyFile; | ||
const wallet = await loadOrCreateWallet(coralnetOptions, keyfile, password); |
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.
const wallet = await loadOrCreateWallet(coralnetOptions, keyfile, password); | |
const wallet = await loadOrCreateWallet(heldernetOptions, keyfile, password); |
Should I go ahead and apply the remaining changes, so we can merge this? As a way to validate it, it could be a good opportunity for me to learn how to deploy it to testnet, and verify it's working. |
@maurolacy please take over this. Orkun is on vacation next week. Can you verifiy the JS scripts work? Here are all the network info for Heldernet: https://gist.github.com/webmaster128/6385a7e1885211d9937ada2688ce34c3 |
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Cw 0.12.0 name service
This includes the upgrade 0.10 -> 0.11 -> 0.12 now in one PR |
Related to #94
Don't merge until cosmwasm/v0.11.0 is released