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

Sunny/key output improvement #1539

Closed
wants to merge 4 commits into from
Closed

Sunny/key output improvement #1539

wants to merge 4 commits into from

Conversation

faboweb
Copy link
Contributor

@faboweb faboweb commented Jul 4, 2018 •
edited
Loading

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG.md
  • Updated Gaia/Examples
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)

@faboweb faboweb requested review from sunnya97 and cwgoes July 4, 2018 08:45
@faboweb faboweb requested a review from ebuchman as a code owner July 4, 2018 08:45
@faboweb faboweb closed this Jul 4, 2018
@faboweb faboweb deleted the fabo/fix-new-key-output branch July 4, 2018 08:49
chillyvee pushed a commit to chillyvee/cosmos-sdk that referenced this pull request Mar 1, 2024
@yaruwangway @glnro @okwme
add bypass-min-fee (cosmos#1539)
3b1758f 
* add bypass-min-fee

* fix: lint

* chore: change comment position

* add bypass min fee test

* fix: lint

* Update ante/fee.go

Co-authored-by: lg <8335464+glnro@users.noreply.github.com>

* Update ante/fee.go

Co-authored-by: lg <8335464+glnro@users.noreply.github.com>

* refactor: FeeWithBypassDecorator -> BypassMinFeeDecorator

* create fallback default bypass min fee messages (cosmos#1567)

* Update app/app.go

Co-authored-by: billy rennekamp <billy.rennekamp@gmail.com>

Co-authored-by: lg <8335464+glnro@users.noreply.github.com>
Co-authored-by: billy rennekamp <billy.rennekamp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunnya97 sunnya97 Awaiting requested review from sunnya97

@cwgoes cwgoes Awaiting requested review from cwgoes

@ebuchman ebuchman Awaiting requested review from ebuchman

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

Successfully merging this pull request may close these issues.
1 participant
@faboweb