helpers for nested create Any objects

[pyramation]

https://github.com/cosmos/groups-ui/blob/master/src/shared-state/groups-store.ts#L363

• fix type issue in comment below
• fix the interface so developer doesn't have to encode decisionPolicy field

import Long from 'long';
const {
    createGroupWithPolicy
} = cosmos.group.v1.MessageComposer.withTypeUrl;

const {
    PercentageDecisionPolicy   
} = cosmos.group.v1;

const msg = createGroupWithPolicy({
    admin: 'a string',
    groupMetadata: 'string here',
    groupPolicyAsAdmin: true,
    groupPolicyMetadata: 'a string here too',
    members: [], // MemberRequest[],
    decisionPolicy: { // types here aren't exactly happy with the encode function for some reason...
        // WE CAN IMPROVE THIS....
        typeUrl: '/cosmos.group.v1.PercentageDecisionPolicy',
        value: PercentageDecisionPolicy.encode({
            percentage: 'a string',
            windows: {
                votingPeriod: {
                    nanos: 0,
                    seconds: Long.fromValue(100)
                },
                minExecutionPeriod: {
                    nanos: 0,
                    seconds: Long.fromValue(100)
                }
            }
        }).finish()
    }
})

[atmoner]

It work!! Good job @pyramation 👏

For the members part, the structure is this (the weight represents the voting power of the users in the group)

    members: [{
      address: "bcna1abcdefgh",
      weight: "10", 
      metadata: "First user"
    }], // MemberRequest[],

[pyramation]

@atmoner I believe I fixed the amino (for ledger support) can you test sometime?

checkout interchain46@0.4.0-rc.4

import { cosmos } from 'interchain46';
const {
    MsgCreateGroupWithPolicy
} = cosmos.group.v1
import type { MsgCreateGroupWithPolicyEncoded } from 'interchain46/types/codegen/cosmos/group/v1/tx'

const policy: MsgCreateGroupWithPolicyEncoded = {
    admin: 'address',
    groupMetadata: 'https://some.ipfs.w3s.link/group_metadata.json',
    groupPolicyAsAdmin: true,
    groupPolicyMetadata: 'https://some.ipfs.w3s.link/group_metadata.json',
    members: [
        {
            address: 'address',
            weight: '10',
            metadata: 'First user'
        }
    ],
    decisionPolicy: PercentageDecisionPolicy.toProtoMsg({
        percentage: '0.5',
        windows: {
            votingPeriod: {
                nanos: 0,
                seconds: Long.fromValue(100)
            },
            minExecutionPeriod: {
                nanos: 0,
                seconds: Long.fromValue(100)
            }
        }
    })
};

// try broadcasting the amino one here:

const msg = {
    typeUrl: '/cosmos.group.v1.MsgCreateGroupWithPolicy',
    value: policy
}

@atmoner example msg in this gist with the new above code

if that works, I think we have a solution, just need to package this all up into a neat interface/api

[ryanchristo]

Hey there 👋

We're in the process of switching to telescope and are now running into this issue. Any updates? How can we help?

[pyramation]

hey @ryanchristo which modules are you using this for? And have you guys upgraded the cosmos sdk that uses the accepts_interface and implements_interface annotations?

If so, you'll already be able to create things much more easily, and then this issue would be great to make it even easier. I think we can have @Zetazzz prioritize this soon.

[pyramation]

what would be great to help, if we can build this feature and @atmoner and @ryanchristo can test the result to ensure it works for your use cases.

[ryanchristo]

which modules are you using this for?

I ran into this issue with the group module and the same message as the example. We're using a fork of v0.46 that adds amino support for gov and group module (added amino support being the reason for a fork - regen-network/cosmos-sdk#49).

I'd be happy to test for our use case. Feel free to add me as a reviewer or tag me in the pull request. Thanks!

[pyramation]

https://gist.github.com/pyramation/13f47ab2d9e6e6a86d3fb5d45999dc7b

here is an example of a test that has the buffer in view

and here is an example of the repo we were testing

