Skip to content
New issue

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

fix(templates): fix simapp simulation method update->delete #4421

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

kimurayu45z
Copy link
Contributor

In simapp template, I found one mistake that uses MsgUpdateXXX where it should use MsgDeleteXXX

@julienrbrt
Copy link
Member

Great catch! Thank you

@julienrbrt julienrbrt added backport/v28.x.y Backport to v28.x.y skip-changelog Don't check changelog for new entries labels Nov 22, 2024
@julienrbrt julienrbrt changed the title fix: simapp update->delete fix(templates): fix simapp simulation method update->delete Nov 22, 2024
@julienrbrt julienrbrt merged commit da55a09 into ignite:main Nov 22, 2024
41 of 49 checks passed
mergify bot pushed a commit that referenced this pull request Nov 22, 2024
(cherry picked from commit da55a09)

# Conflicts:
#	ignite/templates/typed/list/files/simapp/x/{{moduleName}}/simulation/{{typeName}}.go.plush
julienrbrt added a commit that referenced this pull request Nov 22, 2024
…4421) (#4422)

* fix(templates): fix simapp simulation method update->delete (#4421)

(cherry picked from commit da55a09)

# Conflicts:
#	ignite/templates/typed/list/files/simapp/x/{{moduleName}}/simulation/{{typeName}}.go.plush

* conflicts

---------

Co-authored-by: KIMURA Yu <[email protected]>
Co-authored-by: Julien Robert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:configs component:templates skip-changelog Don't check changelog for new entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants