Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Mikhail Melnik <[email protected]>
  • Loading branch information
byshape and ZumZoom authored Feb 27, 2024
1 parent 86f1647 commit 5e8f08a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 3 additions & 1 deletion contracts/extensions/FeeTaker.sol
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,8 @@ contract FeeTaker is IPostInteraction {
SafeERC20.safeTransfer(IERC20(order.takerAsset.get()), feeRecipient, fee);
}

SafeERC20.safeTransfer(IERC20(order.takerAsset.get()), receiver, takingAmount - fee);
unchecked {
SafeERC20.safeTransfer(IERC20(order.takerAsset.get()), receiver, takingAmount - fee);
}
}
}
2 changes: 0 additions & 2 deletions deploy/deploy-FeeTaker.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,12 @@ module.exports = async ({ deployments, getNamedAccounts }) => {

const constructorArgs = [];
const contractName = 'FeeTaker';
const deploymentName = 'FeeTakerDeployment';

await deployAndGetContract({
contractName,
constructorArgs,
deployments,
deployer,
deploymentName,
});
};

Expand Down

0 comments on commit 5e8f08a

Please sign in to comment.