https://github.com/pyramation/interchain46/blob/cd8e22ea63a8197935be02839c0265541ff61c09/src/codegen/cosmos/group/v1/tx.ts#L1906

https://github.com/pyramation/interchain46/blob/cd8e22ea63a8197935be02839c0265541ff61c09/src/codegen/cosmos/group/v1/tx.ts#L4007

[pyramation]

hey @ryanchristo have you guys tried using the toProtoMsg helpers similar to how it's done here?

import { cosmos } from 'interchain46';
const {
    MsgCreateGroupWithPolicy
} = cosmos.group.v1
import type { MsgCreateGroupWithPolicyEncoded } from 'interchain46/types/codegen/cosmos/group/v1/tx'

const policy: MsgCreateGroupWithPolicyEncoded = {
    admin: 'address',
    groupMetadata: 'https://some.ipfs.w3s.link/group_metadata.json',
    groupPolicyAsAdmin: true,
    groupPolicyMetadata: 'https://some.ipfs.w3s.link/group_metadata.json',
    members: [
        {
            address: 'address',
            weight: '10',
            metadata: 'First user'
        }
    ],
    decisionPolicy: PercentageDecisionPolicy.toProtoMsg({
        percentage: '0.5',
        windows: {
            votingPeriod: {
                nanos: 0,
                seconds: Long.fromValue(100)
            },
            minExecutionPeriod: {
                nanos: 0,
                seconds: Long.fromValue(100)
            }
        }
    })
};

// try broadcasting the amino one here:

const msg = {
    typeUrl: '/cosmos.group.v1.MsgCreateGroupWithPolicy',
    value: policy
}

we need somebody to help us test these helpers, would be amazing to get some feedback. The toProtoMsg may need to be enabled in your codegen.js config, these options will need to be enabled for this all to work:

https://github.com/pyramation/interchain46/blob/cd8e22ea63a8197935be02839c0265541ff61c09/scripts/codegen.js#L20-L23

https://github.com/pyramation/interchain46/blob/cd8e22ea63a8197935be02839c0265541ff61c09/scripts/codegen.js#LL41C1-L42C1

a nice to have IMO, to get $typeUrl so when things are decoded, you know what their types are still: https://github.com/pyramation/interchain46/blob/cd8e22ea63a8197935be02839c0265541ff61c09/scripts/codegen.js#L26

[Zetazzz]

Hi, so, just in short, configs are like these:

    interfaces: {
      enabled: true,
      useUnionTypes: false
    },
...
    prototypes:{
        addTypeUrlToDecoders: true,
...
    },
....
    aminoEncoding: {
      enabled: true,
      useRecursiveV2encoding: true,
    },

Other team also encountered this, and I've tested with the listed config and they work!

Hope they also work for you! Thx!!

[ryanchristo]

Sorry for the delay. I started testing this yesterday and ran into some issues. I updated the configuration and made use of the toProtoMsg helper within MsgCreateGroupWithPolicy and then hit the infamous unauthorized error:

Broadcasting transaction failed with code 4 (codespace: sdk). Log: signature verification failed; please verify account number (1), sequence (4) and chain-id (regen-local): unauthorized

Receiving errors with other messages as well now so I believe this is an issue on my end but will test and followup today.

[ryanchristo]

The above error occurs when metadata fields are empty strings or when groupPolicyAsAdmin is false. Not sure if you have any ideas as to what might be going on here. I was able to successfully execute this message using amino when creating a group with policy as admin (i.e. groupPolicyAsAdmin set to true) and with metadata ' ' (non-empty string).

[ryanchristo]

Another message we are hoping to provide amino support for is MsgSubmitProposal but MsgSubmitProposalEncoded does not exist. Wondering if you have any thoughts on how we would support amino for MsgSubmitProposal as well. We are ok with having limited support for which messages if we end up in a situation where we would need to encode each message.

If you want to look at some code, here is the pull request: regen-network/groups-ui#69

[hoangdv2429]

@hoangdv2429