Skip to content

Conversation

xin-hedera
Copy link
Contributor

Description:

This PR fixes the fee schedule key bug in topic by id endpoint

  • Show null fee_schedule_key when it's set to an empty key list key to indicate it's cleared
  • Update acceptance test

Related issue(s):

Fixes #11687

Notes for reviewer:

Checklist

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

@xin-hedera xin-hedera requested a review from a team as a code owner July 28, 2025 20:15
@xin-hedera xin-hedera requested a review from filev94 July 28, 2025 20:15
@xin-hedera xin-hedera linked an issue Jul 28, 2025 that may be closed by this pull request
@xin-hedera xin-hedera self-assigned this Jul 28, 2025
@xin-hedera xin-hedera added bug Type: Something isn't working rest-java Area: Java REST API labels Jul 28, 2025
@xin-hedera xin-hedera added this to the 0.136.0 milestone Jul 28, 2025
@lfdt-bot
Copy link

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-32.68% (target: -1.00%) 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (72d066e) 36336 34035 93.67%
Head commit (6ec9617) 59768 (+23432) 36450 (+2415) 60.99% (-32.68%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#11688) 5 5 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@xin-hedera xin-hedera merged commit 5142c76 into main Jul 29, 2025
31 of 33 checks passed
@xin-hedera xin-hedera deleted the 11687-topics-endpoint-shows-empty-list-for-fee_exempt_key_list-and-fee_schedule_key-when-they-are-cleared branch July 29, 2025 13:48
xin-hedera added a commit that referenced this pull request Jul 29, 2025
- Show null fee_schedule_key when it's set to an empty key list key to indicate it's cleared
- Update acceptance test

Signed-off-by: Xin Li <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Type: Something isn't working rest-java Area: Java REST API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Topics endpoint shows empty list for fee_schedule_key when cleared
3 participants