Deprecate GetTransferAccount in ics20

[colin-axner]

Summary

A module account for ics20 is created and there exists an associated GetTransferAccount function. We never use the transfer module account as every escrow address is created as a regular account. We may want to migrate the escrow addresses to sub-accounts in the future, but a GetTransferAccount function will not be needed for this purpose (escrows will never be made to the module account)

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[crodriguezvega]

Is the following the work needed for this?

1. Delete this function.
2. Delete this function.
3. Remove this in InitGenesis.

Since we're changing the InitGenesis function, is there any considerations that we need to take care of when releasing this?

[colin-axner]

Is the following the work needed for this?

Correct.

Since we're changing the InitGenesis function, is there any considerations that we need to take care of when releasing this?

No, since this is API breaking, it needs to be included in a major release and thus state machine breaking considerations don't matter. Chains needs to run on the same minor or major release