Add a flag to forc contract-id command to show it in bech32 format

[kayagokalp]

Currently forc-contract-id show the contract id in hex format but it is possible that users may want to get in bech32 format as well. We should have a flag for this

[ADTmux]

Hey, I would like to work on this!

[kayagokalp]

Hey, I would like to work on this!

Sounds great! go for it :) assigned to you

[ADTmux]

Hey, quick question, I recently tried to modify https://github.com/FuelLabs/sway/blob/master/forc/src/ops/forc_contract_id.rs and I wanted to test my modifications to the command, could you please lead me to the right part of the documentation that would help me modify the forc binary installed in my setup to get started on this

[kayagokalp]

Hey, quick question, I recently tried to modify https://github.com/FuelLabs/sway/blob/master/forc/src/ops/forc_contract_id.rs and I wanted to test my modifications to the command, could you please lead me to the right part of the documentation that would help me modify the forc binary installed in my setup to get started on this

Great to hear that you are making progress! You can build your version of forc with cargo build and use forc with absolute path ./target/debug/forc from the root of the sway repo. Or you can remove your current forc installation (if you installed it through cargo check your ./cargo/bin/ or if you installed it from fuelup check your ./fuelup/bin) and install your modified version with cargo install. Also consider opening a PR once you feel like it to start review process.

[ADTmux]

Hi, I did integrate the flag, I wanted a hint on how the forc contract-id --help command prints out information so that I could integrate a line on --bech32

[ADTmux]

Hi @kayagokalp could you assist me with this last bit? :)

[kayagokalp]

Hey @ADTmux please open a PR so that i can see your WIP and assist you better. You should add this new flag to https://github.com/FuelLabs/sway/blob/daee1c247c86038bf9a3faefbca80d0aca4ebf0e/forc/src/cli/commands/contract_id.rs#L18C12-L18C19

[sdankel]

@kayagokalp Can we close this since we are removing Bech32 format from the wallet?

[kayagokalp]

Yes that seems reasonable to me :)