cherry pick #5839 to Launchpad

[fedekunze]

Description

closes: #XXXX

---

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

• Targeted PR against correct branch (see CONTRIBUTING.md)
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration tests
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• Re-reviewed Files changed in the Github PR explorer

[alexanderbez]

We just cut the RC-1 release. Is this breaking change something we absolutely need?

[fedekunze]

Pasting my comment from Discord:

it's an API breaking change as well, but it'd be extremely beneficial for Aragon Chain and Ethermint bc we've been working from unstable commits since last year

Is this breaking change something we absolutely need?

It reverts the change from 0.38 that removed the sdk.Result from the simulation responses, so that we have the same behaviour as the 0.37.x series.

[alexanderbez]

ACK. If you need to test this, you'll have to cut another RC and update the release entry.