chainlink icon indicating copy to clipboard operation
chainlink copied to clipboard

ChainlinkClient: misleading dev comment in ChainlinkClient

Open aelmanaa opened this issue 2 years ago • 0 comments

Hello,

really minor remark but the dev comment states that one has to use sendOperatorRequest instead of sendChainlinkRequest for multi-word responses:

https://github.com/smartcontractkit/chainlink/blob/7d9f6168e6d66538c4d57999b1786d988098fb95/contracts/src/v0.8/ChainlinkClient.sol#L115

but when I check Operator.sol, seems like operatorRequest and oracleRequest functions behave the same. oracleRequest was only added for backward compatibility for consumer contracts that still call oracleRequest function. Hence, I don't see why choosing one over the other can help getting multi-word responses, this could be misleading for developers. Or am I missing something?

aelmanaa avatar May 03 '22 15:05 aelmanaa