Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🚨ID-115 Informational: Incorrect Type #205

Open
1 task done
SamGrantDev opened this issue Jan 21, 2025 · 0 comments
Open
1 task done

🚨ID-115 Informational: Incorrect Type #205

SamGrantDev opened this issue Jan 21, 2025 · 0 comments

Comments

@SamGrantDev
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Task description 🔨🔧

Description

The amount field of FungibleTokenPacketData is defined as a ByteArray instead of an integer as per ICS-20. This discrepancy:

  • Results in costly bytearray-to-integer conversions.
  • Risks decoding issues due to differing encoding schemes.

Recommendation

  • Update the amount field to conform to ICS-20 by making it an integer.

Anything else?

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant