Merge pull request #106 from ArnavK-09/patch-1

patch(modelsRestApiClient.ts): type conversion for params
mindsdb · Nov 13, 2024 · 16d40aa · 16d40aa
2 parents d7e7dbe + 2d6b149
commit 16d40aa
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/models/modelsRestApiClient.ts b/src/models/modelsRestApiClient.ts
@@ -110,7 +110,7 @@ export default class ModelsRestApiClient extends ModelsApiClient {
     return whereClause;
   }
 
-  private makeUsingClause (using: string | Array<string>): string {
+  private makeUsingClause (using: string | Array<string>): string {
     if (!using) {
       return '';
     }
@@ -507,7 +507,7 @@ export default class ModelsRestApiClient extends ModelsApiClient {
   ): Promise<void> {
     const deleteQuery = `DROP MODEL ${mysql.escapeId(project)}.${mysql.escapeId(
       model,
-    )}.${mysql.escapeId(v)}`;
+    )}.${mysql.escapeId(v.toString())}`;
     const sqlQueryResult = await this.sqlClient.runQuery(deleteQuery);
     if (sqlQueryResult.error_message) {
       throw new MindsDbError(sqlQueryResult.error_message);
@@ -534,4 +534,4 @@ export default class ModelsRestApiClient extends ModelsApiClient {
             new ModelVersion(project, { ...model, version: v })),
       );
   }
-}
+}