feat(x/ecocredit): add retirement reason

[ryanchristo]

Description

Closes: #1624

Add reason to all messages that can trigger retirement and update tests to include check for reason.

---

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 #1625 (a317ef5) into main (dccf079) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

❗ Current head a317ef5 differs from pull request most recent head f1f6cfe. Consider uploading reports for the commit f1f6cfe to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1625      +/-   ##
==========================================
- Coverage   77.59%   77.51%   -0.09%     
==========================================
  Files         266      264       -2     
  Lines       19457    19321     -136     
==========================================
- Hits        15098    14977     -121     
+ Misses       3490     3479      -11     
+ Partials      869      865       -4     

Impacted Files	Coverage Δ
x/ecocredit/base/keeper/msg_create_batch.go	66.49% <100.00%> (+0.17%)	⬆️
x/ecocredit/base/keeper/msg_retire.go	59.75% <100.00%> (+0.49%)	⬆️
x/ecocredit/base/keeper/msg_send.go	69.40% <100.00%> (+0.23%)	⬆️
x/ecocredit/base/types/v1/msg_retire.go	92.00% <100.00%> (+1.09%)	⬆️
x/ecocredit/base/types/v1/msg_send.go	95.12% <100.00%> (+0.38%)	⬆️
x/ecocredit/base/types/v1/types_batch_issuance.go	100.00% <100.00%> (ø)
x/ecocredit/basket/keeper/msg_take.go	59.77% <100.00%> (+0.68%)	⬆️
x/ecocredit/basket/types/v1/msg_take.go	87.80% <100.00%> (+0.96%)	⬆️
x/ecocredit/marketplace/keeper/msg_buy_direct.go	76.13% <100.00%> (+0.27%)	⬆️
x/ecocredit/marketplace/keeper/utils.go	42.85% <100.00%> (+0.45%)	⬆️
... and 8 more

[blushi]

LGTM
in which release will this be included?

[aleem1314]

lgtm

[ryanchristo]

in which release will this be included?

This will be included in v5.0 (only the bridge event update will be included in v4.1.5).

[ryanchristo]

It's optional so the tx memo field can still be used and interface team can update when ready.