search

terra-money / docs

🗂 The official documentation for the Terra blockchain
https://docs.terra.money
Other
124 stars 203 forks source link

fix: cw20 token factory naming, deps and unit test #493

Closed llllllluc closed 2 years ago

llllllluc commented 2 years ago
  1. config.terrain.json is the name generated by terrain, but the cw20 tutorial used terrain.config.json.
  2. cw20_factory_token is named as cw20-factory-token in crates.io, maybe we should change all naming to cw20-factory-token in the tutorial when creating this new contract.
  3. instantiate_contract and contract_address in test.rs are named instantiate and _contract_address in contract.rs. Need to use the same name in contract.rs as in test.rs to pass unit test.
evanorti commented 2 years ago

Thank you for your contribution, @luuuu-learning!!

evanorti commented 2 years ago

@terran6 will verify today and then approve for merging.