Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

ante: fix fee check #597

Merged
merged 4 commits into from
Nov 16, 2020
Merged

ante: fix fee check #597

merged 4 commits into from
Nov 16, 2020

Conversation

fedekunze
Copy link
Contributor

No description provided.

@fedekunze fedekunze requested a review from noot as a code owner November 13, 2020 14:21
@fedekunze fedekunze requested a review from araskachoi November 13, 2020 14:23
@fedekunze fedekunze requested a review from okwme November 16, 2020 11:10
@fedekunze
Copy link
Contributor Author

@noot @araskachoi please review whenever you are free

okwme
okwme approved these changes Nov 16, 2020
View reviewed changes
Copy link
Contributor

@okwme okwme left a comment

Choose a reason for hiding this comment

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

LGTM!
Great to see this added : )
I'm curious why my modification here didn't throw an error in the antehandler originally though.

@fedekunze fedekunze merged commit 73e5eb5 into development Nov 16, 2020
@fedekunze fedekunze deleted the fedekunze/fix-fee-check branch November 16, 2020 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@okwme okwme okwme approved these changes

@noot noot Awaiting requested review from noot

@araskachoi araskachoi Awaiting requested review from araskachoi

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
@fedekunze @okwme