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

🔧 snekk configs #171

Merged
merged 1 commit into from
Apr 18, 2023
Merged

🔧 snekk configs #171

merged 1 commit into from
Apr 18, 2023

Conversation

vikiival
Copy link
Member

Thank you for your contribution to the KodaDot Indexer.
👇 _ Let's make a quick check before the contribution.

PR type

  • Bugfix
  • Feature
  • Refactoring

What's new?

  • PR closes #<issue_number>
  • This PR is required for kodadot/nft-gallery#<issue_number>

Before submitting Pull Request, please make sure:

  • My contribution builds clean without any errors or warnings
  • I've merged recent default branch -- main and I've no conflicts
  • I've tried to respect high code quality standards
  • I've didn't break any original functionality
  • I've posted a screenshot of demonstrated change in this PR

Optional

  • I've run the indexer and it hasn't failed
  • I've changed schema and performed a migration
  • I found edge cases

Screenshot

  • My contribution is a resolver so I have pic from playground

@vikiival vikiival merged commit edc3a89 into main Apr 18, 2023
@vikiival vikiival deleted the squid-manifest branch April 18, 2023 12:57
Copy link
Contributor

@roiLeo roiLeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is it for?

@@ -0,0 +1,17 @@
manifestVersion: subsquid.io/v0.1
name: sneck
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sneck?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes since we are using new deployment manifest I need to have different names for each indexer

I was thinking

  • sneck fror snek
  • basilick for bsx

@vikiival
Copy link
Member Author

What is it for?

The update script we used is deprecated

and we have to use

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.

2 participants