audit: remove payable from vault functions (#413)

Author: keyleu

solidity/src/vaults/OneWayVault.sol

@@ -586,12 +586,7 @@ contract OneWayVault is
      * @param receiver Address to receive the withdrawn assets on the destination domain (as string)
      * @param owner Address that owns the shares
      */
-    function withdraw(uint256 assets, string calldata receiver, address owner)
-        public
-        payable
-        nonReentrant
-        whenNotPaused
-    {
+    function withdraw(uint256 assets, string calldata receiver, address owner) public nonReentrant whenNotPaused {
         if (_checkAndHandleStaleRate()) {
             return; // Exit early if vault was just paused
         }
@@ -629,12 +624,7 @@ contract OneWayVault is
      * @param receiver Address to receive the redeemed assets on destination domain (as string)
      * @param owner Address that owns the shares
      */
-    function redeem(uint256 shares, string calldata receiver, address owner)
-        public
-        payable
-        nonReentrant
-        whenNotPaused
-    {
+    function redeem(uint256 shares, string calldata receiver, address owner) public nonReentrant whenNotPaused {
         if (_checkAndHandleStaleRate()) {
             return; // Exit early if vault was just paused
         }