Skip to content

Commit

Permalink
fix: smart account signing checktx error (#8665)
Browse files Browse the repository at this point in the history
* fix: smart account signing checktx error

* chore: add changelog entry

(cherry picked from commit 51cdc5c)
  • Loading branch information
PaddyMc authored and mergify[bot] committed Sep 4, 2024
1 parent 6778e95 commit 3926ae5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
* [#8494](https://github.com/osmosis-labs/osmosis/pull/8494) Add additional events in x/lockup, x/superfluid, x/concentratedliquidity
* [#8543](https://github.com/osmosis-labs/osmosis/pull/8543) Add OTEL wiring and new configs in app.toml
* [#8566](https://github.com/osmosis-labs/osmosis/pull/8566) Minor speedup to CalcExitCFMM shares
* [#8665](https://github.com/osmosis-labs/osmosis/pull/8665) fix: smart account signing checktx error


## v25.2.1
Expand Down
4 changes: 2 additions & 2 deletions x/smart-account/authenticator/authentication_request.go
Original file line number Diff line number Diff line change
Expand Up @@ -245,8 +245,8 @@ func GenerateAuthenticationRequest(
AuthenticatorParams: nil,
}

// We do not generate the sign bytes if simulate is true
if simulate {
// We do not generate the sign bytes if simulate is true or isCheckTx is true
if simulate && ctx.IsCheckTx() {
return authRequest, nil
}

Expand Down

0 comments on commit 3926ae5

Please sign in to comment.