From 25e291d9b1903d213829321ade800f306ced2000 Mon Sep 17 00:00:00 2001 From: Matias Date: Wed, 8 Jan 2025 15:02:55 -0300 Subject: [PATCH] fix: cleanup IPaymentCollector and TAPCollector docs --- packages/horizon/contracts/interfaces/IPaymentsCollector.sol | 3 --- .../horizon/contracts/payments/collectors/TAPCollector.sol | 3 ++- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/packages/horizon/contracts/interfaces/IPaymentsCollector.sol b/packages/horizon/contracts/interfaces/IPaymentsCollector.sol index 61854bb71..baaf37722 100644 --- a/packages/horizon/contracts/interfaces/IPaymentsCollector.sol +++ b/packages/horizon/contracts/interfaces/IPaymentsCollector.sol @@ -34,9 +34,6 @@ interface IPaymentsCollector { * @notice Initiate a payment collection through the payments protocol * @dev This function should require the caller to present some form of evidence of the payer's debt to * the receiver. The collector should validate this evidence and, if valid, collect the payment. - * Requirements: - * - The caller must be the data service the RAV was issued to - * - The signer of the RAV must be authorized to sign for the payer * * Emits a {PaymentCollected} event * diff --git a/packages/horizon/contracts/payments/collectors/TAPCollector.sol b/packages/horizon/contracts/payments/collectors/TAPCollector.sol index c8b42b87f..61168d790 100644 --- a/packages/horizon/contracts/payments/collectors/TAPCollector.sol +++ b/packages/horizon/contracts/payments/collectors/TAPCollector.sol @@ -125,7 +125,8 @@ contract TAPCollector is EIP712, GraphDirectory, ITAPCollector { * @notice Initiate a payment collection through the payments protocol * See {IGraphPayments.collect}. * @dev Caller must be the data service the RAV was issued to. - * @dev Service provider must have an active provision with the data service to collect payments + * @dev The signer of the RAV must be authorized to sign for the payer. + * @dev Service provider must have an active provision with the data service to collect payments. * @notice REVERT: This function may revert if ECDSA.recover fails, check ECDSA library for details. */ function collect(IGraphPayments.PaymentTypes paymentType, bytes memory data) external override returns (uint256) {