Skip to content
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

Feat: Parse events #21

Merged
merged 10 commits into from
Apr 17, 2024
Merged

Feat: Parse events #21

merged 10 commits into from
Apr 17, 2024

Conversation

tmrdlt
Copy link
Member

@tmrdlt tmrdlt commented Apr 4, 2024

  • Parse relevant events based on log.event_hash and known event hashes (same for testnet and mainnet)
  • Get reserve0, reserve1, deltaReserve0, deltaReserve1 from log.data or log.args
  • Merge Sync events with succeeding non-Sync events
  • Fixed BigInts in database (use psql Decimal)
  • Fixed sorting of events by utilizing transactionIndex
  • Added @typescript-eslint/strict-boolean-expressions rule and fixed problems

@tmrdlt tmrdlt self-assigned this Apr 4, 2024
@tmrdlt tmrdlt force-pushed the feat/parse-events branch from 0d655e0 to 2abcd37 Compare April 15, 2024 09:23
Base automatically changed from feat/event-based-history to feat/grant-2 April 15, 2024 15:41
@tmrdlt tmrdlt force-pushed the feat/parse-events branch from 7e67532 to 84668db Compare April 15, 2024 15:41
@tmrdlt tmrdlt force-pushed the feat/parse-events branch from 84668db to 7bb89f3 Compare April 15, 2024 15:46
@tmrdlt tmrdlt marked this pull request as ready for review April 16, 2024 10:11
@tmrdlt tmrdlt requested a review from kenodressel April 16, 2024 10:11
@tmrdlt tmrdlt requested a review from kenodressel April 17, 2024 09:32
@tmrdlt tmrdlt merged commit 9d9e2a7 into feat/grant-2 Apr 17, 2024
2 checks passed
@tmrdlt tmrdlt deleted the feat/parse-events branch April 17, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants