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

Vesting account data #158

Closed
wants to merge 32 commits into from
Closed

Conversation

RiccardoM
Copy link
Contributor

@RiccardoM RiccardoM commented Jul 29, 2021

Description

Replaces #157. Originally opened by @HarleyAppleChoi. The base branch has changed named so we re-opened it
close #138

Checklist

  • Targeted PR against correct branch.
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote unit tests.
  • Re-reviewed Files changed in the Github PR explorer.

* apple/fix_bug:
  Fix bug
  Handle error for block and genesis is empty
  Fixes
  Made block time logging trace instead of debug
  Updated Juno to the last version

# Conflicts:
#	database/auth.go
#	modules/auth/utils/accounts.go
* cosmos-v0.40.x-ori:
  Change x/gov params durations to nanoseconds
  Fix how unknown messages are handled
  Delegations fixes
  Fix historical data

# Conflicts:
#	modules/consensus/handle_block.go
* cosmos-v0.40.x-ori:
  Updated how the messages parser is setup
  Added the ability to specify multiple module basics to handle custom messages
  Fixing unbonding delegation query
  Trying to fix some bugs

# Conflicts:
#	modules/auth/handle_fast_sync.go
@HarleyAppleChoi
Copy link
Contributor

close #138

@RiccardoM
Copy link
Contributor Author

Superseeded by #232

@RiccardoM RiccardoM closed this Oct 15, 2021
mergify bot pushed a commit that referenced this pull request Oct 20, 2021
<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v                               ✰  Thanks for creating a PR! ✰    
v    Before smashing the submit button please review the checkboxes.
v    If a checkbox is n/a - please still include it but + a little note why
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

## Description
This PR replaces #158 
close #138

## Checklist
- [x] Targeted PR against correct branch.
- [x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [x] Wrote unit tests.  
- [x] Re-reviewed `Files changed` in the Github PR explorer.
@RiccardoM RiccardoM deleted the apple/vesting_account_data branch October 28, 2021 05:41
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 this pull request may close these issues.

Vesting account data
3 participants