Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Snapshot subgraph migration quest - Ricqcodes #11

Closed
wants to merge 14 commits into from

Conversation

RicqCodes
Copy link

Hello,

I've successfully completed the task related to the Snapshot migration to the Subsquid SDK as per the project requirements.

Kindly review the changes at your convenience.

Best regards,
Ricqcodes

@dariaag
Copy link
Contributor

dariaag commented Oct 2, 2023

block values are missing from the db and there are decoding errors

@RicqCodes
Copy link
Author

Hi thanks for your comment, the reason why I left out the block values was because it was commented out in the data source as found in the subgraph.yaml file, but it has been included now.

Screenshot 2023-10-02 at 14 16 11

@dariaag
Copy link
Contributor

dariaag commented Oct 2, 2023

still was not able to run the squid. make sure that the processor runs without errors before submitting again.
Issues:

  1. you don't need try/catch for database insertion.
  2. Incorrect store in processor.run. Refer to this doc for what to pass to the processor https://docs.subsquid.io/basics/squid-processor/

@RicqCodes
Copy link
Author

Thank you for your feedback I have been able to remove the try/catch block while also restructuring the processor.run parameters received as shown in the docs.

Below is the process running without error, currently at block 14515339 as at the time of this comment!

Screenshot 2023-10-02 at 18 36 43

@RicqCodes
Copy link
Author

@dariaag is this okay to submit?

@belopash
Copy link
Contributor

belopash commented Oct 5, 2023

closed in favor of #19 (to reduce duplications)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants