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

fix: package upgrades, include all minter object fields in export and init genesis #480

Closed
wants to merge 13 commits into from

Conversation

tkernell and others added 13 commits November 24, 2024 09:51
* reporter module

* use legacy dec for tips and commision rates

* re-add max commision rate check

* chore: mv legacy_dec.go to types/legacy_dec.go

* minor refactor for clarity

* checkpoint to pull main

* use LgeacyDec for withdraws

* remove old BigUint to LegacyDec fcns

* linting

* add test

* linting

* separate blobstream testnet child contract

---------

Co-authored-by: danflo27 <[email protected]>
Co-authored-by: akrem <[email protected]>
* add more timestamps for tracking

* cleanup after review

* snapshot limit

* fix tests

* lint

* remove log
* add more timestamps for tracking

* snapshot limit getter

* test snapshot limit getter
* added migration functions to add new fields to bridge keeper. Set upgrade name to match software upgrade plan name

* pushing up a change that was missed

* made changes to deal with circular import

* branch is now building with no circular import errors

* removed migrations.go file in keeper and am just importing the v2 migration package into module.go

* fixed migration function to use right context

* fixed typo

* changed format around to desired format

* saving changes before branching to old tag to test

* rebased from main to hopefully solve some git issues with merge

* lint fixes
* updates

* fix
* scripts for setup and testing

* formatting

* typo

* more formatting
* chore: remove unused files

* reduce for loops in WeightedMode

* explore removing aggregation from endBlock

* add different indexes, fix tests plus more

* uncomment test

* add case to TestAddReport

* add test, rename fields, cleanup

* remove aggregate_report_index field

* chore: cleanup comments and prints

* fix cyclelist rewards

* block bridge deposit report for claimed ids

* rate limit snapshot creation

* wip test

* chore: remove unused files

* mv reward calculations from end block (#457)

* first commit

* refactor and fix getter

* handle leftover fraction plus tests

* change docker image for test

* fix workflow cfg

* separate workflow jobs

* fix x/reporter/autocli

* lint

* upgrade handling

* register handler

* lint

* add msg_server_withdraw_tip_legacy for tips before
upgrade

* fix migrations method

* lint

* fix migrations

* add query to fetch report ids and query_ids

* small test

* add query to autocli
@danflo27 danflo27 changed the base branch from main to auditDec2024 December 18, 2024 18:23
@danflo27 danflo27 marked this pull request as draft December 18, 2024 18:56
@danflo27 danflo27 closed this Dec 18, 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 this pull request may close these issues.

7 participants