Skip to content

Commit

Permalink
fix: set senderAddress in SLCs from CW (#1267)
Browse files Browse the repository at this point in the history
# Background

SLCs generated from CosmWasm contracts were setting the
`ContractAddress` field, but not the `SenderAddress` field. This is an
issue for pigeon feed. We now set both fields on these calls.

# Testing completed

- [ ] test coverage exists or has been added/updated
- [ ] tested in a private testnet

# Breaking changes

- [x] I have checked my code for breaking changes
- [x] If there are breaking changes, there is a supporting migration.
  • Loading branch information
maharifu authored Aug 19, 2024
1 parent fca3518 commit 905c5d8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion x/scheduler/keeper/wasm_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ func (k Keeper) ExecuteWasmJobEventListener() wasmutil.MessengerFnc {
return nil, nil, whoops.Wrap(err, types.ErrWasmExecuteMessageNotValid)
}

msgID, err := k.ExecuteJob(ctx, executeMsg.JobID, executeMsg.Payload, nil, contractAddr)
msgID, err := k.ExecuteJob(ctx, executeMsg.JobID, executeMsg.Payload, contractAddr, contractAddr)
if err != nil {
logger.WithError(err).Error("Failed to trigger job execution.")
return nil, nil, err
Expand Down

0 comments on commit 905c5d8

Please sign in to comment.