Skip to content

chore: fix specification of hook execution order - #1338

Merged
mgarbs merged 1 commit into
hiero-ledger:mainfrom
tinker-michaelj:fix-hook-invocation-order
Nov 13, 2025
Merged

chore: fix specification of hook execution order#1338
mgarbs merged 1 commit into
hiero-ledger:mainfrom
tinker-michaelj:fix-hook-invocation-order

Conversation

@tinker-michaelj

@tinker-michaelj tinker-michaelj commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

Description:

  • The previous text claimed hook calls for fungible transfers always executed before hook calls for NFT transfers; this does not match the implementation, which simply goes in the order of the TokenTransferLists as they appear in the tx body.

@tinker-michaelj
Fix spec of hook execution order
8d5ed20 
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
@tinker-michaelj
tinker-michaelj requested a review from a team as a code owner November 12, 2025 22:38
@mgarbs
mgarbs merged commit 38d712c into hiero-ledger:main Nov 13, 2025
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mgarbs mgarbs mgarbs approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tinker-michaelj @mgarbs