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.

Sign up for GitHub

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

Jump to bottom

Add additional checks on unbonding transaction #14

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Add additional checks on unbonding transaction #14

merged 1 commit into from
Oct 14, 2024

Conversation

KonradStaniec
Copy link
Collaborator

  • Bump babylon version
  • Use new CheckPreSignedUnbondingTxSanity stateless function to pre-elimary check unbonding transaction

Lazar955
Lazar955 approved these changes Oct 14, 2024
View reviewed changes
Copy link
Member

@Lazar955 Lazar955 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@KonradStaniec KonradStaniec merged commit e13957e into main Oct 14, 2024
11 checks passed
@KonradStaniec KonradStaniec deleted the konrastaniec/bump-babylon-version-with-new-check branch October 14, 2024 14:37
KonradStaniec added a commit that referenced this pull request Oct 14, 2024
@KonradStaniec
Add additional checks on unbonding transaction (#14)
c9feded
@KonradStaniec KonradStaniec mentioned this pull request Oct 14, 2024
Merged
KonradStaniec added a commit that referenced this pull request Oct 15, 2024
@KonradStaniec
Add additional checks on unbonding transaction (#14) (#15)
2c6bcfb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lazar955 Lazar955 Lazar955 approved these changes

@gitferry gitferry Awaiting requested review from gitferry

@jrwbabylonlab jrwbabylonlab Awaiting requested review from jrwbabylonlab

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KonradStaniec @Lazar955