Increase MaxMemoCharLength in ICA to a higher number than 256.

[puneet2019]

Summary

Increase MaxMemoCharLength in ICA packet data to match Transfer memo length.

Problem Definition

With coming of callbacks, memo in ICA Packet too can be used to trigger callbacks, and encoding of json might easily exceed the current limit of 256 chars

Proposal

We should take into account the memos that Transfers are using in various routers, and apply it to ICA too. 1024 seems to be a good number

---

For Admin Use

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

[crodriguezvega]

Thanks for opening the issue, @puneet2019! This sounds good to me. Maybe we could even use the same limit as the one we use for the memo field of transfer (32768).

[chandiniv1]

@crodriguezvega can I work on this issue?

[puneet2019]

@crodriguezvega Ah! thanks, I could not find memo length var before, sounds good to me :) thank you!
@chandiniv1 go ahead! reference this Issue to the PR