docs: cosmoscmd migration

[aljo242]

Closes #2931

Please test these steps with a v0.24.0 chain. The guide is pretty long, so I might be missing something.

[tbruyelle]

Thanks for tackling this Alex!

I generated a chain with 0.24 and tried to apply the migration guide, it went smoothly, just sometimes I had a hard time to find where the diffs are. Since I wrote this code migration, it can possibly be even harder for a user I assume ^^'.

That's why I checked the docusaurus documentation and found this awesome // highlight-next-line instruction! Basically you put that before a line you want to highlight (in our case a line that has changed), and it appears hightlighted. Example for app/app.go :

Since I applied the migration on a chain, I can just run git diff to see where those instructions should be added, you can find that in this patch file.

---

Finally, icing on the cake, it's also possible to add custom magic comments (this is how those instructions are called in docusaurus, see related docs). We can add a custom one to specify that a line as been removed. Typically something that renders like that :

Tell me if you're interested, I have locally some docusaurus config changes for that.

[github-actions]

Visit the preview URL for this PR (updated for commit cb37ad5):

https://ignitesrvc-d--pr2949-docs-cosmoscmd-migra-5kh52m2o.web.app

_{(expires Wed, 09 Nov 2022 17:39:21 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 60382552e86c69890c026e09478220ea153394a8}

[aljo242]

Incredible @tbruyelle, thank you !

[RaulBernal]

Please kindly review the comments, thanks

[RaulBernal]

This changes are based in a SDK v0.45.x or v0.46.x ? For the second I guess
I think on Monday I will tweak it for v0.45
Thanks

[aljo242]

@tbruyelle some help with docusaurus config would be great!

[aljo242]

This changes are based in a SDK v0.45.x or v0.46.x ? For the second I guess I think on Monday I will tweak it for v0.45 Thanks

Yes, the latest version of ignite scaffolds v0.46.x chains, but removing cosmoscmd could be done for a v0.45.x or v0.46.x chain.

[tbruyelle]

@tbruyelle some help with docusaurus config would be great!

Sure, do you want to add a custom magic comment to highlight removed lines ?

[aljo242]

@tbruyelle some help with docusaurus config would be great!

Sure, do you want to add a custom magic comment to highlight removed lines ?

Yes please 👍

[tbruyelle]

@tbruyelle some help with docusaurus config would be great!

Sure, do you want to add a custom magic comment to highlight removed lines ?

Yes please +1

It's there 5a3aa9a

I added one magic comment as an example, you need to add the others :)

[aljo242]

@tbruyelle thanks for all of the help. Changes made here

[tbruyelle]

@aljo242 I see 2 other things we may address in the 0.25 migration :

• ICA controller keeper is not wired #2867
• Capability Keeper Seal #1921

Wdyt ? The changes concern the app/app.go file, should we merge them with the current migration ? (I can address them, I noticed the changes while comparing app.go with a chain in 0.24 and an other in 0.25)

[tbruyelle]

@tbruyelle thanks for all of the help. Changes made here

Sorry I couldn't stop myself from adding more magic comments :P 6b327ea

[aljo242]

@aljo242 I see 2 other things we may address in the 0.25 migration :

• ICA controller keeper is not wired #2867
• Capability Keeper Seal #1921

Wdyt ? The changes concern the app/app.go file, should we merge them with the current migration ? (I can address them, I noticed the changes while comparing app.go with a chain in 0.24 and an other in 0.25)

Yes, we should add these as well - great catch! I would really appreciate it if you added. You're a docs legend

[tbruyelle]

@aljo242 I see 2 other things we may address in the 0.25 migration :

• ICA controller keeper is not wired #2867
• Capability Keeper Seal #1921

Wdyt ? The changes concern the app/app.go file, should we merge them with the current migration ? (I can address them, I noticed the changes while comparing app.go with a chain in 0.24 and an other in 0.25)

Yes, we should add these as well - great catch! I would really appreciate it if you added. You're a docs legend

@aljo242 I added 2 new sections for each issues, please review :)

[tbruyelle]

@aljo242 843d0a1

• Moved all v0.25.1 items to v0.25.0
• Added a section for the Dragonberry fix in v0.25.1
• Fixed broken yaml in some markdown headers

[aljo242]

Amazing! merging!