Skip to content
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

[auto-fix] interface type updated for Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposal #249

Draft
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

gurbaj5124871
Copy link
Member

This is an automated generated pr
changelog

  • auto-fix: interface type updated for Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposal

Block Data
network: osmosis-1
height: 18596877

errors

[
  {
    "path": "$input.transactions[7].messages[0].data.content[\"@type\"]",
    "expected": "\"/ibc.core.client.v1.ClientUpdateProposal\"",
    "value": "/cosmos.params.v1beta1.ParameterChangeProposal"
  },
  {
    "path": "$input.transactions[7].messages[0].data.content.subjectClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[7].messages[0].data.content.substituteClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[7].messages[0].data.content.changes",
    "expected": "undefined",
    "value": [
      {
        "subspace": "poolmanager",
        "key": "AuthorizedQuoteDenoms",
        "value": "[\"ibc/498A0751C798A0D9A389AA3691123DADA57DAA4FE165D5C75894505B876BA6E4\",\"factory/osmo1em6xs47hd82806f5cxgyufguxrrc7l0aqx7nzzptjuqgswczk8csavdxek/alloyed/allUSDT\",\"ibc/0CD3A0285E1341859B5E86B6AB7682F023D03E97607CCC1DC95706411D866DF7\",\"factory/osmo1z6r6qdknhgsc0zeracktgpcxf43j6sekq07nw8sxduc9lg0qjjlqfu25e3/alloyed/allBTC\",\"ibc/EA1D43981D5C9A1C4AAEA9C23BB1D4FA126BA9BC7020A25E0AE4AA841EA25DC5\",\"ibc/27394FB092D2ECCD56123C74F36E4C1F926001CEADA9CA97EA622B25F41E5EB2\",\"ibc/D79E7D83AB399BFFF93433E54FAA480C191248FC556924A2A8351AE2638B3877\",\"uosmo\"]"
      }
    ]
  }
]

Copy link
Member Author

Choose a reason for hiding this comment

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

This is an automated generated comment
Block Data
network: osmosis-1
height: 18637676
data:

[
  {
    "content": {
      "@type": "/cosmos.params.v1beta1.ParameterChangeProposal",
      "title": "Alloyed ETH: Quote asset",
      "description": "This proposal would cause allETH to replace ETH.eth.axl as the Quote asset used for Supercharged pool creation. \n\n**Forum Thread**: [https://forum.osmosis.zone/t/alloyed-asset-eth/3002](https://forum.osmosis.zone/t/alloyed-asset-eth/3002)",
      "changes": [
        {
          "subspace": "poolmanager",
          "key": "AuthorizedQuoteDenoms",
          "value": "[\"ibc/498A0751C798A0D9A389AA3691123DADA57DAA4FE165D5C75894505B876BA6E4\",\"factory/osmo1em6xs47hd82806f5cxgyufguxrrc7l0aqx7nzzptjuqgswczk8csavdxek/alloyed/allUSDT\",\"ibc/0CD3A0285E1341859B5E86B6AB7682F023D03E97607CCC1DC95706411D866DF7\",\"factory/osmo1z6r6qdknhgsc0zeracktgpcxf43j6sekq07nw8sxduc9lg0qjjlqfu25e3/alloyed/allBTC\",\"factory/osmo1k6c8jln7ejuqwtqmay3yvzrg3kueaczl96pk067ldg8u835w0yhsw27twm/alloyed/allETH\",\"ibc/27394FB092D2ECCD56123C74F36E4C1F926001CEADA9CA97EA622B25F41E5EB2\",\"ibc/D79E7D83AB399BFFF93433E54FAA480C191248FC556924A2A8351AE2638B3877\",\"uosmo\"]"
        }
      ]
    },
    "initialDeposit": [
      {
        "denom": "uosmo",
        "amount": "1600000000"
      }
    ],
    "proposer": "osmo19w2t4ue7qpdh6022m3yxmxvv3w7jla7u3hfq0r"
  }
]

errors:

