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

Bring local graphql-engine version into sync with production / latest version #1275

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

frankhereford
Copy link
Member

@frankhereford frankhereford commented Aug 25, 2023

Associated issues

This issue is the root cause of this change, but does not directly address it.

This is a change that we would normally do during a release party directly into master, but due to our recent upgrade to the graqphql-engine middleware, this is needed to spin up the local stack with a replicated or copied database from production.

Testing

Steps to test:

  • Spin down your graphql-engine endpoint locally
  • Replicate the production database into your local database via a backup fround on S3 or pulled directly from the read replica. ./vision-zero replicate-db
  • Check to see if your local graphql-engine starts up

Ship list

  • Code reviewed
  • Product manager approved

Copy link
Contributor

@mddilley mddilley left a comment

Choose a reason for hiding this comment

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

This lines up with the version that I used to init the migrations so I'm approving based on that testing. Thanks for the cleaning up the comments too! ✨

Copy link
Member

@patrickm02L patrickm02L left a comment

Choose a reason for hiding this comment

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

Good to go 🚢

@frankhereford frankhereford merged commit 434e384 into master Aug 28, 2023
10 checks passed
@frankhereford frankhereford deleted the bump-local-graphql-engine branch August 28, 2023 14:54
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.

4 participants