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

Deprecate dynamo store #230

Merged
merged 4 commits into from
Apr 15, 2024
Merged

Deprecate dynamo store #230

merged 4 commits into from
Apr 15, 2024

Conversation

stepandel
Copy link
Contributor

@stepandel stepandel commented Apr 10, 2024

PR-Codex overview

The focus of this PR is to refactor the codebase to use Prisma for database operations and remove DynamoDB dependency.

Detailed summary

  • Added PrismaClient for database operations
  • Refactored functions to use Prisma queries instead of DynamoDB
  • Removed unused imports and DynamoDB related code

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Apr 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agora-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2024 11:50pm
agora-next-ens ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2024 11:50pm
agora-next-etherfi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2024 11:50pm

@stepandel stepandel changed the title Stepan/deprecate dynamo Deprecate dynamo store Apr 10, 2024
@stepandel stepandel removed the request for review from andreitr April 10, 2024 23:42
Copy link
Collaborator

@ferrodri ferrodri left a comment

Choose a reason for hiding this comment

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

Looks great :)

@@ -0,0 +1,73 @@
const { PrismaClient } = require("@prisma/client");
Copy link
Contributor

Choose a reason for hiding this comment

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

@stepandel what's the context in which this is run? this is done as part of a script? is this the script you used to port records from dynamo to pg?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, correct -- this is a stand alone script for uploading delegate statements from a json dump into Postgres

@Flip-Liquid Flip-Liquid changed the base branch from main to staging April 15, 2024 16:46
@Flip-Liquid Flip-Liquid merged commit 22fc964 into staging Apr 15, 2024
5 checks passed
@Flip-Liquid Flip-Liquid deleted the stepan/deprecate-dynamo branch April 15, 2024 16:47
kent pushed a commit that referenced this pull request Aug 30, 2024
* add statement migration script

* deperecate dynamo statement store

* move aws package to dev dependencies
jefag pushed a commit that referenced this pull request Oct 4, 2024
* add statement migration script

* deperecate dynamo statement store

* move aws package to dev dependencies
jefag pushed a commit that referenced this pull request Oct 4, 2024
* add statement migration script

* deperecate dynamo statement store

* move aws package to dev dependencies
jefag pushed a commit that referenced this pull request Oct 8, 2024
* add statement migration script

* deperecate dynamo statement store

* move aws package to dev dependencies
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.

3 participants