osmosis icon indicating copy to clipboard operation
osmosis copied to clipboard

fix GetModuleToDistributeCoins

Open catShaark opened this issue 2 years ago • 0 comments

What is the purpose of the change

This is related to this issue here https://github.com/osmosis-labs/osmosis/issues/2327. After discussion we agree that GetModuleToDistributeCoins of incentives keeper has the wrong logic. This PR fixes GetModuleToDistributeCoins

Brief Changelog

Use UpcomingGaugesIterator instead of UpcomingGaugesIteratorAfterTime in GetModuleToDistributeCoins

Testing and Verifying

(Please pick one of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added unit test that validates ...
  • Added integration tests for end-to-end deployment with ...
  • Extended integration test for ...
  • Manually verified the change by ...

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes? no
  • Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
  • How is the feature or change documented? not documented

catShaark avatar Oct 06 '22 10:10 catShaark