-
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
Deduplicate NFT transfers #2164
Conversation
Signed-off-by: Nana-EC <nana.essilfie-conduah@hedera.com>
Codecov Report
@@ Coverage Diff @@
## main #2164 +/- ##
=======================================
Coverage ? 81.72%
Complexity ? 2135
=======================================
Files ? 412
Lines ? 10982
Branches ? 931
=======================================
Hits ? 8975
Misses ? 1695
Partials ? 312 Continue to review full report at Codecov.
|
Signed-off-by: Nana-EC <nana.essilfie-conduah@hedera.com>
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
* Deduplicate nft trasnfers list A duplicate nft transfer entry exists when the /api/v1/transactions?account.id={account} is called and the account in question is the recipient. - Update aggregation to use `distinct jsonb_build_object` - Add an extra test for multiple nft transfers across different tokens in one transaction Signed-off-by: Nana-EC <nana.essilfie-conduah@hedera.com>
* Deduplicate nft trasnfers list Cherry pick Deduplicate nft trasnfers list A duplicate nft transfer entry exists when the /api/v1/transactions?account.id={account} is called and the account in question is the recipient. - Update aggregation to use `distinct jsonb_build_object` - Add an extra test for multiple nft transfers across different tokens in one transaction Signed-off-by: Nana-EC <nana.essilfie-conduah@hedera.com>
Detailed description:
A duplicate nft transfer entry exists when the /api/v1/transactions?account.id={account} is called and the account in question is the recipient.
distinct jsonb_build_object
Which issue(s) this PR fixes:
Fixes #2139
Special notes for your reviewer:
Postgres doesn't support distinct and order by for json. This should be okay though as the rows are mostly ordered prior to aggregation
Checklist