-
Notifications
You must be signed in to change notification settings - Fork 0
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
58 setup queries for nabla backend for campaign and mainnet reward program #62
58 setup queries for nabla backend for campaign and mainnet reward program #62
Conversation
…m/pendulum-chain/pendulum-squids into 58-setup-queries-for-nabla-backend-for-campaign-and-mainnet-reward-program
This reverts commit 4ad7c45.
@pendulum-chain/devs can we please review this? it would unblock the nabla testnet campaign |
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.
Just one minor comment. I already approve and leave it to you to change the name if this is low effort.
I'm working on this as we speak, will push the latest changes requested in the next few hours |
Thank you Bogdan for the update |
This fixes the errors in the console by setting the reference to |
…m/pendulum-chain/pendulum-squids into 58-setup-queries-for-nabla-backend-for-campaign-and-mainnet-reward-program
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.
Can confirm that the error messages don't appear anymore during the processing, good job @bogdanS98 👍
We should change the schema so that the deposit and withdrawal events link to either the respective swap or backstop pool. Otherwise, we are missing very important information and the events are useless.
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
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 too 👍
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.
Let's merge @bogdanS98?
Closes #58.
This should be merged to
main
only after #52 is merged.Tested some of these changes locally by running the nabla indexer tests from this
wasm-deploy
branch, therefore generating some events data. This is not enough, so more events must be generated in order to test all queries.