feat: add governance and related events to custom modules#330
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 54 minutes and 56 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughThis PR adds Cosmos SDK event emissions across multiple modules to track state transitions. Event type and attribute constants are introduced in respective types packages, with x/rewards/types/events.go being a new file. Event emissions are added to keeper methods throughout the codebase: UpdateParams operations emit update_params events; rewards operations emit fund_pool, change_schedule, and reward_distributed events; fee abstraction operations emit update_fee_tokens, module_disabled, and token_disabled events; and the token factory SetMetadata operation emits a set_denom_metadata event. The CHANGELOG documents these additions. Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@copilot resolve the merge conflicts in this pull request |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
Description
update_params,fund_pool,change_schedule, andreward_distributedevents from x/rewardsupdate_paramsandset_denom_metadataevents from tokenfactoryupdate_params,update_fee_tokens,module_disabledandtoken_disabledevents on fee abstractionupdate_paramsevent on oracle moduleType of change
PR Checklist:
Make sure each step was done:
make lint-fix