refactor(x/ecocredit): update to single class fee param

[ryanchristo]

Description

Closes: #1359

This pull request updates the new class fee ORM param to a single class fee. When the class fee is not set, no class fee is required. The proto breaking changes are only breaking changes to the current revision.

Manual Tests: #1466 (comment)

---

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...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers 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...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[codecov]

Codecov Report

Merging #1466 (4dfc862) into main (34a8615) will increase coverage by 0.00%.
The diff coverage is 83.14%.

@@           Coverage Diff           @@
##             main    #1466   +/-   ##
=======================================
  Coverage   78.23%   78.23%           
=======================================
  Files         243      243           
  Lines       18628    18624    -4     
=======================================
- Hits        14573    14571    -2     
+ Misses       3204     3202    -2     
  Partials      851      851           

Impacted Files	Coverage Δ
x/ecocredit/base/types/v1/params.go	43.07% <ø> (ø)
x/ecocredit/client/query.go	0.00% <0.00%> (ø)
x/ecocredit/base/types/v1/msg_update_class_fee.go	56.25% <60.00%> (ø)
x/ecocredit/base/keeper/query_class_fee.go	62.50% <62.50%> (ø)
x/ecocredit/base/keeper/msg_create_class.go	66.66% <80.95%> (-1.76%)	⬇️
x/ecocredit/base/client/query.go	66.73% <100.00%> (ø)
x/ecocredit/base/keeper/msg_update_class_fee.go	78.57% <100.00%> (+5.84%)	⬆️
x/ecocredit/base/keeper/query_params.go	56.00% <100.00%> (ø)
x/ecocredit/base/types/v1/codec.go	100.00% <100.00%> (ø)
x/ecocredit/client/testsuite/query.go	99.52% <100.00%> (+<0.01%)	⬆️
... and 6 more

[aleem1314]

lgtm

[ryanchristo]

Manual Tests: https://hackmd.io/8G5uGrUVS1yI3rxVCadmyQ. Everything works as expected. The only slightly odd behavior is that fee is a required argument in the create-class command and, if the fee is not set, you have to provide a fee with a positive amount even though you will not be charged a fee. I think we should consider making the fee argument a --class-fee flag that is optional (looks like this is already in place for create-basket). Will open a separate issue to address.