desmos-labs / desmos

Improving the well-being of users on social networks through the blockchain technology.
https://desmos.network
Apache License 2.0
104 stars 46 forks source link

fix: set `x/profiles` keeper of msg server into reference as DesmosApp #1222

Closed dadamu closed 12 months ago

dadamu commented 12 months ago

Description

Currently, x/profiles msg server is not a reference as DesmosApp and it is registered before setting IBC keepers. It occurs panics since there is no IBC features inside x/profiles msg server.

To fix it, we should set keeper of msg server into reference as DesmosApp, then the IBC keepers can be properly set.


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

codecov[bot] commented 12 months ago

Codecov Report

Patch coverage is 82.05% of modified lines.

Files Changed Coverage
app/app_config.go ø
app/wasm_config.go ø
x/profiles/keeper/msg_server_app_link.go ø
x/profiles/module.go 0.00%
app/app.go 100.00%
x/profiles/abci.go 100.00%
x/profiles/keeper/invariants.go 100.00%
x/profiles/keeper/keeper.go 100.00%
x/profiles/keeper/msg_server_chain_link.go 100.00%
x/profiles/keeper/msg_server_dtag_transfers.go 100.00%
... and 4 more

:loudspeaker: Thoughts on this report? Let us know!.