-
Notifications
You must be signed in to change notification settings - Fork 108
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
Custom fees REST api #2247
Custom fees REST api #2247
Conversation
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Codecov Report
@@ Coverage Diff @@
## main #2247 +/- ##
============================================
+ Coverage 83.13% 83.27% +0.13%
Complexity 2234 2234
============================================
Files 433 437 +4
Lines 11682 11783 +101
Branches 985 1002 +17
============================================
+ Hits 9712 9812 +100
- Misses 1642 1643 +1
Partials 328 328
Continue to review full report at Codecov.
|
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
logger.error(err); | ||
process.exit(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rethrow will cause unhandled rejection
warning and the nodejs process hanging there.
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
'maximum_amount', ${CustomFee.MAXIMUM_AMOUNT}::text, | ||
'minimum_amount', ${CustomFee.MINIMUM_AMOUNT}::text, | ||
'token_id', ${CustomFee.TOKEN_ID}::text | ||
) order by ${CustomFee.AMOUNT}, ${CustomFee.COLLECTOR_ACCOUNT_ID}, ${CustomFee.DENOMINATING_TOKEN_ID}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
order the custom fees so integration tests can pass. it's interesting that the same data inserted in the same order, vanilla pg and timescaledb return them in opposite orders.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably cause of the extra index in the timescale case
hedera-mirror-rest/__tests__/specs/token-info-05-custom-fees.spec.json
Outdated
Show resolved
Hide resolved
hedera-mirror-rest/__tests__/specs/token-info-05-custom-fees.spec.json
Outdated
Show resolved
Hide resolved
hedera-mirror-rest/__tests__/specs/token-info-05-custom-fees.spec.json
Outdated
Show resolved
Hide resolved
Signed-off-by: Xin Li <xin.li@hedera.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
SonarCloud Quality Gate failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Add assessed custom fees to /api/v1/transactions/:transactionId endpoint - Add custom fees schedule to/api/v1/tokens/:tokenId endpoint - Add model/viewmodel for assessed custom fees and custom fees schedule - Add integration domain support for assessed custom fees and custom fees schedule - Update openapi spec - Fix EntityId throwing exception with nullable undefined value - Move queryQuietly to pool class prototype - Add/Update integration tests Signed-off-by: Xin Li <xin.li@hedera.com> Signed-off-by: Ian Jungmann <ian.jungmann@hedera.com>
Description:
This PR adds HTS custom fees support to the mirror node REST API.
/api/v1/transactions/:transactionId
endpoint/api/v1/tokens/:tokenId
endpointundefined
valuequeryQuietly
to pool class prototypeRelated issue(s):
Fixes #2199
Notes for reviewer:
Checklist