Skip to content

imp: add extra validation to ica msgs (backport #8734)#8738

Merged
srdtrk merged 8 commits intorelease/v8.8.xfrom
mergify/bp/release/v8.8.x/pr-8734
Dec 18, 2025
Merged

imp: add extra validation to ica msgs (backport #8734)#8738
srdtrk merged 8 commits intorelease/v8.8.xfrom
mergify/bp/release/v8.8.x/pr-8734

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 18, 2025

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.

  • Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
  • Include changelog entry when appropriate (e.g. chores should be omitted from changelog).
  • Wrote unit and integration tests if relevant.
  • Updated documentation (docs/) if anything is changed.
  • Added godoc comments if relevant.
  • Self-reviewed Files changed in the GitHub PR explorer.
  • Provide a conventional commit message to follow the repository standards.

This is an automatic backport of pull request #8734 done by [Mergify](https://mergify.com).

* imp: add extra validation

* chore: lint

* imp: error msg

* test: fix tests

* docs: document the validation change

---------

Co-authored-by: Alex | Cosmos Labs <alex@cosmoslabs.io>
(cherry picked from commit 7574333)

# Conflicts:
#	docs/docs/02-apps/02-interchain-accounts/07-tx-encoding.md
#	modules/apps/27-interchain-accounts/host/keeper/relay_test.go
@mergify mergify bot added the conflicts label Dec 18, 2025
@mergify
Copy link
Contributor Author

mergify bot commented Dec 18, 2025

Cherry-pick of 7574333 has failed:

On branch mergify/bp/release/v8.8.x/pr-8734
Your branch is up to date with 'origin/release/v8.8.x'.

You are currently cherry-picking commit 7574333c.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   modules/apps/27-interchain-accounts/types/codec.go

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   docs/docs/02-apps/02-interchain-accounts/07-tx-encoding.md
	both modified:   modules/apps/27-interchain-accounts/host/keeper/relay_test.go

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@srdtrk srdtrk removed the conflicts label Dec 18, 2025
@srdtrk srdtrk merged commit 31be276 into release/v8.8.x Dec 18, 2025
19 checks passed
@srdtrk srdtrk deleted the mergify/bp/release/v8.8.x/pr-8734 branch December 18, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant