feat(account): add initial hook API support (#65)#72
feat(account): add initial hook API support (#65)#72amkr6207 wants to merge 3 commits intohiero-ledger:mainfrom
Conversation
|
@Ndacyayisenga-droid, @hendrikebbers, @manishdait CI failure looks unrelated to this PR code.
This happened before Maven verify, so it looks like a flaky infra/setup issue in the Solo action step. Could you please re-run the workflow? |
|
After re-run, CI has passed, so this looks flaky. Small suggestion: to reduce flaky CI runs, should we consider removing the explicit |
Thanks for that observation, We should actually unpin it and we update to the latest version of the solo-action |
Applied your feedback on CI config in this PR:
Thanks! |
|
CI was repeatedly failing in I added a small test-stability fix:
This addresses the flaky mirror-node empty-response case observed in Solo-based CI runs. Latest CI re-run is now passing. Thanks! |
4b32b93 to
115efd6
Compare
|
Hi @Ndacyayisenga-droid, all PR commits are now GPG + DCO signed. Could you please re-check? |
Signed-off-by: Aman <amkr6207@gmail.com>
Signed-off-by: Aman <amkr6207@gmail.com>
… response Signed-off-by: Aman <amkr6207@gmail.com>
115efd6 to
0fd5d4e
Compare
Description
This is PR 1 for #65.
It adds base account hook API/contracts in
hiero-enterprise-baseonly (no implementation yet).HookExtensionPointandHookDetailsAccountHookUpdateRequestandAccountHookUpdateResultaddHook,deleteHook, andupdateHookstoAccountClientexecuteAccountHookUpdateTransaction(...)toProtocolLayerClientRelated issue
Part of #65
Notes for reviewer
This PR only adds API interfaces and models; implementation will come in follow-up PRs.
Checklist
./mvnw -pl hiero-enterprise-base -DskipTests compile)