Skip to content

Breaking change: Require Redis (for v1.0.0) #550

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

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Jun 21, 2024

  • BREAKING CHANGE: Require REDIS_URL env var for v1.0.0 release. Engine will not start if this env var is unavailable.
  • Update examples from Mumbai (deprecated) -> Amoy
  • Fix incorrect tx status code (e.g. from reverted execution)

PR-Codex overview

The focus of this PR is to update chain IDs and names to use "80002" and "Polygon Amoy Testnet" respectively across various files.

Detailed summary

  • Updated chain IDs and names to "80002" and "Polygon Amoy Testnet" respectively.
  • Changed status check in processTransactionReceiptsWorker.ts to use "success" string comparison.
  • Modified redis.ts to export a constant redis and handle error, connect, and reconnect events.
  • Adjusted various schemas to reflect the new chain ID and name "80002" and "Polygon Amoy Testnet".

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

@arcoraven arcoraven merged commit d0f508a into main Jun 21, 2024
4 checks passed
@arcoraven arcoraven deleted the ph/breakingChangeRequireRedis branch June 21, 2024 00:22
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.

2 participants