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

Grants Stack Indexer MACIQF migration #569

Merged
merged 2 commits into from
Jun 18, 2024
Merged

Conversation

nijoe1
Copy link

@nijoe1 nijoe1 commented Jun 11, 2024

  • Handled all MACIQF events comprehensively.
  • Introduced two new schemas:
    • MACI signups
    • Private donation messages
  • Maintained the existing indexer functionality, with one modification:
    • Updated the handling of the RecipientStatusUpdated event types to distinguish between them due to the identical event names.

@nijoe1 nijoe1 self-assigned this Jun 11, 2024
@nijoe1
Copy link
Author

nijoe1 commented Jun 11, 2024

On the staging explorer at round/11155111/186, two projects are displayed. However, in the current commit of the indexer, no projects are present. Despite being approved, as verified through GraphQL, these projects are not visible in the explorer. This issue might be due to IPFS failures, as I've observed numerous "failed to parse metadata" errors. I investigated this to ensure that the issue is not solely related to the MACI migration in this PR.

@@ -307,6 +307,38 @@ export async function migrate<T>(db: Kysely<T>, schemaName: string) {
.addUniqueConstraint("unique_v2ProjectId", ["v2ProjectId"])
.execute();

await schema
Copy link
Member

Choose a reason for hiding this comment

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

Could we reuse donation and add a data field ?

@thelostone-mc thelostone-mc changed the base branch from main to MACI June 18, 2024 09:24
@thelostone-mc thelostone-mc merged commit 96a742f into gitcoinco:MACI Jun 18, 2024
1 check failed
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.

None yet

2 participants