Breaking change: Require Redis (for v1.0.0) - Githubissues

thirdweb-dev / engine

The most powerful backend engine for web3 apps.

https://thirdweb.com/engine

Apache License 2.0

143 stars 52 forks source link

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

Closed arcoraven closed 3 months ago

arcoraven commented 3 months ago

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}