Fixes for: multiple NFTs claim and partial withdrawal connection history

dcSpark / carp

A modular indexer for Cardano with an SQL Postgres backend (an alternative to the well known cardano-db-sync)

MIT License

105 stars 22 forks source link

Closed gostkin closed 11 months ago

gostkin commented 11 months ago

While adding paima funnel 2 bugs were found:

if claimed projected nft had more than 1 asset then only first one was saved as event
if a locked asset is locked based on partial withdrawal, previous lock wasn't linked to the new one. so it was impossible to identify which lock created a new one and update the state. in this pr partial withdrawals are linked with subsequent locks

SebastienGllmt commented 11 months ago

See comment I made about Javascript integer precision here and make sure we didn't make the same mistake on the Carp client

I approved the PR since everything other than that seemed ok