Skip to content

Conversation

@Wgil
Copy link
Contributor

@Wgil Wgil commented Jul 1, 2025

Adds token listing steps

This will serve as our official documentation for any user to add a token and a pool to the Elys Network.

Open questions

While writing the docs, I created the following questions for us to improve and clarify the process:

  • assetprofile add-entry:
    • commit_enabled and withdraw_enabled: is this used? Otherwise, remove them.
    • permissions: is this used? Otherwise, remove it.
    • address: should we remove and keep authority address as the only address?
    • transfer-limit: is this used? Otherwise, remove it.
  • assetinfo band-ticker or elys-ticker: is this being used?
  • amm create-pool weights: should we remove this option and keep 100:100 as default?
  • amm create-pool fee denom: never specified different than USDC. Should wee keep it as a required parameter?
  • check out pool multipliers section. do we want users to execute that? What/how multiplier would be allowed/suggested?
  • pool_max_leverage_ratio: how would we suggest/handle this value?
  • vault creation: are we suggesting any values? users wont have context on how to set these values.
  • MsgUpdateEnabledPools: should we refactor this so users dont have to send the whole list of enabled pools? Maybe we can have a MsgAddEnabledPool and MsgRemoveEnabledPool or just a toggle?

fenriz07
fenriz07 previously approved these changes Jul 1, 2025
Copy link
Contributor

@fenriz07 fenriz07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


## Reporting the Asset Price

<To be completed>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fenriz07 @avkr003 can you guys please help me fill out this information with the Ojo related instructions?

@Wgil Wgil marked this pull request as ready for review July 2, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants