From 08b375f39651bceb66319db33713b80ecdc9a18b Mon Sep 17 00:00:00 2001
From: Velenir <velenir@gmail.com>
Date: Mon, 15 Apr 2024 17:29:32 +0200
Subject: [PATCH 1/3] LOrders/getRequiredBalance filter

---
 src/methods/limitOrders/getOrders.ts | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/methods/limitOrders/getOrders.ts b/src/methods/limitOrders/getOrders.ts
index 426ea6ced..ad3b672e1 100644
--- a/src/methods/limitOrders/getOrders.ts
+++ b/src/methods/limitOrders/getOrders.ts
@@ -32,6 +32,7 @@ export type LimitOrdersUserParams = (
 export type GetRequiredAllowanceParams = {
   maker: Address;
   token?: Address;
+  filter?: { taker?: Address; notTaker?: Address; type?: LimitOrderType };
 };
 
 type GetLimitOrderByHash = (
@@ -57,6 +58,7 @@ export type GetLimitOrdersFunctions = {
    * @param userParams - parameters to get allowance for active orders
    * @param {string} userParams.maker - user to get required allowances for
    * @param {string=} userParams.token - if given `token`, the mapping will contain that token address only
+   * @param {{taker?:Address;notTaker?:Address;type?:LimitOrderType}=} userParams.filter - extra filter to get balance required for type=P2P or type=LIMIT orders separately
    * @param {AbortSignal=} signal - AbortSignal passed to fetcher
    * @returns `{Lowercase<Address> => wei number as string}` mapping of token to fillableBalance
    */
@@ -103,10 +105,16 @@ export const constructGetLimitOrders = ({
   const getRequiredBalance: GetRequiredBalance = async (userParams, signal) => {
     const baseFetchURL = getBaseFetchURLByEntityType('fillablebalance');
     const userURL = `${baseFetchURL}/${userParams.maker}` as const;
-    const fetchURL = userParams.token
+    let routeURL = userParams.token
       ? (`${userURL}/${userParams.token}` as const)
       : userURL;
 
+    const search = constructSearchString<
+      NonNullable<GetRequiredAllowanceParams['filter']>
+    >(userParams.filter || {});
+
+    const fetchURL = `${routeURL}${search}` as const;
+
     const response = await fetcher<
       Record<string, string>,
       GetRequiredBalanceURL

From a3ac77c678e8e2b66ee92ab122fd5de4b1e83b50 Mon Sep 17 00:00:00 2001
From: Velenir <velenir@gmail.com>
Date: Mon, 15 Apr 2024 17:30:13 +0200
Subject: [PATCH 2/3] LOrders/buildOrderData build OTC with taker=EOA only

---
 src/methods/limitOrders/helpers/buildOrderData.ts | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/methods/limitOrders/helpers/buildOrderData.ts b/src/methods/limitOrders/helpers/buildOrderData.ts
index f1ff2688d..60eddf009 100644
--- a/src/methods/limitOrders/helpers/buildOrderData.ts
+++ b/src/methods/limitOrders/helpers/buildOrderData.ts
@@ -90,14 +90,9 @@ export function buildOrderData({
     taker = ZERO_ADDRESS;
   } else {
     // otherwise for p2p order ->
-    if (AppVersion === '6') {
-      // limit taker to EOA for v6 version (no Arbitrary Token Swaps + OTC Fill, or OTC Fill through AugustusSwapper)
-      taker = takerInNonce;
-    } else {
-      // on v5
-      // -> fill through Augustus only
-      taker = AugustusAddress;
-    }
+    // all new P2P Orders only fillable directly by EOA taker
+    // not through Augustus, this also disables BUY+FILL Order flow
+    taker = takerInNonce;
   }
 
   const order: OrderData = {

From 5a3644406f0cfc9eb9f7cd2e48d714dabc66e2c6 Mon Sep 17 00:00:00 2001
From: Velenir <velenir@gmail.com>
Date: Mon, 15 Apr 2024 17:31:52 +0200
Subject: [PATCH 3/3] LOrders/approveTakerTokenForLimitOrder/deprecate for OTC
 Orders

---
 src/methods/limitOrders/approveForOrder.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/methods/limitOrders/approveForOrder.ts b/src/methods/limitOrders/approveForOrder.ts
index c80650b07..128351ec6 100644
--- a/src/methods/limitOrders/approveForOrder.ts
+++ b/src/methods/limitOrders/approveForOrder.ts
@@ -7,6 +7,10 @@ export type ApproveTokenForLimitOrderFunctions<T> = {
   /** @description approving AugustusRFQ as spender for makerAsset */
   approveMakerTokenForLimitOrder: ApproveToken<T>;
   /** @description approving AugustusSwapper as spender for takerAsset for Limit Orders that will be executed through it */
+  /** @deprecated for OTC Orders approve directly the contract that the Order will be filled through:
+   * AugustusSwapper for old P2P orders;
+   * AugustusRFQ for new P2P orders
+   */
   approveTakerTokenForLimitOrder: ApproveToken<T>;
 };