[
  {
    "path": "$input.transactions[4].messages[0].data.content[\"@type\"]",
    "expected": "\"/ibc.core.client.v1.ClientUpdateProposal\"",
    "value": "/cosmos.params.v1beta1.ParameterChangeProposal"
  },
  {
    "path": "$input.transactions[4].messages[0].data.content.subjectClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[4].messages[0].data.content.substituteClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[4].messages[0].data.content.changes",
    "expected": "undefined",
    "value": [
      {
        "subspace": "poolmanager",
        "key": "AuthorizedQuoteDenoms",
        "value": "[\"ibc/498A0751C798A0D9A389AA3691123DADA57DAA4FE165D5C75894505B876BA6E4\",\"factory/osmo1em6xs47hd82806f5cxgyufguxrrc7l0aqx7nzzptjuqgswczk8csavdxek/alloyed/allUSDT\",\"ibc/0CD3A0285E1341859B5E86B6AB7682F023D03E97607CCC1DC95706411D866DF7\",\"factory/osmo1z6r6qdknhgsc0zeracktgpcxf43j6sekq07nw8sxduc9lg0qjjlqfu25e3/alloyed/allBTC\",\"factory/osmo1k6c8jln7ejuqwtqmay3yvzrg3kueaczl96pk067ldg8u835w0yhsw27twm/alloyed/allETH\",\"ibc/27394FB092D2ECCD56123C74F36E4C1F926001CEADA9CA97EA622B25F41E5EB2\",\"ibc/D79E7D83AB399BFFF93433E54FAA480C191248FC556924A2A8351AE2638B3877\",\"uosmo\"]"
      }
    ]
  }
]

interface:

export interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposal {
    type: string;
    data: Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalData;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalData {
    content: Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalContent;
    initialDeposit: Osmosis1TrxMsgCosmosGovV1Beta1MsgSubmitProposalInitialDepositItem[];
    proposer: string;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalContent {
    '@type': string;
    title: string;
    description: string;
    changes: Osmosis1TrxMsgCosmosGovV1Beta1MsgSubmitProposalChangesItem[];
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalChangesItem {
    subspace: string;
    key: string;
    value: string;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalInitialDepositItem {
    denom: string;
    amount: string;
}

Copy link
Member Author

Choose a reason for hiding this comment

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

This is an automated generated comment
Block Data
network: osmosis-1
height: 19100515
data:

[
  {
    "content": {
      "@type": "/cosmos.params.v1beta1.ParameterChangeProposal",
      "title": "Increase scope of Protocol Fee Controller subDAO and endorse 0.2% ATOM/OSMO protocol fee",
      "description": "This proposal would grant increased authority to the Protocol Fee Controller subDAO to adjust Protocol Fees on pairings beyond their initial scope. \n\nOne proposed use of this will be to create a discrepancy in protocol fee between OSMO routed tokens and Stable routed tokens to encourage routing via Stable tokens to be more widely used. This proposal endorses a Protocol Fee of 0.2% on ATOM/OSMO as the current major OSMO liquidity pairing as an initial endorsement of this methodology. \n\nThis proposal also adds the Governance module address to the list of Admin addresses for the protocol fee settings so that fee changes may be done directly by governance if required in the future. \n# Protocol Fee Controller subDAO \nThe Protocol Fee Controller subDAO has elevated permissions to adjust the protocol fee between two denominations onchain which were granted in [Proposal 629](https://daodao.zone/dao/osmosis/proposals/629) and can be viewed [here](https://daodao.zone/dao/osmo162wk8qc3w5s9hfs8dm76wrqnk6fjmsez2t4kk6zyugmrlzgds8sqfesmlm/home) \n\nMembership consists of a 4/7 multisig of Validators and Community members. \n\n# Current Scope \nThe Protocol Fee Controller subDAO's initial scope was to adjust the protocol fees on specific asset pairings to ensure that they are reasonable compared to the typical spread factors used in their pools. \n\nThis scope is currently set to deciding on a protocol fee of the default set by governance, an alternative lower protocol fee of 20% of the default, or zero. \n\nTypical settings have been to use: \n\n* 20% of the default (0.02%) for Like pairings (E.g, LST/Unstaked equivalent or Alternative Stable tokens pairings, which typically have 0.01% or 0.05% spread factors. \n\n* 0% for assets that are identical beyond bridging differentiation. E.g, USDT.axl/USDT.kava or WBTC/allBTC, which typically have 0% or 0.01% spread factors. \n\n* Default for all other pairings. \n\n# Proposed Scope \nThis proposal expands the scope of the Protocol Fee Controller to approve the setting protocol fees on pairings at a greater selection of settings compared to the default protocol fee. \n\n|Tier|Current Equivalent|Example Usage| \n|--- | --- | --- | \n|0%|0%|Composable pairings/Fee promotions| \n|20%|0.02%|Like pairings/Fee promotions| \n|50%|0.05%|Fee promotions| \n|100%|0.1%|Default| \n|200%|0.2%|Routing Influence| \n|500%|0.5%|Routing Influence/High volume/fee pairings| \n\nThis removes the reliance on the existing typical spread factor for a route as well as allowing more flexibility and variety in the protocol fee levels set on specific routes. \n\nThe Protocol Fee Controller must not use these abilities to effectively change the default protocol fee for trading on Osmosis but may change it to impact subsets of pools. \n\nOsmosis Governance retains the right to remove the elevated permissions from the subDAO and/or manually overwrite fee levels on pairings. \n\n## Example Usages \n\nThis additional permission will allow several uses to be explored; this list is not exhaustive or mandatory for implementation, although the initial preferred routing trial is included in this signalling proposal. \n\n**Establishing new markets through fee promotions** \n\nFor markets established elsewhere but not present on Osmosis, the protocol fee could be reduced or removed to make Osmosis a more profitable place for liquidity providers to provide liquidity for the pairing. This is a form of passive incentivization in that the protocol sacrifices income in the short term to establish a market in the longer term, which would be a future source of revenue. Unlike liquidity bootstrapping spending, this comes at no upfront cost to the protocol and so has no cost if liquidity is not attracted, but encourages more organic generation of previously unlisted markets. \n\n**Raising fees during periods of high volatility in established markets** \n\nDuring periods of high volatility such as during initial listings or sudden market movements, the protocol fee could be raised in order to benefit from high volatility on certain pairings. \n\nThis enables Osmosis to increase fees on markets for which it is the primary location, potentially decreasing volatility or increasing protocol fee yield share. \n\n**Encouraging the formation of preferred routing** \n\nThe main mechanism for encouraging the routing of a wide range of assets through common pairings has been to incentivize the preferred routing. Changing the fees on a route can make this guidance more passive. By lowering or increasing protocol fees, the proportion of fees generated by volume that is held back for the protocol shifts as well as the lowest execution route that volume follows, making the lower protocol fee route more preferable to Liquidity Providers. \n\nThe first experiment proposed is to raise to OSMO/ATOM protocol fee from 0.1% to 0.2%. \n\nThis is intended to capitalize on the largest liquidity pairing present on Osmosis, the ATOM/OSMO liquidity that has been historically sticky even as incentives have been removed. \n\nBy raising the protocol fee on this routing, Osmosis will benefit more from this seemingly inactive liquidity as well as encouraging active liquidity providers to move to the more active ATOM/STABLE and OSMO/STABLE pairings, consolidating liquidity. \n\nThis experiment is one of two initiatives that aim to encourage the primary pairings of OSMO and other listed assets with Stable tokens, moving away from the previous methodology of OSMO as the main routing token for the chain, initially proposed in [Proposal 187](https://daodao.zone/dao/osmosis/proposals/187). \n\n**Forum Thread**: [https://forum.osmosis.zone/t/increase-scope-of-protocol-fee-controller-and-raise-atom-osmo-route-fee/3034](https://forum.osmosis.zone/t/increase-scope-of-protocol-fee-controller-and-raise-atom-osmo-route-fee/3034)",
      "changes": [
        {
          "subspace": "poolmanager",
          "key": "AdminAddresses",
          "value": "[\"osmo162wk8qc3w5s9hfs8dm76wrqnk6fjmsez2t4kk6zyugmrlzgds8sqfesmlm\",\"osmo10d07y265gmmuvt4z0w9aw880jnsr700jjeq4qp\"]"
        }
      ]
    },
    "initialDeposit": [
      {
        "denom": "uosmo",
        "amount": "1600000000"
      }
    ],
    "proposer": "osmo19w2t4ue7qpdh6022m3yxmxvv3w7jla7u3hfq0r"
  }
]

errors:

[
  {
    "path": "$input.transactions[9].messages[0].data.content[\"@type\"]",
    "expected": "\"/ibc.core.client.v1.ClientUpdateProposal\"",
    "value": "/cosmos.params.v1beta1.ParameterChangeProposal"
  },
  {
    "path": "$input.transactions[9].messages[0].data.content.subjectClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[9].messages[0].data.content.substituteClientId",
    "expected": "string"
  },
  {
    "path": "$input.transactions[9].messages[0].data.content.changes",
    "expected": "undefined",
    "value": [
      {
        "subspace": "poolmanager",
        "key": "AdminAddresses",
        "value": "[\"osmo162wk8qc3w5s9hfs8dm76wrqnk6fjmsez2t4kk6zyugmrlzgds8sqfesmlm\",\"osmo10d07y265gmmuvt4z0w9aw880jnsr700jjeq4qp\"]"
      }
    ]
  }
]

interface:

export interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposal {
    type: string;
    data: Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalData;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalData {
    content: Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalContent;
    initialDeposit: Osmosis1TrxMsgCosmosGovV1Beta1MsgSubmitProposalInitialDepositItem[];
    proposer: string;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalContent {
    '@type': string;
    title: string;
    description: string;
    changes: Osmosis1TrxMsgCosmosGovV1Beta1MsgSubmitProposalChangesItem[];
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalChangesItem {
    subspace: string;
    key: string;
    value: string;
}
interface Osmosis1TrxMsgCosmosGovV1beta1MsgSubmitProposalInitialDepositItem {
    denom: string;
    amount: string;
}

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.

1 participant