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

x/sequencers: simplify bond update allowed to take sequencer #1059

Closed
Tracked by #1055
danwt opened this issue Aug 9, 2024 · 1 comment · Fixed by #1015
Closed
Tracked by #1055

x/sequencers: simplify bond update allowed to take sequencer #1059

danwt opened this issue Aug 9, 2024 · 1 comment · Fixed by #1015

Comments

@danwt
Copy link
Contributor

danwt commented Aug 9, 2024

// check if the sequencer already exists
sequencer, found := k.GetSequencer(ctx, senderAddress)
if !found {
return types.Sequencer{}, types.ErrUnknownSequencer
}

should take sequencer as arg, not return it

#1031 (comment)

@danwt
Copy link
Contributor Author

danwt commented Oct 16, 2024

superseded by #1328

@danwt danwt closed this as completed Oct 16, 2024
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 a pull request may close this issue.

2 participants