Closed Yaroms closed 1 month ago
This diff introduces significant enhancements across various components of the application, focusing on integrating a new SpecKeeper
dependency, restructuring message types in Protocol Buffers, and updating CLI commands. The changes streamline initialization processes, improve organizational structures, and modify how rewards and subscription data are queried and handled, ultimately enhancing the application's functionality.
Files | Change Summary |
---|---|
app/app.go |
Updated New function to include an additional SpecKeeper parameter for improved initialization. |
proto/.../rewards/base_pay.proto |
Reorganized import statements and redefined message structures; introduced new BasePayWithIndex message type. |
proto/.../rewards/genesis.proto |
Changed base_pays from repeated BasePayGenesis to repeated BasePayWithIndex in GenesisState . |
proto/.../rewards/query.proto |
Replaced ProviderReward RPC with SpecTrackedInfo , updating request and response message types accordingly. |
proto/.../subscription/cu_tracker.proto |
Added new TrackedCuInfo message; adjusted indentation for existing messages. |
proto/.../subscription/query.proto |
Added new RPC methods TrackedUsage and EstimatedRewards . |
scripts/protocgen.sh |
Changed shebang to #!/usr/bin/env bash . |
testutil/.../tester.go |
Removed QueryRewardsProviderReward method from Tester struct. |
testutil/.../keepers_init.go |
Modified keeper initializations to include ks.Spec . |
testutil/.../subscription.go |
Added nil argument in dualstakingkeeper.NewKeeper function call. |
x/rewards/.../query.go |
Replaced command CmdQueryProviderReward() with CmdSpecTrackedInfo() . |
x/rewards/.../query_spec_tracked_info.go |
Renamed CmdQueryProviderReward to CmdSpecTrackedInfo with updated functionality. |
x/rewards/.../base_pay.go |
Transitioned methods to use BasePayWithIndex instead of BasePayIndex . |
x/rewards/.../genesis.go |
Updated DefaultGenesis function to use BasePayWithIndex for BasePays . |
x/subscription/.../cli/query.go |
Added CmdEstimatedRewards() and CmdTrackedCuUsage() commands. |
x/subscription/.../keeper/keeper.go |
Introduced specKeeper field to Keeper struct, updating related methods. |
x/subscription/.../expected_keepers.go |
Enhanced keeper interfaces with new methods for various functionalities. |
sequenceDiagram
participant App
participant SpecKeeper
participant RewardsKeeper
participant Query
App->>SpecKeeper: Initialize with SpecKeeper
App->>RewardsKeeper: Initialize with RewardsKeeper
App->>Query: Execute SpecTrackedInfo
Query->>SpecKeeper: Fetch spec-related data
Query->>RewardsKeeper: Fetch rewards data
RewardsKeeper-->>Query: Return rewards data
SpecKeeper-->>Query: Return spec data
Query-->>App: Return combined response
A Rabbit's Celebration of Change
π° In fields so wide, beneath the sky,
New keepers hop, as time goes by.
With specs to track and rewards to share,
Each change a dance, a joyful flair!
So let us play and let us sing,
For code and bunnies bring spring! πΌ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
2β167 testsβ +1βββ2β167 :white_check_mark: +1βββ25m 58s :stopwatch: +48s ββ143 suites Β±0βββββββ0 :zzz: Β±0β ββββ7 filesββ Β±0βββββββ0 :x: Β±0β
Results for commit 4ad59849.βΒ± Comparison against base commit bc12d329.
:recycle: This comment has been updated with latest results.
Description
Closes: #XXXX
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changemain
branchReviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores