search

liftedinit / manifest-ledger

CosmosSDK-based blockchain ledger for the Manifest Network
Apache License 2.0
0 stars 2 forks source link

Add support for burning native tokens #63

Closed fmorency closed 1 month ago

fmorency commented 1 month ago

@Reecepbcups, do we have the means to burn native tokens, e.g., umfx?

I tried using the MsgBurn from the tokenfactory module where the sender is the POA admin but it failed with

"proposal execution failed on proposal 4, because of error message /osmosis.tokenfactory.v1beta1.MsgBurn at position 0: unauthorized account"

I haven't found anything in the x/bank module to allow burning native tokens and my research in the Cosmos repo came back empty-handed.

Reecepbcups commented 1 month ago

Tokenfactory can only burn tokens from the tokenfactory module itself. umfx is not

Does not look like SDK v50 has a burn, but it is in main :. We could add a simple command in manifest module to bank.Burn() umfx from a senders held balance if you want image

fmorency commented 1 month ago

That's what I thought, thanks for confirming.

Yes, adding a simple command in the manifest module until we upgrade to 0.51 would be great. I will change the title of this ticket to track it.

Thanks @Reecepbcups