A. McDermott
A. McDermott
@SPIKESPIGEL404 The last 2 look good! But _all_ of the PR's commits need to be signed and verified. The very first one is unsigned rn. You can do a rebase...
saddest rebase in the universe. new branch here: https://github.com/coinbase/x402/pull/600
@emiliolanzalaco ERC-6492 signatures will pass verification off-chain, yes. However, when you try to settle a 3009 transfer using an undeployed smart wallet, the settlement will inevitably fail. When the facilitator...
@emiliolanzalaco Related PR that moves the check out of verify and into settle. Verify will still fail for undeployed wallets that aren't 6492 wrapped, since there's no path to success:...
Fully in agreement here. Making the paywall modular is [already being tackled](https://github.com/coinbase/x402/pull/600) as part of v2 👍
@iolloyd LGTM but you need verified signatures on your commits
Your changes have been incorporated into https://github.com/coinbase/x402/pull/705 🙌