0xProject / 0x-monorepo

0x protocol monorepo - includes our smart contracts and many developer tools
Other
1.41k stars 464 forks source link

[WIP] add LibTokenSpender and convert to using that #2734

Closed smarx closed 3 years ago

smarx commented 3 years ago

Description

For gas savings, we now skip the allowance target altogether. Allowances should instead be set directly on the exchange proxy itself. There is a fallback, though, to try spending from the allowance target if the original transfer fails.

Testing instructions

yarn build && yarn test from contracts/zero-ex

Types of changes

Checklist: