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: transfer multiple tokens at once (#200) #205

Closed
wants to merge 9 commits into from

Conversation

arianejasuwienas
Copy link
Contributor

Description:
Remvoed from 151 for investigation.

Related issue(s):

#200

Notes for reviewer:

Checklist

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

…okenService interface (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
…okenService interface (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
…okenService interface (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
…okenService interface (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
…methods order, moving 22 const to lib, removing redundant comment (#151)

Signed-off-by: Mariusz Jasuwienas <[email protected]>
Signed-off-by: Mariusz Jasuwienas <[email protected]>
Signed-off-by: Mariusz Jasuwienas <[email protected]>
@arianejasuwienas
Copy link
Contributor Author

@acuarica, it seems my initial implementation of this method was indeed not aligned with the implementation proposed in SmartContracts services. I was misled by the method's description... The solution from this PR should work as expected.

@arianejasuwienas arianejasuwienas deleted the 200-transfer-multiple-tokens-at-once branch January 20, 2025 15:23
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