Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Store tx receipts #29

Merged
merged 33 commits into from
Jul 11, 2023
Merged

Store tx receipts #29

merged 33 commits into from
Jul 11, 2023

Conversation

fkrause98
Copy link
Contributor

@fkrause98 fkrause98 commented Jul 10, 2023 •
edited
Loading

Changes:

  • Make the block fetcher also store transaction receipts.
  • Update tests to check stored transaction and receipts validity against RPC data.

fkrause98 and others added 27 commits July 4, 2023 18:57
@fkrause98
Add postgres yml file
09b3ed5
@fkrause98
Add draft GenServer and block migration
2e2cc32
@fkrause98
Add tx migration, schema and block schema
9765ea0
@fkrause98
Working commit
167f1ce
@fkrause98
Update blocks and transactions schemas
61acd85
@fkrause98
Use BlockFetcher only in Prod
ea80955
@fkrause98
Move block schema, update some fetcher code, mix format
3d5313a
@fkrause98
Update tx fields
6590ea9
@fkrause98
Lower fetch interval
0637600
@fkrause98
Mix format + update tx fields
42eef27
@fkrause98
Remove unnecesary comment
87eb1d2
@fkrause98
Add block fetcher tests
f05f03f
@fkrause98
Add postgres to CI job
f509beb
@fkrause98
Format
acc219a
@fkrause98
Add infura api key secret read
a7be459
@fkrause98
Undo wrong template change
9fdc79e
@fkrause98
Format
05d056a
@fkrause98
Update makefile and readme
39eaa51
@fkrause98
Fix CI
5568206
@fkrause98
Add seed
a4ec4a9
@fkrause98
Update block index to show DB data
b4b8593
@fkrause98
Make block fetching take transaction receipts
5f2d956
@fkrause98
Update function comment
cd52b81
@fkrause98
Merge branch 'main' into show-db-data
0426e62
@fkrause98
Add create-seed make target
cc28961
@fkrause98
Delete seed.sql
a26d3c7
@fkrause98
Format
6295772
@fkrause98 fkrause98 marked this pull request as ready for review July 10, 2023 20:51
@jrchatruc
Copy link
Member

Both the home and blocks pages are breaking for me in this PR

@fkrause98
Copy link
Contributor Author

fkrause98 commented Jul 11, 2023 •
edited
Loading

@jrchatruc

You're right, this is on me. This PR changes were originally from #28 and I left some things here which broke the pages you've mentioned. I've double checked and they should be working now.

@jrchatruc jrchatruc merged commit d32cbb2 into main Jul 11, 2023
1 check passed
@jrchatruc jrchatruc deleted the store-tx-receipts branch July 11, 2023 21:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jrchatruc jrchatruc jrchatruc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fkrause98 @jrchatruc