We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider refactoring to improve modularity and readability.
+ func (k *Keeper) handleHostZone(ctx sdk.Context, zone *icstypes.Zone) error {...} + func (k *Keeper) handleOsmosisZone(ctx sdk.Context, zone *icstypes.Zone) error {...} + func (k *Keeper) handleUmeeZone(ctx sdk.Context, zone *icstypes.Zone) error {...}
Originally posted by @coderabbitai[bot] in #1620 (comment)
Consider logging errors before returning them for better debuggability.
+ k.Logger(ctx).Error("channel not found", "channel", zone.TransferChannel) + k.Logger(ctx).Error("error marshaling osmosis denom", "error", err.Error()) + k.Logger(ctx).Error("error marshaling umee denom", "error", err.Error())
Also applies to: 201-201, 214-214, 256-256
The text was updated successfully, but these errors were encountered:
is this still available?
Sorry, something went wrong.
This module is currently being refactored by @arhamchordia.
arhamchordia
No branches or pull requests
Originally posted by @coderabbitai[bot] in #1620 (comment)
Also applies to: 201-201, 214-214, 256-256
Originally posted by @coderabbitai[bot] in #1620 (comment)
The text was updated successfully, but these errors were encountered: