Remove sdk.MustSortJSON from interchain account packet data GetBytes function

[colin-axner]

Summary

See reasoning in #3915. Do not switch to standard json library as this changes the json econding due to the use of the enum on the Type field.

Problem Definition

Sorting is unnecessary and has been deprecated in the SDK.

We cannot change to standard json because the encoding is different:

json

{"type":1,"data":"ZGF0YQ==","memo":"memo"}

proto3 json

{"type":"TYPE_EXECUTE_TX","data":"ZGF0YQ==","memo":"memo"}

NOTE: proto3 json can decode the standard json encoding, but the standard json library cannot decode the proto3 json encoding

Proposal

Remove sdk.MustSortJSON usage in InterchainAccountPacketData.GetBytes()

---

For Admin Use

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