Skip to content

Commit

Permalink
fix: explanation on requestRedeem
Browse files Browse the repository at this point in the history
  • Loading branch information
0xTimepunk committed Jun 21, 2024
1 parent 8e50fe5 commit d8bffce
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ERCS/erc-7540.md
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ The output `requestId` is used to discriminate the request along with the `contr

In either case, the `shares` MUST be removed from the custody of `owner` upon `requestRedeem` and burned by the time the request is Claimed.

Redeem Request approval of `shares` for a `msg.sender` NOT equal to `owner` may come either from ERC-20 approval over the `shares` of `owner` or if the `owner` has approved the `msg.sender` as an operator.
Redeem Request approval of `shares` for a `msg.sender` NOT equal to `owner` may come either from ERC-20 approval over the `shares` of `owner` or if the `owner` has approved the `msg.sender` as an operator. This MUST be consistent with similar behaviour pointed out in [ERC-6909](./eip-6909.md), within "Approvals and Operators" section: "In accordance with the transferFrom method, spenders with operator permission are not subject to allowance restrictions, spenders with infinite approvals SHOULD NOT have their allowance deducted on delegated transfers, but spenders with non-infinite approvals MUST have their balance deducted on delegated transfers."

When the Request is Claimable, `claimableRedeemRequest` will be increased for the `controller`. `redeem` or `withdraw` can subsequently be called by `controller` to receive `assets`. A Request MAY transition straight to Claimable state but MUST NOT skip the Claimable state.

Expand Down

0 comments on commit d8bffce

Please sign in to comment.