Commit bac72c0

Add ERC: Token Puller (#1587) * Token Puller Interface New ERC that proposed a standardized way of pulling funds from an account, decoupling spending from asset management strategies. * Some fixes and additions * Discussion link + Fixes based on feedback 1. Added discussing link 2. Use of ERC-5267 for EIP-712 domain instead of DOMAIN_SEPARATOR() (thanks @amxx) 3. Remove documentation of nonces(), just reference existing ERCs. * EIP number assigned + title changed * CI Linter fixes + Security Considerations section * docs(erc-8187): add reference implementation link * Revision fixes: linting, removal of product references, rational->motivation, others - Markdown linting - External links (even to github) are not allowed, so I changed it for backticks. - Code in backticks (even in headers) - In pullFrom, specify only the tokens must be trasferred, without indicating that must be an ERC-20 transfer. That wasn't needed and might confuse that always the ERC20 transfer should come from the puller when in many cases can be sent directly by the source. - Removed references to specific products in different parts of the ERC. - Moved a paragraph that was in the Rational section that belongs to Motivation section. - Removed some not so relevant points in the abstract, making it shorter * Fix: fixes EIP-validator errors * Fix: several style and grammar issues of @jochem-brouwer revision * Fix: reference implementation + domain separator - Adds the reference implementation to this repository, under assets/erc-8187. - Fix the digest computation documentation, replicating the description used in ERC-2612. * Fix HTML link * Fix: ERC-8187 revision fixes from @jochem-brouwer review - Add ERC-1271 to requires - Clarify permitPull event emission requirements and allow non-spender callers - Drop redundant infinite-allowance check in transferPullAllowance - Point @custom:specification links to eips.ethereum.org - Fix grammar and style issues across spec, README, and contracts * Fix duplicated MUST clause

GitHub Diff