refactor(minor-rewards): msg and state separation

[fish-sammy]

Description

Todos

• Unit tests
• Manual tests
• Documentation
• Connect epics/issues

Convention Checklist

• Each contract should have a client mod for others to interact with it.
• Derive macros
  • EnsurePermissions: Contract permission control
  • IntoContractError: Conversion from custom contract errors to axelar_wasm_std::error::ContractError
  • IntoEvent: Event serialization
  • migrate_from_version: Contract version management in the migrate function
• The state mod and msg mod should use separate data structures so that internal state changes do not break the contract interface. Check out the interchain-token-service for reference.
  • msg.rs should never use any type from the state.rs
  • Shared types must be defined in a separate exported mod. If those types have already been defined somewhere else, then they should get re-exported in the exported mod

Steps to Test

Expected Behaviour

Notes

[codecov]

Codecov Report

Attention: Patch coverage is 89.02439% with 9 lines in your changes missing coverage. Please review.

Project coverage is 93.47%. Comparing base (fed53af) to head (5681ca2).

Files with missing lines	Patch %	Lines
contracts/rewards/src/contract.rs	75.67%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #761      +/-   ##
==========================================
- Coverage   93.50%   93.47%   -0.03%     
==========================================
  Files         233      233              
  Lines       38893    38948      +55     
==========================================
+ Hits        36366    36407      +41     
- Misses       2527     2541      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cgorenflo]

I don't understand this change. Why own the PoolId if you only need it for references?