-
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
Fix incorrect fixed fee denominating token id #2358
Conversation
Signed-off-by: Xin Li <xin.li@hedera.com>
Codecov Report
@@ Coverage Diff @@
## main #2358 +/- ##
=========================================
Coverage 83.59% 83.59%
- Complexity 2257 2259 +2
=========================================
Files 437 437
Lines 11961 11963 +2
Branches 1018 1020 +2
=========================================
+ Hits 9999 10001 +2
Misses 1647 1647
Partials 315 315
Continue to review full report at Codecov.
|
.../src/main/java/com/hedera/mirror/importer/parser/record/entity/EntityRecordItemListener.java
Show resolved
Hide resolved
.../src/main/java/com/hedera/mirror/importer/parser/record/entity/EntityRecordItemListener.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Xin Li <xin.li@hedera.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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
- Set the denominating token id of a fixed fee to the attached token id when it's the sentinel token 0.0.0 Signed-off-by: Xin Li <xin.li@hedera.com>
- Set the denominating token id of a fixed fee to the attached token id when it's the sentinel token 0.0.0 Signed-off-by: Xin Li <xin.li@hedera.com>
Description:
0.0.0
Related issue(s):
Fixes #2357
Notes for reviewer:
Please refer to the ticket for the context.
Checklist