Create a new Constructor function that creates transfer message with Metadata

[AdityaSripal]

Create a new constructor function that takes in metadata that gets included in the packet data struct. This should not replace the existing method to maintain backwards compatibility

[colin-axner]

I'd prefer to break API when merging into v6. We could create a new constructor, but I don't see a reason to maintain it long term, think it'd just add confusion. I think it makes more sense for pre v6 versions to expliticly create their packet data with:

packetData := transfertypes.FungibleTokenPacketData{
    ...
    Memo: memo, 
}

[nicolaslara]

Do we know which versions we're targeting for the backports?

[crodriguezvega]

Do we know which versions we're targeting for the backports?

v2.5.x, v3.4.x, v4.2.x and v5.1.x.

[colin-axner]

closed by #2305