daimo-eth / daimo

Real world Ethereum
https://daimo.com
GNU General Public License v3.0
358 stars 29 forks source link

api,contract: remove ethIndexer, use eth_transfers #1179

Closed dcposch closed 3 months ago

dcposch commented 3 months ago

Summary

Replaces our aggregated balance-based ETH inbox.

This simplifies a lot of code. We handle ETH symmetrically with other tokens now.

Deployment

The CREATE MATERIALIZED VIEW for this took almost 2h on staging.

image
vercel[bot] commented 3 months ago

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

Name Status Preview Comments Updated (UTC)
daimo-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2024 1:09pm
daimo-web-stage ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2024 1:09pm
dcposch commented 3 months ago

Shipping once deployment succeeds.

dcposch commented 3 months ago

eth_transfers works

actual swap call reverted https://jiffyscan.xyz/userOpHash/0x118e0edd5d69cd3722fb33bc2c2698b91b72a60e82dc2563a8c3a3a1d5f3ade6?network=base&section=logs

Screenshot 2024-06-27 at 18 12 23 Screenshot 2024-06-27 at 18 12 29 Screenshot 2024-06-27 at 18 12 31 Screenshot 2024-06-27 at 18 12 37
socket-security[bot] commented 3 months ago

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@vercel/og@0.6.2 environment, filesystem, network +1 7.64 MB vercel-release-bot
npm/@wagmi/cli@1.5.2 environment, filesystem Transitive: network +15 484 kB awkweb

View full report↗︎