For OrderRecord event 'receiver' is missing

[Umiiii]

WEB3-DEX/contracts/8/DexRouter.sol

Line 397 in 556df81

emit OrderRecord(

    function _smartSwapInternal(
        BaseRequest memory baseRequest,
        uint256[] memory batchesAmount,
        RouterPath[][] memory batches,
        address payer,
        address refundTo,
        address receiver
    ) private returns (uint256 returnAmount) {
    
        emit OrderRecord(
            fromToken,
            _baseRequest.toToken,
            tx.origin,
 -->      
            _baseRequest.fromTokenAmount,
          returnAmount
        );
        return returnAmount;
    }

Since OKX DEX support swapReceiverAddress, and the contract allows different receiver, it would be better if we can emit the receiver.

[bixia]

yes, it make sense. we will consider add this in our next release.

[clar]

good news!

[akira02]

This is an excellent suggestion. I fully endorse it and hope it gets considered.
This feature would be very helpful for devs.