ENG 388 Remove governance hook from erc20 module

[danburck]

Description

Closes: https://linear.app/tharsis/issue/ENG-388/remove-governance-hook-from-erc20-module

The x/erc20 audit revealed, that Token Pair proposers should perform proper due diligence prior to registering the proposal. This PR removes the governance hook to streamline the proposal process.

• impr(erc20): remove toke pair voting period param
• impr(erc20): remove governance hook

[linear]

ENG-388 Remove governance hook from erc20 module

context: we want the proposals (especially ERC20 registration) to be properly reviewed

• Remove param
• Remove hook
• Update keeper and app

[codecov]

Codecov Report

Merging #190 (77c8999) into main (71c4413) will decrease coverage by 0.48%.
The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   72.78%   72.30%   -0.49%     
==========================================
  Files          33       32       -1     
  Lines        2065     1993      -72     
==========================================
- Hits         1503     1441      -62     
+ Misses        499      491       -8     
+ Partials       63       61       -2     

Impacted Files	Coverage Δ
x/erc20/keeper/keeper.go	100.00% <ø> (ø)
app/app.go	80.58% <100.00%> (-0.09%)	⬇️
x/erc20/types/params.go	100.00% <100.00%> (ø)

[fedekunze]

ACK, can you add a state machine breaking changelog entry?