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

feat: methods for transfer association and approval from the IHTS #188

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

arianejasuwienas
Copy link
Contributor

@arianejasuwienas arianejasuwienas commented Jan 14, 2025

Description:

Adding the implementation of the methods from the IHederaTokenService interface. I will allow users to perform token-based operations by interacting with the HTS contract at the 0x167 address. The token-based operations introduced in this PR were already implemented on the token side through interfaces compatible with IERC721, IERC20, and IHRC719, but they have now been extended to support HTS calls as well.

For example, in addition to transferring a fungible token using the IERC20 interface:

IERC20(token).transferFrom(sender, recipient, amount);  

you can now perform the same operation via the HTS interface:

IHederaTokenService(address(0x167)).transferToken(token, sender, recipient, amount);

Related issue(s):

#151

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

…okenService interface (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
@arianejasuwienas arianejasuwienas force-pushed the 151-hts-methods-implementation branch from b8cdcce to f28b6ef Compare January 15, 2025 09:45
@arianejasuwienas arianejasuwienas changed the title 151 hts methods implementation feat: methods for transfer association and approval from the IHTS Jan 15, 2025
@arianejasuwienas arianejasuwienas marked this pull request as ready for review January 15, 2025 10:41
@arianejasuwienas arianejasuwienas requested a review from a team as a code owner January 15, 2025 10:41
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

Successfully merging this pull request may close these issues.

1 participant