From 83d20e70cb343a3c191b7be8ddc610f3d98265e4 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Fri, 17 Feb 2023 10:09:02 +0300
Subject: [PATCH 01/25] fix: sql client type calls

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 packages/nocodb/src/lib/meta/api/tableApis.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/nocodb/src/lib/meta/api/tableApis.ts b/packages/nocodb/src/lib/meta/api/tableApis.ts
index 8eb3d6a10e..8e02d5aca8 100644
--- a/packages/nocodb/src/lib/meta/api/tableApis.ts
+++ b/packages/nocodb/src/lib/meta/api/tableApis.ts
@@ -151,7 +151,7 @@ export async function tableCreate(req: Request<any, any, TableReqType>, res) {
   const sqlClient = await NcConnectionMgrv2.getSqlClient(base);
 
   let tableNameLengthLimit = 255;
-  const sqlClientType = sqlClient.clientType;
+  const sqlClientType = sqlClient.knex.clientType();
   if (sqlClientType === 'mysql2' || sqlClientType === 'mysql') {
     tableNameLengthLimit = 64;
   } else if (sqlClientType === 'pg') {
@@ -298,7 +298,7 @@ export async function tableUpdate(req: Request<any, any>, res) {
   const sqlClient = await NcConnectionMgrv2.getSqlClient(base);
 
   let tableNameLengthLimit = 255;
-  const sqlClientType = sqlClient.clientType;
+  const sqlClientType = sqlClient.knex.clientType();
   if (sqlClientType === 'mysql2' || sqlClientType === 'mysql') {
     tableNameLengthLimit = 64;
   } else if (sqlClientType === 'pg') {

From ad0832ec15ae7f6c3c94baa06425b5b7302a99f3 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Fri, 17 Feb 2023 11:02:04 +0300
Subject: [PATCH 02/25] fix: ts types for sql clients

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 .../src/lib/db/sql-client/lib/oracle/OracleClient.ts   |  2 +-
 packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts           | 10 +++++++++-
 packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2.ts      |  2 +-
 packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2Trans.ts |  2 +-
 .../src/lib/db/sql-migrator/lib/KnexMigratorv2.ts      |  7 ++++---
 .../src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts  |  9 ++++++++-
 packages/nocodb/src/lib/meta/NcMetaMgr.ts              |  2 +-
 packages/nocodb/src/lib/utils/common/BaseApiBuilder.ts |  2 +-
 .../nocodb/src/lib/utils/common/NcConnectionMgr.ts     |  2 +-
 .../nocodb/src/lib/utils/common/NcConnectionMgrv2.ts   |  2 +-
 10 files changed, 28 insertions(+), 12 deletions(-)

diff --git a/packages/nocodb/src/lib/db/sql-client/lib/oracle/OracleClient.ts b/packages/nocodb/src/lib/db/sql-client/lib/oracle/OracleClient.ts
index 34779ebbe8..4f01353ba0 100644
--- a/packages/nocodb/src/lib/db/sql-client/lib/oracle/OracleClient.ts
+++ b/packages/nocodb/src/lib/db/sql-client/lib/oracle/OracleClient.ts
@@ -1913,7 +1913,7 @@ class OracleClient extends KnexClient {
    * @returns {Number} code
    * @returns {String} message
    */
-  async totalRecords(_args: any = {}) {
+  async totalRecords(_args: any = {}): Promise<Result> {
     // @ts-ignore
     const func = this.totalRecords.name;
     throw new Error('Function not supported for oracle yet');
diff --git a/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts b/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
index 9bbd2fa238..02b5c4e343 100644
--- a/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
+++ b/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
@@ -17,6 +17,14 @@ import NcConnectionMgr from '../../utils/common/NcConnectionMgr';
 import { customAlphabet } from 'nanoid';
 import Debug from '../util/Debug';
 import Result from '../util/Result';
+
+import type MssqlClient from '../sql-client/lib/mssql/MssqlClient';
+import type MysqlClient from '../sql-client/lib/mysql/MysqlClient';
+import type OracleClient from '../sql-client/lib/oracle/OracleClient';
+import type PGClient from '../sql-client/lib/pg/PgClient';
+import type SnowflakeClient from '../sql-client/lib/snowflake/SnowflakeClient';
+import type SqliteClient from '../sql-client/lib/sqlite/SqliteClient';
+
 const randomID = customAlphabet('1234567890abcdefghijklmnopqrstuvwxyz_', 20);
 const log = new Debug('SqlMgr');
 
@@ -337,7 +345,7 @@ export default class SqlMgr {
    * @returns
    * @memberof SqlMgr
    */
-  public async projectGetSqlClient(args) {
+  public async projectGetSqlClient(args): Promise<SnowflakeClient | MysqlClient | SqliteClient | MssqlClient | OracleClient | PGClient> {
     const func = this.projectGetSqlClient.name;
     log.api(`${func}:args:`, args);
 
diff --git a/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2.ts b/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2.ts
index 89ee530036..8c7d189461 100644
--- a/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2.ts
+++ b/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2.ts
@@ -118,7 +118,7 @@ export default class SqlMgrv2 {
     return sqlMigrationStatements;
   }
 
-  protected async getSqlClient(base: Base): Promise<any> {
+  protected async getSqlClient(base: Base) {
     return NcConnectionMgrv2.getSqlClient(base);
   }
 }
diff --git a/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2Trans.ts b/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2Trans.ts
index a5def5c184..82852fed52 100644
--- a/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2Trans.ts
+++ b/packages/nocodb/src/lib/db/sql-mgr/v2/SqlMgrv2Trans.ts
@@ -52,7 +52,7 @@ export default class SqlMgrv2Trans extends SqlMgrv2 {
     }
   }
 
-  protected async getSqlClient(base: Base): Promise<any> {
+  protected async getSqlClient(base: Base) {
     return NcConnectionMgrv2.getSqlClient(base, this.trx);
   }
 
diff --git a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
index cc1ad02a34..cdf4d8f1f1 100644
--- a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
+++ b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
@@ -1,6 +1,7 @@
 import fs from 'fs';
 import path from 'path';
 import { promisify } from 'util';
+import { Knex } from  'knex';
 
 import glob from 'glob';
 import SqlClientFactory from '../../sql-client/lib/SqlClientFactory';
@@ -428,7 +429,7 @@ export default class KnexMigratorv2 {
     // }
   }
 
-  protected async getSqlClient(base: Base): Promise<any> {
+  protected async getSqlClient(base: Base) {
     return NcConnectionMgrv2.getSqlClient(base);
   }
 
@@ -753,7 +754,7 @@ export default class KnexMigratorv2 {
             const vm = this;
 
             const trx = sqlClient.knex.isTransaction
-              ? sqlClient.knex
+              ? sqlClient.knex as Knex.Transaction
               : await sqlClient.knex.transaction();
             try {
               for (const query of upStatements) {
@@ -920,7 +921,7 @@ export default class KnexMigratorv2 {
           const vm = this;
 
           const trx = sqlClient.knex.isTransaction
-            ? sqlClient.knex
+            ? sqlClient.knex as Knex.Transaction
             : await sqlClient.knex.transaction();
           try {
             for (const query of downStatements) {
diff --git a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
index 7bfe6a2bdc..037cf6d559 100644
--- a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
+++ b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
@@ -11,6 +11,13 @@ import Noco from '../../../Noco';
 import { XKnex } from '../../sql-data-mapper';
 import NcMetaIO from '../../../meta/NcMetaIO';
 
+import type MssqlClient from '../../sql-client/lib/mssql/MssqlClient';
+import type MysqlClient from '../../sql-client/lib/mysql/MysqlClient';
+import type OracleClient from '../../sql-client/lib/oracle/OracleClient';
+import type PGClient from '../../sql-client/lib/pg/PgClient';
+import type SnowflakeClient from '../../sql-client/lib/snowflake/SnowflakeClient';
+import type SqliteClient from '../../sql-client/lib/sqlite/SqliteClient';
+
 export default class KnexMigratorv2Tans extends KnexMigratorv2 {
   protected sqlClient: any;
   protected ncMeta: NcMetaIO;
@@ -24,7 +31,7 @@ export default class KnexMigratorv2Tans extends KnexMigratorv2 {
   protected get metaDb(): XKnex {
     return this.ncMeta.knex || Noco.ncMeta.knex;
   }
-  protected async getSqlClient(base: Base): Promise<any> {
+  protected async getSqlClient(base: Base): Promise<SnowflakeClient | MysqlClient | SqliteClient | MssqlClient | OracleClient | PGClient> {
     return this.sqlClient || NcConnectionMgrv2.getSqlClient(base);
   }
 }
diff --git a/packages/nocodb/src/lib/meta/NcMetaMgr.ts b/packages/nocodb/src/lib/meta/NcMetaMgr.ts
index 02b2d7b607..aecaff7189 100644
--- a/packages/nocodb/src/lib/meta/NcMetaMgr.ts
+++ b/packages/nocodb/src/lib/meta/NcMetaMgr.ts
@@ -2909,7 +2909,7 @@ export default class NcMetaMgr {
     }
   }
 
-  protected async projectGetSqlClient(args): Promise<any> {
+  protected async projectGetSqlClient(args) {
     const builder = this.getBuilder(args);
     return builder?.getSqlClient();
   }
diff --git a/packages/nocodb/src/lib/utils/common/BaseApiBuilder.ts b/packages/nocodb/src/lib/utils/common/BaseApiBuilder.ts
index 210eb61a32..69294bdfc3 100644
--- a/packages/nocodb/src/lib/utils/common/BaseApiBuilder.ts
+++ b/packages/nocodb/src/lib/utils/common/BaseApiBuilder.ts
@@ -187,7 +187,7 @@ export default abstract class BaseApiBuilder<T extends Noco>
     return this.connectionConfig?.meta?.dbAlias;
   }
 
-  public async getSqlClient(): Promise<any> {
+  public async getSqlClient() {
     return NcConnectionMgr.getSqlClient({
       dbAlias: this.dbAlias,
       env: this.config.env,
diff --git a/packages/nocodb/src/lib/utils/common/NcConnectionMgr.ts b/packages/nocodb/src/lib/utils/common/NcConnectionMgr.ts
index 9c9c4c55a0..ca7f4fe53d 100644
--- a/packages/nocodb/src/lib/utils/common/NcConnectionMgr.ts
+++ b/packages/nocodb/src/lib/utils/common/NcConnectionMgr.ts
@@ -149,7 +149,7 @@ export default class NcConnectionMgr {
     env: string;
     config: NcConfig;
     projectId: string;
-  }): Promise<any> {
+  }) {
     const knex = this.get({
       dbAlias,
       env,
diff --git a/packages/nocodb/src/lib/utils/common/NcConnectionMgrv2.ts b/packages/nocodb/src/lib/utils/common/NcConnectionMgrv2.ts
index 06386e3391..bb55673cf6 100644
--- a/packages/nocodb/src/lib/utils/common/NcConnectionMgrv2.ts
+++ b/packages/nocodb/src/lib/utils/common/NcConnectionMgrv2.ts
@@ -149,7 +149,7 @@ export default class NcConnectionMgrv2 {
   //   return config?.envs?.[env]?.db?.find(db => db?.meta?.dbAlias === dbAlias);
   // }
 
-  public static async getSqlClient(base: Base, _knex = null): Promise<any> {
+  public static async getSqlClient(base: Base, _knex = null) {
     const knex = _knex || this.get(base);
     return SqlClientFactory.create({
       knex,

From cf974fd6c82e39df68e5b7b244d08b9fdd2a9ade Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Sat, 18 Feb 2023 15:22:54 +0530
Subject: [PATCH 03/25] feat(gui): Avoid closing the LTAR model if the element
 is added by holding ALT/OPT key

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../virtual-cell/components/ListItems.vue     | 31 +++++++++++++++++--
 1 file changed, 29 insertions(+), 2 deletions(-)

diff --git a/packages/nc-gui/components/virtual-cell/components/ListItems.vue b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
index 711030c443..b779171289 100644
--- a/packages/nc-gui/components/virtual-cell/components/ListItems.vue
+++ b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
@@ -48,14 +48,20 @@ const saveRow = inject(SaveRowInj, () => {})
 
 const selectedRowIndex = ref(0)
 
+const isAltKeyDown = ref(false)
+
 const linkRow = async (row: Record<string, any>) => {
   if (isNew.value) {
     addLTARRef(row, column?.value as ColumnType)
-    saveRow()
+    saveRow!()
   } else {
     await link(row)
   }
-  vModel.value = false
+  if (isAltKeyDown.value) {
+    loadChildrenExcludedList()
+  } else {
+    vModel.value = false
+  }
 }
 
 /** reload list on modal open */
@@ -159,6 +165,27 @@ useSelectedCellKeyupListener(vModel, (e: KeyboardEvent) => {
 const activeRow = (vNode?: InstanceType<typeof Card>) => {
   vNode?.$el?.scrollIntoView({ block: 'nearest', inline: 'nearest' })
 }
+
+// set variable to true when alt key is pressed
+const keyDownHandler = (e: KeyboardEvent) => {
+  isAltKeyDown.value = e.altKey
+}
+
+// set variable to false when key is released
+const keyUpHandler = (e: KeyboardEvent) => {
+  isAltKeyDown.value = e.altKey
+}
+
+// add event listeners when vModel is true and remove when false
+watch(vModel, (nextVal) => {
+  if (nextVal) {
+    document.addEventListener('keydown', keyDownHandler)
+    document.addEventListener('keyup', keyUpHandler)
+  } else {
+    document.removeEventListener('keydown', keyDownHandler)
+    document.removeEventListener('keyup', keyUpHandler)
+  }
+})
 </script>
 
 <template>

From bf8bb8659604fc65ffed0883b765538d6a802b03 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 16:16:16 +0800
Subject: [PATCH 04/25] fix(nc-gui): add missing filterChildrenRead permission

---
 packages/nc-gui/lib/constants.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/nc-gui/lib/constants.ts b/packages/nc-gui/lib/constants.ts
index bb98afbaeb..bfaa00720d 100644
--- a/packages/nc-gui/lib/constants.ts
+++ b/packages/nc-gui/lib/constants.ts
@@ -57,6 +57,7 @@ export const rolePermissions = {
       fieldsSync: true,
       gridColUpdate: true,
       filterSync: true,
+      filterChildrenRead: true,
       csvImport: true,
       apiDocs: true,
       projectSettings: true,

From fbbc69fd7209387ca505e3a652c1064c10f80e46 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 17:20:43 +0800
Subject: [PATCH 05/25] fix(nocodb-sdk): incorrect type

---
 packages/nocodb-sdk/src/lib/Api.ts | 2 +-
 scripts/sdk/swagger.json           | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/nocodb-sdk/src/lib/Api.ts b/packages/nocodb-sdk/src/lib/Api.ts
index 93ae72c9ec..a72029e453 100644
--- a/packages/nocodb-sdk/src/lib/Api.ts
+++ b/packages/nocodb-sdk/src/lib/Api.ts
@@ -502,7 +502,7 @@ export interface HookType {
   description?: string;
   env?: string;
   type?: string;
-  event?: 'After' | 'Before';
+  event?: 'after' | 'before';
   operation?: 'insert' | 'delete' | 'update';
   async?: boolean;
   payload?: string;
diff --git a/scripts/sdk/swagger.json b/scripts/sdk/swagger.json
index a2690a160c..796123f9ff 100644
--- a/scripts/sdk/swagger.json
+++ b/scripts/sdk/swagger.json
@@ -9175,8 +9175,8 @@
           "event": {
             "type": "string",
             "enum": [
-              "After",
-              "Before"
+              "after",
+              "before"
             ]
           },
           "operation": {

From 22e100a38905a76146e11f6a20fcfcc302c0c41f Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 17:23:14 +0800
Subject: [PATCH 06/25] refactor(nocodb): use extractProps to extract the props

---
 packages/nocodb/src/lib/models/Column.ts      | 62 ++++++++++---------
 packages/nocodb/src/lib/models/FormView.ts    | 57 +++++++----------
 .../nocodb/src/lib/models/FormViewColumn.ts   | 10 +--
 packages/nocodb/src/lib/models/GalleryView.ts | 33 +++++-----
 packages/nocodb/src/lib/models/GridView.ts    | 20 +++---
 packages/nocodb/src/lib/models/Hook.ts        | 60 +++++++++++-------
 packages/nocodb/src/lib/models/HookFilter.ts  | 18 +++---
 packages/nocodb/src/lib/models/KanbanView.ts  | 22 ++++---
 packages/nocodb/src/lib/models/Plugin.ts      | 17 ++---
 9 files changed, 155 insertions(+), 144 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Column.ts b/packages/nocodb/src/lib/models/Column.ts
index 6d7b2d5345..a66c976545 100644
--- a/packages/nocodb/src/lib/models/Column.ts
+++ b/packages/nocodb/src/lib/models/Column.ts
@@ -25,6 +25,7 @@ import addFormulaErrorIfMissingColumn from '../meta/helpers/addFormulaErrorIfMis
 import { NcError } from '../meta/helpers/catchError';
 import QrCodeColumn from './QrCodeColumn';
 import BarcodeColumn from './BarcodeColumn';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Column<T = any> implements ColumnType {
   public fk_model_id: string;
@@ -873,7 +874,11 @@ export default class Column<T = any> implements ColumnType {
     );
   }
 
-  static async update(colId: string, column: any, ncMeta = Noco.ncMeta) {
+  static async update(
+    colId: string,
+    column: Partial<Column>,
+    ncMeta = Noco.ncMeta
+  ) {
     const oldCol = await Column.get({ colId }, ncMeta);
 
     switch (oldCol.uidt) {
@@ -965,33 +970,34 @@ export default class Column<T = any> implements ColumnType {
         break;
       }
     }
-    const updateObj = {
-      column_name: column.column_name,
-      title: column.title,
-      uidt: column.uidt,
-      dt: column.dt,
-      np: column.np,
-      ns: column.ns,
-      clen: column.clen,
-      cop: column.cop,
-      pk: column.pk,
-      rqd: column.rqd,
-      un: column.un,
-      ct: column.ct,
-      ai: column.ai,
-      unique: column.unique,
-      cdf: column.cdf,
-      cc: column.cc,
-      csn: column.csn,
-      dtx: column.dtx,
-      dtxp: column.dtxp,
-      dtxs: column.dtxs,
-      au: column.au,
-      pv: column.pv,
-      system: column.system,
-      validate: null,
-      meta: column.meta,
-    };
+
+    const updateObj = extractProps(column, [
+      'column_name',
+      'title',
+      'uidt',
+      'dt',
+      'np',
+      'ns',
+      'clen',
+      'cop',
+      'pk',
+      'rqd',
+      'un',
+      'ct',
+      'ai',
+      'unique',
+      'cdf',
+      'cc',
+      'csn',
+      'dtx',
+      'dtxp',
+      'dtxs',
+      'au',
+      'pv',
+      'system',
+      'validate',
+      'meta',
+    ]);
 
     if (column.validate) {
       if (typeof column.validate === 'string')
diff --git a/packages/nocodb/src/lib/models/FormView.ts b/packages/nocodb/src/lib/models/FormView.ts
index 49ee5c4da2..463f761cd1 100644
--- a/packages/nocodb/src/lib/models/FormView.ts
+++ b/packages/nocodb/src/lib/models/FormView.ts
@@ -5,6 +5,7 @@ import { deserializeJSON, serializeJSON } from '../utils/serialize';
 import FormViewColumn from './FormViewColumn';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class FormView implements FormType {
   show: boolean;
@@ -86,44 +87,34 @@ export default class FormView implements FormType {
   ) {
     // get existing cache
     const key = `${CacheScope.FORM_VIEW}:${formId}`;
-    const o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    const updateObj = extractProps(body, [
+      'heading',
+      'subheading',
+      'success_msg',
+      'redirect_url',
+      'redirect_after_secs',
+      'email',
+      'banner_image_url',
+      'logo_url',
+      'submit_another_form',
+      'show_blank_form',
+      'meta',
+    ]);
     if (o) {
-      o.heading = body.heading;
-      o.subheading = body.subheading;
-      o.success_msg = body.success_msg;
-      o.redirect_url = body.redirect_url;
-      o.redirect_after_secs = body.redirect_after_secs;
-      o.email = body.email;
-      o.banner_image_url = body.banner_image_url;
-      o.logo_url = body.logo_url;
-      o.submit_another_form = body.submit_another_form;
-      o.show_blank_form = body.show_blank_form;
-      o.meta = body.meta;
+      o = { ...o, ...updateObj };
       // set cache
       await NocoCache.set(key, o);
-      o.meta = serializeJSON(body.meta);
     }
+
+    if (updateObj.meta) {
+      updateObj.meta = serializeJSON(updateObj.meta);
+    }
+
     // update meta
-    return await ncMeta.metaUpdate(
-      null,
-      null,
-      MetaTable.FORM_VIEW,
-      {
-        heading: body.heading,
-        subheading: body.subheading,
-        success_msg: body.success_msg,
-        redirect_url: body.redirect_url,
-        redirect_after_secs: body.redirect_after_secs,
-        email: body.email,
-        banner_image_url: body.banner_image_url,
-        logo_url: body.logo_url,
-        submit_another_form: body.submit_another_form,
-        show_blank_form: body.show_blank_form,
-      },
-      {
-        fk_view_id: formId,
-      }
-    );
+    return await ncMeta.metaUpdate(null, null, MetaTable.FORM_VIEW, updateObj, {
+      fk_view_id: formId,
+    });
   }
 
   async getColumns(ncMeta = Noco.ncMeta) {
diff --git a/packages/nocodb/src/lib/models/FormViewColumn.ts b/packages/nocodb/src/lib/models/FormViewColumn.ts
index 8cd3fb3efe..e1b648db59 100644
--- a/packages/nocodb/src/lib/models/FormViewColumn.ts
+++ b/packages/nocodb/src/lib/models/FormViewColumn.ts
@@ -150,7 +150,7 @@ export default class FormViewColumn implements FormColumnType {
     body: Partial<FormViewColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = extractProps(body, [
+    const updateObj = extractProps(body, [
       'label',
       'help',
       'description',
@@ -164,13 +164,13 @@ export default class FormViewColumn implements FormColumnType {
     const key = `${CacheScope.FORM_VIEW_COLUMN}:${columnId}`;
     const o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
     if (o) {
-      Object.assign(o, insertObj);
+      Object.assign(o, updateObj);
       // set cache
       await NocoCache.set(key, o);
     }
 
-    if (insertObj.meta) {
-      insertObj.meta = serializeJSON(insertObj.meta);
+    if (updateObj.meta) {
+      updateObj.meta = serializeJSON(updateObj.meta);
     }
 
     // update meta
@@ -178,7 +178,7 @@ export default class FormViewColumn implements FormColumnType {
       null,
       null,
       MetaTable.FORM_VIEW_COLUMNS,
-      insertObj,
+      updateObj,
       columnId
     );
   }
diff --git a/packages/nocodb/src/lib/models/GalleryView.ts b/packages/nocodb/src/lib/models/GalleryView.ts
index 9c76f1de8e..85cf4779ef 100644
--- a/packages/nocodb/src/lib/models/GalleryView.ts
+++ b/packages/nocodb/src/lib/models/GalleryView.ts
@@ -3,6 +3,7 @@ import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import { GalleryColumnType, GalleryType, UITypes } from 'nocodb-sdk';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class GalleryView implements GalleryType {
   fk_view_id?: string;
@@ -90,16 +91,19 @@ export default class GalleryView implements GalleryType {
   ) {
     // get existing cache
     const key = `${CacheScope.GALLERY_VIEW}:${galleryId}`;
-    const o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    const updateObj = extractProps(body, [
+      'next_enabled',
+      'prev_enabled',
+      'cover_image_idx',
+      'cover_image',
+      'restrict_types',
+      'restrict_size',
+      'restrict_number',
+      'fk_cover_image_col_id',
+    ]);
     if (o) {
-      o.next_enabled = body.next_enabled;
-      o.prev_enabled = body.prev_enabled;
-      o.cover_image_idx = body.cover_image_idx;
-      o.cover_image = body.cover_image;
-      o.restrict_types = body.restrict_types;
-      o.restrict_size = body.restrict_size;
-      o.restrict_number = body.restrict_number;
-      o.fk_cover_image_col_id = body.fk_cover_image_col_id;
+      o = { ...o, ...updateObj };
       // set cache
       await NocoCache.set(key, o);
     }
@@ -108,16 +112,7 @@ export default class GalleryView implements GalleryType {
       null,
       null,
       MetaTable.GALLERY_VIEW,
-      {
-        next_enabled: body.next_enabled,
-        prev_enabled: body.prev_enabled,
-        cover_image_idx: body.cover_image_idx,
-        cover_image: body.cover_image,
-        restrict_types: body.restrict_types,
-        restrict_size: body.restrict_size,
-        restrict_number: body.restrict_number,
-        fk_cover_image_col_id: body.fk_cover_image_col_id,
-      },
+      updateObj,
       {
         fk_view_id: galleryId,
       }
diff --git a/packages/nocodb/src/lib/models/GridView.ts b/packages/nocodb/src/lib/models/GridView.ts
index 40a98b0c9e..77310c64cb 100644
--- a/packages/nocodb/src/lib/models/GridView.ts
+++ b/packages/nocodb/src/lib/models/GridView.ts
@@ -3,6 +3,7 @@ import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import GridViewColumn from './GridViewColumn';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class GridView {
   fk_view_id: string;
@@ -69,23 +70,16 @@ export default class GridView {
   ) {
     // get existing cache
     const key = `${CacheScope.GRID_VIEW}:${viewId}`;
-    const o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
+    const updateObj = extractProps(body, ['row_height']);
     if (o) {
-      o.row_height = body.row_height;
+      o = { ...o, ...updateObj };
       // set cache
       await NocoCache.set(key, o);
     }
     // update meta
-    return await ncMeta.metaUpdate(
-      null,
-      null,
-      MetaTable.GRID_VIEW,
-      {
-        row_height: body.row_height,
-      },
-      {
-        fk_view_id: viewId,
-      }
-    );
+    return await ncMeta.metaUpdate(null, null, MetaTable.GRID_VIEW, updateObj, {
+      fk_view_id: viewId,
+    });
   }
 }
diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index d87aa994bf..e90e1e01c2 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -10,6 +10,7 @@ import Model from './Model';
 import NocoCache from '../cache/NocoCache';
 import Filter from './Filter';
 import HookFilter from './HookFilter';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Hook implements HookType {
   id?: string;
@@ -18,7 +19,7 @@ export default class Hook implements HookType {
   description?: string;
   env?: string;
   type?: string;
-  event?: 'After' | 'Before';
+  event?: 'after' | 'before';
   operation?: 'insert' | 'delete' | 'update';
   async?: boolean;
   payload?: string;
@@ -175,27 +176,41 @@ export default class Hook implements HookType {
     hook: Partial<Hook>,
     ncMeta = Noco.ncMeta
   ) {
-    const updateObj = {
-      title: hook.title,
-      description: hook.description,
-      env: hook.env,
-      type: hook.type,
-      event: hook.event?.toLowerCase?.(),
-      operation: hook.operation?.toLowerCase?.(),
-      async: hook.async,
-      payload: !!hook.payload,
-      url: hook.url,
-      headers: hook.headers,
-      condition: !!hook.condition,
-      notification:
-        hook.notification && typeof hook.notification === 'object'
-          ? JSON.stringify(hook.notification)
-          : hook.notification,
-      retries: hook.retries,
-      retry_interval: hook.retry_interval,
-      timeout: hook.timeout,
-      active: hook.active,
-    };
+    const updateObj = extractProps(hook, [
+      'title',
+      'description',
+      'env',
+      'type',
+      'event',
+      'operation',
+      'async',
+      'payload',
+      'url',
+      'headers',
+      'condition',
+      'notification',
+      'retries',
+      'retry_interval',
+      'timeout',
+      'active',
+    ]);
+
+    if (updateObj.event) {
+      updateObj.event = updateObj.event.toLowerCase() as 'after' | 'before';
+    }
+
+    if (updateObj.operation) {
+      updateObj.operation = updateObj.operation.toLowerCase() as
+        | 'insert'
+        | 'delete'
+        | 'update';
+    }
+
+    if (updateObj.notification) {
+      updateObj.notification && typeof updateObj.notification === 'object'
+        ? JSON.stringify(updateObj.notification)
+        : updateObj.notification;
+    }
 
     // get existing cache
     const key = `${CacheScope.HOOK}:${hookId}`;
@@ -203,7 +218,6 @@ export default class Hook implements HookType {
     if (o) {
       // update data
       o = { ...o, ...updateObj };
-      o.notification = updateObj.notification;
       // set cache
       await NocoCache.set(key, o);
     }
diff --git a/packages/nocodb/src/lib/models/HookFilter.ts b/packages/nocodb/src/lib/models/HookFilter.ts
index 956df57f8c..b9a1d36e18 100644
--- a/packages/nocodb/src/lib/models/HookFilter.ts
+++ b/packages/nocodb/src/lib/models/HookFilter.ts
@@ -10,6 +10,7 @@ import {
 import View from './View';
 import { FilterType, UITypes } from 'nocodb-sdk';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Filter {
   id: string;
@@ -138,15 +139,14 @@ export default class Filter {
   }
 
   static async update(id, filter: Partial<Filter>, ncMeta = Noco.ncMeta) {
-    const updateObj = {
-      fk_column_id: filter.fk_column_id,
-      comparison_op: filter.comparison_op,
-      value: filter.value,
-      fk_parent_id: filter.fk_parent_id,
-
-      is_group: filter.is_group,
-      logical_op: filter.logical_op,
-    };
+    const updateObj = extractProps(filter, [
+      'fk_column_id',
+      'comparison_op',
+      'value',
+      'fk_parent_id',
+      'is_group',
+      'logical_op',
+    ]);
     // get existing cache
     const key = `${CacheScope.FILTER_EXP}:${id}`;
     let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
diff --git a/packages/nocodb/src/lib/models/KanbanView.ts b/packages/nocodb/src/lib/models/KanbanView.ts
index 52bd2bf088..9097478733 100644
--- a/packages/nocodb/src/lib/models/KanbanView.ts
+++ b/packages/nocodb/src/lib/models/KanbanView.ts
@@ -3,6 +3,7 @@ import { KanbanType, UITypes } from 'nocodb-sdk';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class KanbanView implements KanbanType {
   fk_view_id: string;
@@ -99,13 +100,20 @@ export default class KanbanView implements KanbanType {
     // get existing cache
     const key = `${CacheScope.KANBAN_VIEW}:${kanbanId}`;
     let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
-    const updateObj = {
-      ...body,
-      meta:
-        typeof body.meta === 'string'
-          ? body.meta
-          : JSON.stringify(body.meta ?? {}),
-    };
+
+    const updateObj = extractProps(body, [
+      'title',
+      'fk_cover_image_col_id',
+      'meta',
+    ]);
+
+    if (updateObj.meta) {
+      updateObj.meta =
+        typeof updateObj.meta === 'string'
+          ? updateObj.meta
+          : JSON.stringify(updateObj.meta ?? {});
+    }
+
     if (o) {
       o = { ...o, ...updateObj };
       // set cache
diff --git a/packages/nocodb/src/lib/models/Plugin.ts b/packages/nocodb/src/lib/models/Plugin.ts
index b5f2097568..d53757af51 100644
--- a/packages/nocodb/src/lib/models/Plugin.ts
+++ b/packages/nocodb/src/lib/models/Plugin.ts
@@ -2,6 +2,7 @@ import { PluginType } from 'nocodb-sdk';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import Noco from '../Noco';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Plugin implements PluginType {
   id?: string;
@@ -56,13 +57,15 @@ export default class Plugin implements PluginType {
   }
 
   public static async update(pluginId: string, plugin: Partial<PluginType>) {
-    const updateObj = {
-      input:
-        plugin.input && typeof plugin.input === 'object'
-          ? JSON.stringify(plugin.input)
-          : plugin.input,
-      active: plugin.active,
-    };
+    const updateObj = extractProps(plugin, ['input', 'active']);
+
+    if (updateObj.input) {
+      updateObj.input =
+        updateObj.input && typeof updateObj.input === 'object'
+          ? JSON.stringify(updateObj.input)
+          : updateObj.input;
+    }
+
     // get existing cache
     const key = `${CacheScope.PLUGIN}:${pluginId}`;
     let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);

From 05d11202b831bc511755a81cac177eb09011de93 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 18:11:23 +0800
Subject: [PATCH 07/25] fix(nocodb): fix type

---
 .../src/lib/version-upgrader/ncProjectUpgraderV2_0090000.ts     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/nocodb/src/lib/version-upgrader/ncProjectUpgraderV2_0090000.ts b/packages/nocodb/src/lib/version-upgrader/ncProjectUpgraderV2_0090000.ts
index 2f604b18df..4bd94c8574 100644
--- a/packages/nocodb/src/lib/version-upgrader/ncProjectUpgraderV2_0090000.ts
+++ b/packages/nocodb/src/lib/version-upgrader/ncProjectUpgraderV2_0090000.ts
@@ -1231,7 +1231,7 @@ async function migrateWebhooks(ctx: MigrateCtxV1, ncMeta: any) {
     env: string;
     tn: string;
     type: string;
-    event: 'After' | 'Before';
+    event: 'after' | 'before';
     operation: 'delete' | 'update' | 'insert';
     async: boolean;
     payload: string;

From 353cc8ea60299baa7a11944660d6a7152eb62b9a Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 19:04:10 +0800
Subject: [PATCH 08/25] refactor(nocodb): use extractProps for insertObj

---
 packages/nocodb/src/lib/models/Audit.ts       | 38 ++++-----
 .../nocodb/src/lib/models/BarcodeColumn.ts    | 15 ++--
 packages/nocodb/src/lib/models/Column.ts      | 77 +++++++++++--------
 packages/nocodb/src/lib/models/Filter.ts      | 43 ++++++-----
 packages/nocodb/src/lib/models/FormView.ts    | 34 ++++----
 .../nocodb/src/lib/models/FormViewColumn.ts   | 43 ++++++-----
 .../nocodb/src/lib/models/FormulaColumn.ts    | 17 ++--
 packages/nocodb/src/lib/models/GalleryView.ts | 32 ++++----
 .../src/lib/models/GalleryViewColumn.ts       | 23 +++---
 packages/nocodb/src/lib/models/GridView.ts    | 17 ++--
 .../nocodb/src/lib/models/GridViewColumn.ts   | 25 +++---
 packages/nocodb/src/lib/models/Hook.ts        | 64 ++++++++-------
 packages/nocodb/src/lib/models/HookFilter.ts  | 25 +++---
 packages/nocodb/src/lib/models/KanbanView.ts  | 21 ++---
 .../nocodb/src/lib/models/KanbanViewColumn.ts | 23 +++---
 .../lib/models/LinkToAnotherRecordColumn.ts   | 38 ++++-----
 .../nocodb/src/lib/models/LookupColumn.ts     | 14 ++--
 packages/nocodb/src/lib/models/Model.ts       | 45 +++++++----
 .../src/lib/models/ModelRoleVisibility.ts     | 21 ++---
 packages/nocodb/src/lib/models/Project.ts     | 24 +++---
 packages/nocodb/src/lib/models/ProjectUser.ts | 17 ++--
 .../nocodb/src/lib/models/QrCodeColumn.ts     | 14 ++--
 .../nocodb/src/lib/models/RollupColumn.ts     | 14 ++--
 .../nocodb/src/lib/models/SelectOption.ts     | 15 +++-
 packages/nocodb/src/lib/models/Sort.ts        | 24 +++---
 packages/nocodb/src/lib/models/SyncLogs.ts    | 15 ++--
 packages/nocodb/src/lib/models/SyncSource.ts  | 18 ++---
 packages/nocodb/src/lib/models/View.ts        | 34 ++++----
 28 files changed, 438 insertions(+), 352 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Audit.ts b/packages/nocodb/src/lib/models/Audit.ts
index a512f96b48..9ea8e589d2 100644
--- a/packages/nocodb/src/lib/models/Audit.ts
+++ b/packages/nocodb/src/lib/models/Audit.ts
@@ -2,6 +2,7 @@ import { AuditOperationTypes, AuditType } from 'nocodb-sdk';
 import { MetaTable } from '../utils/globals';
 import Noco from '../Noco';
 import Model from './Model';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Audit implements AuditType {
   id?: string;
@@ -48,27 +49,28 @@ export default class Audit implements AuditType {
       return;
     }
     const insertAudit = async () => {
-      if (!audit.project_id && audit.fk_model_id) {
-        audit.project_id = (
-          await Model.getByIdOrName({ id: audit.fk_model_id }, ncMeta)
+      const insertObj = extractProps(audit, [
+        'user',
+        'ip',
+        'base_id',
+        'project_id',
+        'row_id',
+        'fk_model_id',
+        'op_type',
+        'op_sub_type',
+        'status',
+        'description',
+        'details',
+        'created_at',
+        'updated_at',
+      ]);
+      if (!insertObj.project_id && insertObj.fk_model_id) {
+        insertObj.project_id = (
+          await Model.getByIdOrName({ id: insertObj.fk_model_id }, ncMeta)
         ).project_id;
       }
 
-      return await ncMeta.metaInsert2(null, null, MetaTable.AUDIT, {
-        user: audit.user,
-        ip: audit.ip,
-        base_id: audit.base_id,
-        project_id: audit.project_id,
-        row_id: audit.row_id,
-        fk_model_id: audit.fk_model_id,
-        op_type: audit.op_type,
-        op_sub_type: audit.op_sub_type,
-        status: audit.status,
-        description: audit.description,
-        details: audit.details,
-        created_at: audit.created_at,
-        updated_at: audit.updated_at,
-      });
+      return await ncMeta.metaInsert2(null, null, MetaTable.AUDIT, insertObj);
     };
 
     if (forceAwait) {
diff --git a/packages/nocodb/src/lib/models/BarcodeColumn.ts b/packages/nocodb/src/lib/models/BarcodeColumn.ts
index 6f274f3f86..745de9200c 100644
--- a/packages/nocodb/src/lib/models/BarcodeColumn.ts
+++ b/packages/nocodb/src/lib/models/BarcodeColumn.ts
@@ -14,16 +14,17 @@ export default class BarcodeColumn {
   }
 
   public static async insert(
-    data: Partial<BarcodeColumn>,
+    barcodeColumn: Partial<BarcodeColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_BARCODE, {
-      fk_column_id: data.fk_column_id,
-      fk_barcode_value_column_id: data.fk_barcode_value_column_id,
-      barcode_format: data.barcode_format,
-    });
+    const insertObj = extractProps(barcodeColumn, [
+      'fk_column_id',
+      'fk_barcode_value_column_id',
+      'barcode_format',
+    ]);
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_BARCODE, insertObj);
 
-    return this.read(data.fk_column_id, ncMeta);
+    return this.read(barcodeColumn.fk_column_id, ncMeta);
   }
   public static async read(columnId: string, ncMeta = Noco.ncMeta) {
     let column =
diff --git a/packages/nocodb/src/lib/models/Column.ts b/packages/nocodb/src/lib/models/Column.ts
index a66c976545..154e0a4098 100644
--- a/packages/nocodb/src/lib/models/Column.ts
+++ b/packages/nocodb/src/lib/models/Column.ts
@@ -86,41 +86,50 @@ export default class Column<T = any> implements ColumnType {
   ) {
     if (!column.fk_model_id) NcError.badRequest('Missing model id');
 
-    const insertObj: any = {
-      id: column?.id,
-      fk_model_id: column.fk_model_id,
-      column_name: column.column_name || column.cn,
-      title: column.title || column._cn,
-      uidt: column.uidt,
-      dt: column.dt,
-      np: column.np,
-      ns: column.ns,
-      clen: column.clen,
-      cop: column.cop,
-      pk: column.pk,
-      rqd: column.rqd,
-      un: column.un,
-      ct: column.ct,
-      ai: column.ai,
-      unique: column.unique,
-      cdf: column.cdf,
-      cc: column.cc,
-      csn: column.csn,
-      dtx: column.dtx,
-      dtxp: column.dtxp,
-      dtxs: column.dtxs,
-      au: column.au,
-      pv: column.pv,
-      order: column.order,
-      project_id: column.project_id,
-      base_id: column.base_id,
-      system: column.system,
-      meta:
-        column.meta && typeof column.meta === 'object'
-          ? JSON.stringify(column.meta)
-          : column.meta,
-    };
+    // TODO: fix type
+    const insertObj = extractProps(column as any, [
+      'id',
+      'fk_model_id',
+      'column_name',
+      'title',
+      'uidt',
+      'dt',
+      'np',
+      'ns',
+      'clen',
+      'cop',
+      'pk',
+      'rqd',
+      'un',
+      'ct',
+      'ai',
+      'unique',
+      'cdf',
+      'cc',
+      'csn',
+      'dtx',
+      'dtxp',
+      'dtxs',
+      'au',
+      'pv',
+      'order',
+      'project_id',
+      'base_id',
+      'system',
+      'meta',
+    ]);
+
+    if (!insertObj.column_name) {
+      insertObj.column_name = column.cn;
+    }
 
+    if (!insertObj.title) {
+      insertObj.title = column._cn;
+    }
+
+    if (insertObj.meta && typeof insertObj.meta === 'object') {
+      insertObj.meta = JSON.stringify(insertObj.meta);
+    }
     if (column.validate) {
       if (typeof column.validate === 'string')
         insertObj.validate = column.validate;
diff --git a/packages/nocodb/src/lib/models/Filter.ts b/packages/nocodb/src/lib/models/Filter.ts
index 619b6a741d..1e6a0dfaee 100644
--- a/packages/nocodb/src/lib/models/Filter.ts
+++ b/packages/nocodb/src/lib/models/Filter.ts
@@ -77,29 +77,30 @@ export default class Filter {
   }
 
   public static async insert(
-    filter: Partial<FilterType>,
+    filter: Partial<FilterType> & { order?: number },
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      id: filter.id,
-      fk_view_id: filter.fk_view_id,
-      fk_hook_id: filter.fk_hook_id,
-      fk_column_id: filter.fk_column_id,
-      comparison_op: filter.comparison_op,
-      value: filter.value,
-      fk_parent_id: filter.fk_parent_id,
-
-      is_group: filter.is_group,
-      logical_op: filter.logical_op,
-
-      project_id: filter.project_id,
-      base_id: filter.base_id,
-      order: await ncMeta.metaGetNextOrder(MetaTable.FILTER_EXP, {
-        [filter.fk_hook_id ? 'fk_hook_id' : 'fk_view_id']: filter.fk_hook_id
-          ? filter.fk_hook_id
-          : filter.fk_view_id,
-      }),
-    };
+    const insertObj = extractProps(filter, [
+      'id',
+      'fk_view_id',
+      'fk_hook_id',
+      'fk_column_id',
+      'comparison_op',
+      'value',
+      'fk_parent_id',
+      'is_group',
+      'logical_op',
+      'project_id',
+      'base_id',
+      'order',
+    ]);
+
+    insertObj.order = await ncMeta.metaGetNextOrder(MetaTable.FILTER_EXP, {
+      [filter.fk_hook_id ? 'fk_hook_id' : 'fk_view_id']: filter.fk_hook_id
+        ? filter.fk_hook_id
+        : filter.fk_view_id,
+    });
+
     if (!(filter.project_id && filter.base_id)) {
       let model: { project_id?: string; base_id?: string };
       if (filter.fk_view_id) {
diff --git a/packages/nocodb/src/lib/models/FormView.ts b/packages/nocodb/src/lib/models/FormView.ts
index 463f761cd1..5f10ce1803 100644
--- a/packages/nocodb/src/lib/models/FormView.ts
+++ b/packages/nocodb/src/lib/models/FormView.ts
@@ -54,22 +54,24 @@ export default class FormView implements FormType {
   }
 
   static async insert(view: Partial<FormView>, ncMeta = Noco.ncMeta) {
-    const insertObj = {
-      fk_view_id: view.fk_view_id,
-      project_id: view.project_id,
-      base_id: view.base_id,
-      heading: view.heading,
-      subheading: view.subheading,
-      success_msg: view.success_msg,
-      redirect_url: view.redirect_url,
-      redirect_after_secs: view.redirect_after_secs,
-      email: view.email,
-      banner_image_url: view.banner_image_url,
-      logo_url: view.logo_url,
-      submit_another_form: view.submit_another_form,
-      show_blank_form: view.show_blank_form,
-      meta: serializeJSON(view.meta),
-    };
+    const insertObj = extractProps(view, [
+      'fk_view_id',
+      'project_id',
+      'base_id',
+      'heading',
+      'subheading',
+      'success_msg',
+      'redirect_url',
+      'redirect_after_secs',
+      'email',
+      'banner_image_url',
+      'logo_url',
+      'submit_another_form',
+      'show_blank_form',
+    ]);
+    if (insertObj.meta) {
+      insertObj.meta = serializeJSON(insertObj.meta);
+    }
     if (!(view.project_id && view.base_id)) {
       const viewRef = await View.get(view.fk_view_id);
       insertObj.project_id = viewRef.project_id;
diff --git a/packages/nocodb/src/lib/models/FormViewColumn.ts b/packages/nocodb/src/lib/models/FormViewColumn.ts
index e1b648db59..577d5ad5ca 100644
--- a/packages/nocodb/src/lib/models/FormViewColumn.ts
+++ b/packages/nocodb/src/lib/models/FormViewColumn.ts
@@ -55,27 +55,32 @@ export default class FormViewColumn implements FormColumnType {
   }
 
   static async insert(column: Partial<FormViewColumn>, ncMeta = Noco.ncMeta) {
-    const insertObj: Partial<FormViewColumn> = {
-      fk_view_id: column.fk_view_id,
-      fk_column_id: column.fk_column_id,
-      order: await ncMeta.metaGetNextOrder(MetaTable.FORM_VIEW_COLUMNS, {
-        fk_view_id: column.fk_view_id,
-      }),
-      show: column.show,
-      project_id: column.project_id,
-      base_id: column.base_id,
-      label: column.label,
-      help: column.help,
-      description: column.description,
-      required: column.required,
-    };
-
-    if (column.meta) {
-      insertObj.meta = serializeJSON(column.meta);
+    const insertObj = extractProps(column, [
+      'fk_view_id',
+      'fk_column_id',
+      'show',
+      'project_id',
+      'base_id',
+      'label',
+      'help',
+      'description',
+      'required',
+      'meta',
+    ]);
+
+    insertObj.order = await ncMeta.metaGetNextOrder(
+      MetaTable.FORM_VIEW_COLUMNS,
+      {
+        fk_view_id: insertObj.fk_view_id,
+      }
+    );
+
+    if (insertObj.meta) {
+      insertObj.meta = serializeJSON(insertObj.meta);
     }
 
-    if (!(column.project_id && column.base_id)) {
-      const viewRef = await View.get(column.fk_view_id, ncMeta);
+    if (!(insertObj.project_id && insertObj.base_id)) {
+      const viewRef = await View.get(insertObj.fk_view_id, ncMeta);
       insertObj.project_id = viewRef.project_id;
       insertObj.base_id = viewRef.base_id;
     }
diff --git a/packages/nocodb/src/lib/models/FormulaColumn.ts b/packages/nocodb/src/lib/models/FormulaColumn.ts
index 106b78934c..90e4f22c61 100644
--- a/packages/nocodb/src/lib/models/FormulaColumn.ts
+++ b/packages/nocodb/src/lib/models/FormulaColumn.ts
@@ -14,17 +14,18 @@ export default class FormulaColumn {
   }
 
   public static async insert(
-    data: Partial<FormulaColumn>,
+    formulaColumn: Partial<FormulaColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_FORMULA, {
-      fk_column_id: data.fk_column_id,
-      formula_raw: data.formula_raw,
-      formula: data.formula,
-      error: data.error,
-    });
+    const insertObj = extractProps(formulaColumn, [
+      'fk_column_id',
+      'formula_raw',
+      'formula',
+      'error',
+    ]);
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_FORMULA, insertObj);
 
-    return this.read(data.fk_column_id, ncMeta);
+    return this.read(formulaColumn.fk_column_id, ncMeta);
   }
   public static async read(columnId: string, ncMeta = Noco.ncMeta) {
     let column =
diff --git a/packages/nocodb/src/lib/models/GalleryView.ts b/packages/nocodb/src/lib/models/GalleryView.ts
index 85cf4779ef..2cf2f6ce8f 100644
--- a/packages/nocodb/src/lib/models/GalleryView.ts
+++ b/packages/nocodb/src/lib/models/GalleryView.ts
@@ -52,21 +52,23 @@ export default class GalleryView implements GalleryType {
       .then((v) => v?.getModel(ncMeta))
       .then((m) => m.getColumns(ncMeta));
 
-    const insertObj = {
-      project_id: view.project_id,
-      base_id: view.base_id,
-      fk_view_id: view.fk_view_id,
-      fk_cover_image_col_id:
-        view?.fk_cover_image_col_id ||
-        columns?.find((c) => c.uidt === UITypes.Attachment)?.id,
-      next_enabled: view.next_enabled,
-      prev_enabled: view.prev_enabled,
-      cover_image_idx: view.cover_image_idx,
-      cover_image: view.cover_image,
-      restrict_types: view.restrict_types,
-      restrict_size: view.restrict_size,
-      restrict_number: view.restrict_number,
-    };
+    const insertObj = extractProps(view, [
+      'project_id',
+      'base_id',
+      'fk_view_id',
+      'next_enabled',
+      'prev_enabled',
+      'cover_image_idx',
+      'cover_image',
+      'restrict_types',
+      'restrict_size',
+      'restrict_number',
+    ]);
+
+    insertObj.fk_cover_image_col_id =
+      view?.fk_cover_image_col_id ||
+      columns?.find((c) => c.uidt === UITypes.Attachment)?.id;
+
     if (!(view.project_id && view.base_id)) {
       const viewRef = await View.get(view.fk_view_id);
       insertObj.project_id = viewRef.project_id;
diff --git a/packages/nocodb/src/lib/models/GalleryViewColumn.ts b/packages/nocodb/src/lib/models/GalleryViewColumn.ts
index 75ca96f4f1..3ccc1e73c3 100644
--- a/packages/nocodb/src/lib/models/GalleryViewColumn.ts
+++ b/packages/nocodb/src/lib/models/GalleryViewColumn.ts
@@ -2,6 +2,7 @@ import Noco from '../Noco';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class GalleryViewColumn {
   id: string;
@@ -43,16 +44,20 @@ export default class GalleryViewColumn {
     column: Partial<GalleryViewColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      fk_view_id: column.fk_view_id,
-      fk_column_id: column.fk_column_id,
-      order: await ncMeta.metaGetNextOrder(MetaTable.GALLERY_VIEW_COLUMNS, {
+    const insertObj = extractProps(column, [
+      'fk_view_id',
+      'fk_column_id',
+      'show',
+      'project_id',
+      'base_id',
+    ]);
+
+    insertObj.order = await ncMeta.metaGetNextOrder(
+      MetaTable.GALLERY_VIEW_COLUMNS,
+      {
         fk_view_id: column.fk_view_id,
-      }),
-      show: column.show,
-      project_id: column.project_id,
-      base_id: column.base_id,
-    };
+      }
+    );
 
     if (!(column.project_id && column.base_id)) {
       const viewRef = await View.get(column.fk_view_id, ncMeta);
diff --git a/packages/nocodb/src/lib/models/GridView.ts b/packages/nocodb/src/lib/models/GridView.ts
index 77310c64cb..880254261a 100644
--- a/packages/nocodb/src/lib/models/GridView.ts
+++ b/packages/nocodb/src/lib/models/GridView.ts
@@ -41,14 +41,15 @@ export default class GridView {
   }
 
   static async insert(view: Partial<GridView>, ncMeta = Noco.ncMeta) {
-    const insertObj = {
-      fk_view_id: view.fk_view_id,
-      project_id: view.project_id,
-      base_id: view.base_id,
-      row_height: view.row_height,
-    };
-    if (!(view.project_id && view.base_id)) {
-      const viewRef = await View.get(view.fk_view_id, ncMeta);
+    const insertObj = extractProps(view, [
+      'fk_view_id',
+      'project_id',
+      'base_id',
+      'row_height',
+    ]);
+
+    if (!(insertObj.project_id && insertObj.base_id)) {
+      const viewRef = await View.get(insertObj.fk_view_id, ncMeta);
       insertObj.project_id = viewRef.project_id;
       insertObj.base_id = viewRef.base_id;
     }
diff --git a/packages/nocodb/src/lib/models/GridViewColumn.ts b/packages/nocodb/src/lib/models/GridViewColumn.ts
index e1d0398276..f3a3385a70 100644
--- a/packages/nocodb/src/lib/models/GridViewColumn.ts
+++ b/packages/nocodb/src/lib/models/GridViewColumn.ts
@@ -67,18 +67,19 @@ export default class GridViewColumn implements GridColumnType {
   }
 
   static async insert(column: Partial<GridViewColumn>, ncMeta = Noco.ncMeta) {
-    const insertObj = {
-      fk_view_id: column.fk_view_id,
-      fk_column_id: column.fk_column_id,
-      order:
-        column?.order ??
-        (await ncMeta.metaGetNextOrder(MetaTable.GRID_VIEW_COLUMNS, {
-          fk_view_id: column.fk_view_id,
-        })),
-      show: column.show,
-      project_id: column.project_id,
-      base_id: column.base_id,
-    };
+    const insertObj = extractProps(column, [
+      'fk_view_id',
+      'fk_column_id',
+      'show',
+      'project_id',
+      'base_id',
+    ]);
+
+    insertObj.order =
+      column?.order ??
+      (await ncMeta.metaGetNextOrder(MetaTable.GRID_VIEW_COLUMNS, {
+        fk_view_id: column.fk_view_id,
+      }));
 
     if (!(column.project_id && column.base_id)) {
       const viewRef = await View.get(column.fk_view_id, ncMeta);
diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index e90e1e01c2..ac640dc8c7 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -122,32 +122,44 @@ export default class Hook implements HookType {
     >,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      fk_model_id: hook.fk_model_id,
-      title: hook.title,
-      description: hook.description,
-      env: hook.env,
-      type: hook.type,
-      event: hook.event?.toLowerCase?.(),
-      operation: hook.operation?.toLowerCase?.(),
-      async: hook.async,
-      payload: !!hook.payload,
-      url: hook.url,
-      headers: hook.headers,
-      condition: hook.condition,
-      notification:
-        hook.notification && typeof hook.notification === 'object'
-          ? JSON.stringify(hook.notification)
-          : hook.notification,
-      retries: hook.retries,
-      retry_interval: hook.retry_interval,
-      timeout: hook.timeout,
-      active: hook.active,
-      project_id: hook.project_id,
-      base_id: hook.base_id,
-      created_at: hook.created_at,
-      updated_at: hook.updated_at,
-    };
+    const insertObj = extractProps(hook, [
+      'fk_model_id',
+      'title',
+      'description',
+      'env',
+      'type',
+      'event',
+      'operation',
+      'async',
+      'url',
+      'headers',
+      'notification',
+      'retries',
+      'retry_interval',
+      'timeout',
+      'active',
+      'project_id',
+      'base_id',
+      'created_at',
+      'updated_at',
+    ]);
+
+    if (insertObj.event) {
+      insertObj.event = insertObj.event.toLowerCase() as 'after' | 'before';
+    }
+
+    if (insertObj.operation) {
+      insertObj.operation = insertObj.operation.toLowerCase() as
+        | 'insert'
+        | 'delete'
+        | 'update';
+    }
+
+    if (insertObj.notification) {
+      insertObj.notification && typeof insertObj.notification === 'object'
+        ? JSON.stringify(insertObj.notification)
+        : insertObj.notification;
+    }
 
     if (!(hook.project_id && hook.base_id)) {
       const model = await Model.getByIdOrName({ id: hook.fk_model_id }, ncMeta);
diff --git a/packages/nocodb/src/lib/models/HookFilter.ts b/packages/nocodb/src/lib/models/HookFilter.ts
index b9a1d36e18..81167954d2 100644
--- a/packages/nocodb/src/lib/models/HookFilter.ts
+++ b/packages/nocodb/src/lib/models/HookFilter.ts
@@ -49,20 +49,19 @@ export default class Filter {
     filter: Partial<FilterType>,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      id: filter.id,
-      fk_view_id: filter.fk_view_id,
-      fk_column_id: filter.fk_column_id,
-      comparison_op: filter.comparison_op,
-      value: filter.value,
-      fk_parent_id: filter.fk_parent_id,
-
-      is_group: filter.is_group,
-      logical_op: filter.logical_op,
+    const insertObj = extractProps(filter, [
+      'id',
+      'fk_view_id',
+      'fk_column_id',
+      'comparison_op',
+      'value',
+      'fk_parent_id',
+      'is_group',
+      'logical_op',
+      'project_id',
+      'base_id',
+    ]);
 
-      project_id: filter.project_id,
-      base_id: filter.base_id,
-    };
     if (!(filter.project_id && filter.base_id)) {
       const model = await Column.get({ colId: filter.fk_column_id }, ncMeta);
       insertObj.project_id = model.project_id;
diff --git a/packages/nocodb/src/lib/models/KanbanView.ts b/packages/nocodb/src/lib/models/KanbanView.ts
index 9097478733..d72a1b46f8 100644
--- a/packages/nocodb/src/lib/models/KanbanView.ts
+++ b/packages/nocodb/src/lib/models/KanbanView.ts
@@ -64,16 +64,17 @@ export default class KanbanView implements KanbanType {
       .then((v) => v?.getModel(ncMeta))
       .then((m) => m.getColumns(ncMeta));
 
-    const insertObj = {
-      project_id: view.project_id,
-      base_id: view.base_id,
-      fk_view_id: view.fk_view_id,
-      fk_grp_col_id: view.fk_grp_col_id,
-      fk_cover_image_col_id:
-        view?.fk_cover_image_col_id ||
-        columns?.find((c) => c.uidt === UITypes.Attachment)?.id,
-      meta: view.meta,
-    };
+    const insertObj = extractProps(view, [
+      'project_id',
+      'base_id',
+      'fk_view_id',
+      'fk_grp_col_id',
+      'meta',
+    ]);
+
+    insertObj.fk_cover_image_col_id =
+      view?.fk_cover_image_col_id ||
+      columns?.find((c) => c.uidt === UITypes.Attachment)?.id;
 
     if (!(view.project_id && view.base_id)) {
       const viewRef = await View.get(view.fk_view_id);
diff --git a/packages/nocodb/src/lib/models/KanbanViewColumn.ts b/packages/nocodb/src/lib/models/KanbanViewColumn.ts
index 87d58fe285..c86e5a2f14 100644
--- a/packages/nocodb/src/lib/models/KanbanViewColumn.ts
+++ b/packages/nocodb/src/lib/models/KanbanViewColumn.ts
@@ -2,6 +2,7 @@ import Noco from '../Noco';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import View from './View';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class KanbanViewColumn {
   id: string;
@@ -40,16 +41,20 @@ export default class KanbanViewColumn {
     return view && new KanbanViewColumn(view);
   }
   static async insert(column: Partial<KanbanViewColumn>, ncMeta = Noco.ncMeta) {
-    const insertObj = {
-      fk_view_id: column.fk_view_id,
-      fk_column_id: column.fk_column_id,
-      order: await ncMeta.metaGetNextOrder(MetaTable.KANBAN_VIEW_COLUMNS, {
+    const insertObj = extractProps(column, [
+      'fk_view_id',
+      'fk_column_id',
+      'show',
+      'project_id',
+      'base_id',
+    ]);
+
+    insertObj.order = await ncMeta.metaGetNextOrder(
+      MetaTable.KANBAN_VIEW_COLUMNS,
+      {
         fk_view_id: column.fk_view_id,
-      }),
-      show: column.show,
-      project_id: column.project_id,
-      base_id: column.base_id,
-    };
+      }
+    );
 
     if (!(column.project_id && column.base_id)) {
       const viewRef = await View.get(column.fk_view_id, ncMeta);
diff --git a/packages/nocodb/src/lib/models/LinkToAnotherRecordColumn.ts b/packages/nocodb/src/lib/models/LinkToAnotherRecordColumn.ts
index 1234ed1e86..bbed654ea2 100644
--- a/packages/nocodb/src/lib/models/LinkToAnotherRecordColumn.ts
+++ b/packages/nocodb/src/lib/models/LinkToAnotherRecordColumn.ts
@@ -1,9 +1,9 @@
 import Noco from '../Noco';
 import Column from './Column';
 import Model from './Model';
-// import NocoCache from '../cache/NocoCache';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class LinkToAnotherRecordColumn {
   fk_column_id?: string;
@@ -89,27 +89,21 @@ export default class LinkToAnotherRecordColumn {
     data: Partial<LinkToAnotherRecordColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_RELATIONS, {
-      fk_column_id: data.fk_column_id,
-
-      // ref_db_alias
-      type: data.type,
-      // db_type:
-
-      fk_child_column_id: data.fk_child_column_id,
-      fk_parent_column_id: data.fk_parent_column_id,
-
-      fk_mm_model_id: data.fk_mm_model_id,
-      fk_mm_child_column_id: data.fk_mm_child_column_id,
-      fk_mm_parent_column_id: data.fk_mm_parent_column_id,
-
-      ur: data.ur,
-      dr: data.dr,
-
-      fk_index_name: data.fk_index_name,
-      fk_related_model_id: data.fk_related_model_id,
-      virtual: data.virtual,
-    });
+    const insertObj = extractProps(data, [
+      'fk_column_id',
+      'type',
+      'fk_child_column_id',
+      'fk_parent_column_id',
+      'fk_mm_model_id',
+      'fk_mm_child_column_id',
+      'fk_mm_parent_column_id',
+      'ur',
+      'dr',
+      'fk_index_name',
+      'fk_related_model_id',
+      'virtual',
+    ]);
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_RELATIONS, insertObj);
     return this.read(data.fk_column_id, ncMeta);
   }
 
diff --git a/packages/nocodb/src/lib/models/LookupColumn.ts b/packages/nocodb/src/lib/models/LookupColumn.ts
index 3737afc5bd..91c51e9a67 100644
--- a/packages/nocodb/src/lib/models/LookupColumn.ts
+++ b/packages/nocodb/src/lib/models/LookupColumn.ts
@@ -1,8 +1,8 @@
 import Noco from '../Noco';
-// import NocoCache from '../cache/NocoCache';
 import Column from './Column';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class LookupColumn {
   fk_relation_column_id: string;
@@ -29,11 +29,13 @@ export default class LookupColumn {
     data: Partial<LookupColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_LOOKUP, {
-      fk_column_id: data.fk_column_id,
-      fk_relation_column_id: data.fk_relation_column_id,
-      fk_lookup_column_id: data.fk_lookup_column_id,
-    });
+    const insertObj = extractProps(data, [
+      'fk_column_id',
+      'fk_relation_column_id',
+      'fk_lookup_column_id',
+    ]);
+
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_LOOKUP, insertObj);
 
     await NocoCache.appendToList(
       CacheScope.COL_LOOKUP,
diff --git a/packages/nocodb/src/lib/models/Model.ts b/packages/nocodb/src/lib/models/Model.ts
index c22c115d02..4e937d14b6 100644
--- a/packages/nocodb/src/lib/models/Model.ts
+++ b/packages/nocodb/src/lib/models/Model.ts
@@ -22,6 +22,7 @@ import View from './View';
 import { NcError } from '../meta/helpers/catchError';
 import Audit from './Audit';
 import { sanitize } from '../db/sql-data-mapper/lib/sql/helpers/sanitize';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Model implements TableType {
   copy_enabled: boolean;
@@ -103,25 +104,39 @@ export default class Model implements TableType {
     },
     ncMeta = Noco.ncMeta
   ) {
+    const insertObj = extractProps(model, [
+      'table_name',
+      'title',
+      'mm',
+      'order',
+      'created_at',
+      'updated_at',
+      'id',
+    ]);
+
+    if (insertObj.mm) {
+      insertObj.mm = !!insertObj.mm;
+    }
+
+    if (!insertObj.order) {
+      insertObj.order = await ncMeta.metaGetNextOrder(
+        MetaTable.FORM_VIEW_COLUMNS,
+        {
+          project_id: projectId,
+          base_id: baseId,
+        }
+      );
+    }
+
+    if (!insertObj.type) {
+      insertObj.type = ModelTypes.TABLE;
+    }
+
     const { id } = await ncMeta.metaInsert2(
       projectId,
       baseId,
       MetaTable.MODELS,
-      {
-        table_name: model.table_name,
-        title: model.title,
-        mm: !!model.mm,
-        order:
-          model.order ||
-          (await ncMeta.metaGetNextOrder(MetaTable.FORM_VIEW_COLUMNS, {
-            project_id: projectId,
-            base_id: baseId,
-          })),
-        type: model.type || ModelTypes.TABLE,
-        created_at: model.created_at,
-        updated_at: model.updated_at,
-        id: model.id,
-      }
+      insertObj
     );
 
     await NocoCache.appendToList(
diff --git a/packages/nocodb/src/lib/models/ModelRoleVisibility.ts b/packages/nocodb/src/lib/models/ModelRoleVisibility.ts
index f6a6a1df2f..74f6515315 100644
--- a/packages/nocodb/src/lib/models/ModelRoleVisibility.ts
+++ b/packages/nocodb/src/lib/models/ModelRoleVisibility.ts
@@ -8,6 +8,7 @@ import {
 } from '../utils/globals';
 import NocoCache from '../cache/NocoCache';
 import View from './View';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class ModelRoleVisibility implements ModelRoleVisibilityType {
   id?: string;
@@ -134,17 +135,17 @@ export default class ModelRoleVisibility implements ModelRoleVisibilityType {
     >,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      role: body.role,
-      disabled: body.disabled,
-      fk_view_id: body.fk_view_id,
-      project_id: body.project_id,
-      base_id: body.base_id,
-      created_at: body.created_at,
-      updated_at: body.updated_at,
-    };
+    const insertObj = extractProps(body, [
+      'role',
+      'disabled',
+      'fk_view_id',
+      'project_id',
+      'base_id',
+      'created_at',
+      'updated_at',
+    ]);
 
-    if (!(body.project_id && body.base_id)) {
+    if (!(insertObj.project_id && insertObj.base_id)) {
       const view = await View.get(body.fk_view_id, ncMeta);
       insertObj.project_id = view.project_id;
       insertObj.base_id = view.base_id;
diff --git a/packages/nocodb/src/lib/models/Project.ts b/packages/nocodb/src/lib/models/Project.ts
index cf1cdf5a4d..ec69d51b88 100644
--- a/packages/nocodb/src/lib/models/Project.ts
+++ b/packages/nocodb/src/lib/models/Project.ts
@@ -36,25 +36,27 @@ export default class Project implements ProjectType {
   }
 
   public static async createProject(
-    projectBody: ProjectType & {
+    project: ProjectType & {
       created_at?;
       updated_at?;
     },
     ncMeta = Noco.ncMeta
   ): Promise<Project> {
+    const insertObj = extractProps(project, [
+      'id',
+      'title',
+      'prefix',
+      'description',
+      'is_meta',
+      'created_at',
+      'updated_at',
+    ]);
+
     const { id: projectId } = await ncMeta.metaInsert2(
       null,
       null,
       MetaTable.PROJECT,
-      {
-        id: projectBody?.id,
-        title: projectBody.title,
-        prefix: projectBody.prefix,
-        description: projectBody.description,
-        is_meta: projectBody.is_meta,
-        created_at: projectBody.created_at,
-        updated_at: projectBody.updated_at,
-      }
+      insertObj
     );
 
     await NocoCache.appendToList(
@@ -63,7 +65,7 @@ export default class Project implements ProjectType {
       `${CacheScope.PROJECT}:${projectId}`
     );
 
-    for (const base of projectBody.bases) {
+    for (const base of project.bases) {
       await Base.createBase(
         {
           type: base.config?.client,
diff --git a/packages/nocodb/src/lib/models/ProjectUser.ts b/packages/nocodb/src/lib/models/ProjectUser.ts
index c4c7454a31..d62f5490e8 100644
--- a/packages/nocodb/src/lib/models/ProjectUser.ts
+++ b/packages/nocodb/src/lib/models/ProjectUser.ts
@@ -8,6 +8,7 @@ import {
 import Noco from '../Noco';
 import NocoCache from '../cache/NocoCache';
 import User from './User';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class ProjectUser {
   project_id: string;
@@ -22,17 +23,19 @@ export default class ProjectUser {
     projectUser: Partial<ProjectUser & { created_at?: any; updated_at?: any }>,
     ncMeta = Noco.ncMeta
   ) {
+    const insertObj = extractProps(projectUser, [
+      'fk_user_id',
+      'project_id',
+      'roles',
+      'created_at',
+      'updated_at',
+    ]);
+
     const { project_id, fk_user_id } = await ncMeta.metaInsert2(
       null,
       null,
       MetaTable.PROJECT_USERS,
-      {
-        fk_user_id: projectUser.fk_user_id,
-        project_id: projectUser.project_id,
-        roles: projectUser.roles,
-        created_at: projectUser.created_at,
-        updated_at: projectUser.updated_at,
-      },
+      insertObj,
       true
     );
 
diff --git a/packages/nocodb/src/lib/models/QrCodeColumn.ts b/packages/nocodb/src/lib/models/QrCodeColumn.ts
index 0dc39f8d2d..9610b69f06 100644
--- a/packages/nocodb/src/lib/models/QrCodeColumn.ts
+++ b/packages/nocodb/src/lib/models/QrCodeColumn.ts
@@ -12,15 +12,17 @@ export default class QrCodeColumn {
   }
 
   public static async insert(
-    data: Partial<QrCodeColumn>,
+    qrCode: Partial<QrCodeColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_QRCODE, {
-      fk_column_id: data.fk_column_id,
-      fk_qr_value_column_id: data.fk_qr_value_column_id,
-    });
+    const insertObj = extractProps(qrCode, [
+      'fk_column_id',
+      'fk_qr_value_column_id',
+    ]);
+
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_QRCODE, insertObj);
 
-    return this.read(data.fk_column_id, ncMeta);
+    return this.read(qrCode.fk_column_id, ncMeta);
   }
   public static async read(columnId: string, ncMeta = Noco.ncMeta) {
     let column =
diff --git a/packages/nocodb/src/lib/models/RollupColumn.ts b/packages/nocodb/src/lib/models/RollupColumn.ts
index a66512300a..14fc0d3366 100644
--- a/packages/nocodb/src/lib/models/RollupColumn.ts
+++ b/packages/nocodb/src/lib/models/RollupColumn.ts
@@ -2,6 +2,7 @@ import Noco from '../Noco';
 import Column from './Column';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
 import NocoCache from '../cache/NocoCache';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class RollupColumn {
   fk_column_id;
@@ -20,12 +21,13 @@ export default class RollupColumn {
     data: Partial<RollupColumn>,
     ncMeta = Noco.ncMeta
   ) {
-    await ncMeta.metaInsert2(null, null, MetaTable.COL_ROLLUP, {
-      fk_column_id: data.fk_column_id,
-      fk_relation_column_id: data.fk_relation_column_id,
-      fk_rollup_column_id: data.fk_rollup_column_id,
-      rollup_function: data.rollup_function,
-    });
+    const insertObj = extractProps(data, [
+      'fk_column_id',
+      'fk_relation_column_id',
+      'fk_rollup_column_id',
+      'rollup_function',
+    ]);
+    await ncMeta.metaInsert2(null, null, MetaTable.COL_ROLLUP, insertObj);
 
     await NocoCache.appendToList(
       CacheScope.COL_ROLLUP,
diff --git a/packages/nocodb/src/lib/models/SelectOption.ts b/packages/nocodb/src/lib/models/SelectOption.ts
index ac2af034c9..baf25e2672 100644
--- a/packages/nocodb/src/lib/models/SelectOption.ts
+++ b/packages/nocodb/src/lib/models/SelectOption.ts
@@ -1,6 +1,7 @@
 import Noco from '../Noco';
 import NocoCache from '../cache/NocoCache';
 import { CacheGetType, CacheScope, MetaTable } from '../utils/globals';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class SelectOption {
   title: string;
@@ -16,11 +17,23 @@ export default class SelectOption {
     data: Partial<SelectOption>,
     ncMeta = Noco.ncMeta
   ) {
+    // title: string;
+    // fk_column_id: string;
+    // color: string;
+    // order: number;
+
+    const insertObj = extractProps(data, [
+      'title',
+      'fk_column_id',
+      'color',
+      'order',
+    ]);
+
     const { id } = await ncMeta.metaInsert2(
       null,
       null,
       MetaTable.COL_SELECT_OPTIONS,
-      data
+      insertObj
     );
 
     await NocoCache.appendToList(
diff --git a/packages/nocodb/src/lib/models/Sort.ts b/packages/nocodb/src/lib/models/Sort.ts
index 355ca6a725..1d90324f52 100644
--- a/packages/nocodb/src/lib/models/Sort.ts
+++ b/packages/nocodb/src/lib/models/Sort.ts
@@ -9,6 +9,7 @@ import {
 } from '../utils/globals';
 import NocoCache from '../cache/NocoCache';
 import { SortType } from 'nocodb-sdk';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class Sort {
   id: string;
@@ -35,11 +36,20 @@ export default class Sort {
   }
 
   public static async insert(
-    sortObj: Partial<Sort> & { push_to_top?: boolean },
+    sortObj: Partial<Sort> & { push_to_top?: boolean; order?: number },
     ncMeta = Noco.ncMeta
   ) {
+    const insertObj = extractProps(sortObj, [
+      'id',
+      'fk_view_id',
+      'fk_column_id',
+      'direction',
+      'project_id',
+      'base_id',
+    ]);
+
     // todo: implement a generic function
-    const order = sortObj.push_to_top
+    insertObj.order = sortObj.push_to_top
       ? 1
       : (+(
           await ncMeta
@@ -50,16 +60,6 @@ export default class Sort {
             })
             .first()
         )?.order || 0) + 1;
-
-    const insertObj = {
-      id: sortObj.id,
-      fk_view_id: sortObj.fk_view_id,
-      fk_column_id: sortObj.fk_column_id,
-      direction: sortObj.direction,
-      project_id: sortObj.project_id,
-      base_id: sortObj.base_id,
-      order,
-    };
     if (!(sortObj.project_id && sortObj.base_id)) {
       const model = await Column.get({ colId: sortObj.fk_column_id }, ncMeta);
       insertObj.project_id = model.project_id;
diff --git a/packages/nocodb/src/lib/models/SyncLogs.ts b/packages/nocodb/src/lib/models/SyncLogs.ts
index 035debc75e..d10e814c56 100644
--- a/packages/nocodb/src/lib/models/SyncLogs.ts
+++ b/packages/nocodb/src/lib/models/SyncLogs.ts
@@ -1,5 +1,6 @@
 import Noco from '../Noco';
 import { MetaTable } from '../utils/globals';
+import { extractProps } from '../meta/helpers/extractProps';
 
 export default class SyncLogs {
   id?: string;
@@ -34,13 +35,13 @@ export default class SyncLogs {
     >,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      project_id: syncLog?.project_id,
-      fk_sync_source_id: syncLog?.fk_sync_source_id,
-      time_taken: syncLog?.time_taken,
-      status: syncLog?.status,
-      status_details: syncLog?.status_details,
-    };
+    const insertObj = extractProps(syncLog, [
+      'project_id',
+      'fk_sync_source_id',
+      'time_taken',
+      'status',
+      'status_details',
+    ]);
 
     const { id } = await ncMeta.metaInsert2(
       null,
diff --git a/packages/nocodb/src/lib/models/SyncSource.ts b/packages/nocodb/src/lib/models/SyncSource.ts
index 4f169fae12..25292a3576 100644
--- a/packages/nocodb/src/lib/models/SyncSource.ts
+++ b/packages/nocodb/src/lib/models/SyncSource.ts
@@ -73,15 +73,15 @@ export default class SyncSource {
     >,
     ncMeta = Noco.ncMeta
   ) {
-    const insertObj = {
-      id: syncSource?.id,
-      title: syncSource?.title,
-      type: syncSource?.type,
-      details: syncSource?.details,
-      project_id: syncSource?.project_id,
-      base_id: syncSource?.base_id,
-      fk_user_id: syncSource?.fk_user_id,
-    };
+    const insertObj = extractProps(syncSource, [
+      'id',
+      'title',
+      'type',
+      'details',
+      'project_id',
+      'base_id',
+      'fk_user_id',
+    ]);
 
     if (insertObj.details && typeof insertObj.details === 'object') {
       insertObj.details = JSON.stringify(insertObj.details);
diff --git a/packages/nocodb/src/lib/models/View.ts b/packages/nocodb/src/lib/models/View.ts
index 64a2df528c..b58fe429ac 100644
--- a/packages/nocodb/src/lib/models/View.ts
+++ b/packages/nocodb/src/lib/models/View.ts
@@ -244,25 +244,29 @@ export default class View implements ViewType {
       },
     ncMeta = Noco.ncMeta
   ) {
+    const insertObj = extractProps(view, [
+      'id',
+      'title',
+      'is_default',
+      'type',
+      'fk_model_id',
+      'project_id',
+      'base_id',
+      'created_at',
+      'updated_at',
+      'meta',
+    ]);
+
     // get order value
-    const order = await ncMeta.metaGetNextOrder(MetaTable.VIEWS, {
+    insertObj.order = await ncMeta.metaGetNextOrder(MetaTable.VIEWS, {
       fk_model_id: view.fk_model_id,
     });
 
-    const insertObj = {
-      id: view.id,
-      title: view.title,
-      show: true,
-      is_default: view.is_default,
-      order,
-      type: view.type,
-      fk_model_id: view.fk_model_id,
-      project_id: view.project_id,
-      base_id: view.base_id,
-      created_at: view.created_at,
-      updated_at: view.updated_at,
-      meta: view.meta ?? {},
-    };
+    insertObj.show = true;
+
+    if (!insertObj.meta) {
+      insertObj.meta = {};
+    }
 
     insertObj.meta = stringifyMetaProp(insertObj);
 

From a5ad94704fdde7c9a88272acf36a193b91c5de4d Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 19:42:18 +0800
Subject: [PATCH 09/25] fix(nocodb): replace notification

---
 packages/nocodb/src/lib/models/Hook.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index ac640dc8c7..1fdda44cae 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -230,6 +230,8 @@ export default class Hook implements HookType {
     if (o) {
       // update data
       o = { ...o, ...updateObj };
+      // replace notification
+      o.notification = updateObj.notification;
       // set cache
       await NocoCache.set(key, o);
     }

From b7a4b4c5aaf9a8f9a1b61936a291dad14d84943c Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 19:42:57 +0800
Subject: [PATCH 10/25] refactor(nocodb): updateObj.notification

---
 packages/nocodb/src/lib/models/Hook.ts | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index 1fdda44cae..f66f3490dc 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -218,10 +218,8 @@ export default class Hook implements HookType {
         | 'update';
     }
 
-    if (updateObj.notification) {
-      updateObj.notification && typeof updateObj.notification === 'object'
-        ? JSON.stringify(updateObj.notification)
-        : updateObj.notification;
+    if (updateObj.notification && typeof updateObj.notification === 'object') {
+      uupdateObj.notification = JSON.stringify(updateObj.notification);
     }
 
     // get existing cache

From 7592d94e9df368fe1ea91955fc6eb0e5297bf753 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 19:43:09 +0800
Subject: [PATCH 11/25] fix(nocodb): typo

---
 packages/nocodb/src/lib/models/Hook.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index f66f3490dc..a6c0f37a3c 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -219,7 +219,7 @@ export default class Hook implements HookType {
     }
 
     if (updateObj.notification && typeof updateObj.notification === 'object') {
-      uupdateObj.notification = JSON.stringify(updateObj.notification);
+      updateObj.notification = JSON.stringify(updateObj.notification);
     }
 
     // get existing cache

From 5b126c89779aa2832c41e8f1d6217c2c37ae0266 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 20:04:13 +0800
Subject: [PATCH 12/25] refactor(nocodb): json stringify if input is object
 only

---
 packages/nocodb/src/lib/models/Hook.ts   | 6 ++----
 packages/nocodb/src/lib/models/Plugin.ts | 7 ++-----
 2 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Hook.ts b/packages/nocodb/src/lib/models/Hook.ts
index a6c0f37a3c..7c32d09bbd 100644
--- a/packages/nocodb/src/lib/models/Hook.ts
+++ b/packages/nocodb/src/lib/models/Hook.ts
@@ -155,10 +155,8 @@ export default class Hook implements HookType {
         | 'update';
     }
 
-    if (insertObj.notification) {
-      insertObj.notification && typeof insertObj.notification === 'object'
-        ? JSON.stringify(insertObj.notification)
-        : insertObj.notification;
+    if (insertObj.notification && typeof insertObj.notification === 'object') {
+      insertObj.notification = JSON.stringify(insertObj.notification);
     }
 
     if (!(hook.project_id && hook.base_id)) {
diff --git a/packages/nocodb/src/lib/models/Plugin.ts b/packages/nocodb/src/lib/models/Plugin.ts
index d53757af51..711feb700e 100644
--- a/packages/nocodb/src/lib/models/Plugin.ts
+++ b/packages/nocodb/src/lib/models/Plugin.ts
@@ -59,11 +59,8 @@ export default class Plugin implements PluginType {
   public static async update(pluginId: string, plugin: Partial<PluginType>) {
     const updateObj = extractProps(plugin, ['input', 'active']);
 
-    if (updateObj.input) {
-      updateObj.input =
-        updateObj.input && typeof updateObj.input === 'object'
-          ? JSON.stringify(updateObj.input)
-          : updateObj.input;
+    if (updateObj.input && typeof updateObj.input === 'object') {
+      updateObj.input = JSON.stringify(updateObj.input);
     }
 
     // get existing cache

From e1bee8a068f7c04eea88894db5c3b4b87f6ddd43 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Fri, 17 Feb 2023 15:10:27 +0300
Subject: [PATCH 13/25] fix: map first column as display value if no pk

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 .../src/lib/meta/api/helpers/populateMeta.ts  |  2 ++
 .../meta/helpers/mapDefaultDisplayValue.ts    | 24 ++++++++-----------
 2 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts b/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
index ff7861a529..4fe72f70ac 100644
--- a/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
+++ b/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
@@ -228,6 +228,8 @@ export async function populateMeta(base: Base, project: Project): Promise<any> {
     return async () => {
       const columns = (await sqlClient.columnList({ tn: table.table_name }))
         ?.data?.list;
+      
+      mapDefaultDisplayValue(columns);
 
       /* create nc_models and its rows if it doesn't exists  */
       models2[table.table_name] = await Model.insert(project.id, base.id, {
diff --git a/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts b/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
index 8f12d40d13..3fe911f447 100644
--- a/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
+++ b/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
@@ -4,31 +4,27 @@ export default function mapDefaultDisplayValue<T extends ColumnType>(
   columnsArr: Array<T>
 ): void | T {
   if (!columnsArr.some((column) => column.pv)) {
-    let len = columnsArr.length;
-    let pkIndex = -1;
-
-    while (len--) {
-      if (columnsArr[len].pk) {
-        pkIndex = len;
-        break;
-      }
-    }
+    const pkIndex = columnsArr.findIndex((column) => column.pk)
 
     // if PK is at the end of table
     if (pkIndex === columnsArr.length - 1) {
       if (pkIndex > 0) {
         columnsArr[pkIndex - 1].pv = true;
         return columnsArr[pkIndex - 1];
+      } else if (columnsArr.length > 0) {
+        columnsArr[0].pv = true;
+        return columnsArr[0];
       }
-    }
     // pk is not at the end of table
-    else if (pkIndex > -1) {
+    } else if (pkIndex > -1) { 
       columnsArr[pkIndex + 1].pv = true;
       return columnsArr[pkIndex + 1];
-    }
     //  no pk at all
-    else {
-      // todo:
+    } else {
+      if (columnsArr.length > 0) {
+        columnsArr[0].pv = true;
+        return columnsArr[0];
+      }
     }
   }
 }

From 500c017d752dac143979f2f35fd7c3ac54b6b868 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Fri, 17 Feb 2023 15:12:10 +0300
Subject: [PATCH 14/25] fix: keep single display value per model

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 packages/nocodb/src/lib/models/Model.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Model.ts b/packages/nocodb/src/lib/models/Model.ts
index c22c115d02..e9070d6d76 100644
--- a/packages/nocodb/src/lib/models/Model.ts
+++ b/packages/nocodb/src/lib/models/Model.ts
@@ -561,14 +561,14 @@ export default class Model implements TableType {
     ncMeta = Noco.ncMeta
   ) {
     const model = await this.getWithInfo({ id: tableId });
-    const currentPvCol = model.displayValue;
     const newPvCol = model.columns.find((c) => c.id === columnId);
 
     if (!newPvCol) NcError.badRequest('Column not found');
 
-    if (currentPvCol) {
+    // drop existing primary column/s
+    for (const col of model.columns?.filter((c) => c.pv) || []) {
       // get existing cache
-      const key = `${CacheScope.COLUMN}:${currentPvCol.id}`;
+      const key = `${CacheScope.COLUMN}:${col.id}`;
       const o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT);
       if (o) {
         o.pv = false;
@@ -583,7 +583,7 @@ export default class Model implements TableType {
         {
           pv: false,
         },
-        currentPvCol.id
+        col.id
       );
     }
 

From 95f357d2bf68456721ea3065d74d1a7bffa26813 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Fri, 17 Feb 2023 15:48:23 +0300
Subject: [PATCH 15/25] fix: handle missing display value in upgrader

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 .../ncStickyColumnUpgrader.ts                 | 44 +++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts b/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
index 67b02ce842..0a6e16c21e 100644
--- a/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
+++ b/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
@@ -38,6 +38,50 @@ export default async function ({ ncMeta }: NcUpgraderCtx) {
       view_columns_meta.push(col_meta);
     }
 
+    // if no display value column is set
+    if (!view_columns_meta.some((column) => column.pv)) {
+      const pkIndex = view_columns_meta.findIndex((column) => column.pk)
+  
+      // if PK is at the end of table
+      if (pkIndex === view_columns_meta.length - 1) {
+        if (pkIndex > 0) {
+          await ncMeta.metaUpdate(
+            null,
+            null,
+            MetaTable.COLUMNS,
+            { pv: true },
+            view_columns_meta[pkIndex - 1].id
+          );
+        } else if (view_columns_meta.length > 0) {
+          await ncMeta.metaUpdate(
+            null,
+            null,
+            MetaTable.COLUMNS,
+            { pv: true },
+            view_columns_meta[0].id
+          );
+        }
+      // pk is not at the end of table
+      } else if (pkIndex > -1) { 
+        await ncMeta.metaUpdate(
+          null,
+          null,
+          MetaTable.COLUMNS,
+          { pv: true },
+          view_columns_meta[pkIndex + 1].id
+        );
+      //  no pk at all
+      } else if (view_columns_meta.length > 0) {
+        await ncMeta.metaUpdate(
+          null,
+          null,
+          MetaTable.COLUMNS,
+          { pv: true },
+          view_columns_meta[0].id
+        );
+      }
+    }
+
     const primary_value_column_meta = view_columns_meta.find((col) => col.pv);
 
     if (primary_value_column_meta) {

From a2bfae7e47ef795c0d8c37be758452b204ec6077 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Mon, 20 Feb 2023 15:22:16 +0300
Subject: [PATCH 16/25] fix: update version of sticky column upgrader

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 packages/nocodb/src/lib/Noco.ts                        | 2 +-
 packages/nocodb/src/lib/version-upgrader/NcUpgrader.ts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/nocodb/src/lib/Noco.ts b/packages/nocodb/src/lib/Noco.ts
index 6c7ead8acd..b66287a446 100644
--- a/packages/nocodb/src/lib/Noco.ts
+++ b/packages/nocodb/src/lib/Noco.ts
@@ -105,7 +105,7 @@ export default class Noco {
   constructor() {
     process.env.PORT = process.env.PORT || '8080';
     // todo: move
-    process.env.NC_VERSION = '0104004';
+    process.env.NC_VERSION = '0105002';
 
     // if env variable NC_MINIMAL_DBS is set, then disable project creation with external sources
     if (process.env.NC_MINIMAL_DBS) {
diff --git a/packages/nocodb/src/lib/version-upgrader/NcUpgrader.ts b/packages/nocodb/src/lib/version-upgrader/NcUpgrader.ts
index cd84032692..c125701a4c 100644
--- a/packages/nocodb/src/lib/version-upgrader/NcUpgrader.ts
+++ b/packages/nocodb/src/lib/version-upgrader/NcUpgrader.ts
@@ -43,8 +43,8 @@ export default class NcUpgrader {
         { name: '0100002', handler: ncFilterUpgrader },
         { name: '0101002', handler: ncAttachmentUpgrader },
         { name: '0104002', handler: ncAttachmentUpgrader_0104002 },
-        { name: '0104003', handler: ncStickyColumnUpgrader },
         { name: '0104004', handler: ncFilterUpgrader_0104004 },
+        { name: '0105002', handler: ncStickyColumnUpgrader },
       ];
       if (!(await ctx.ncMeta.knexConnection?.schema?.hasTable?.('nc_store'))) {
         return;

From 883d4214a85c5d01e3178426acc02e7e047e6c37 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Mon, 20 Feb 2023 18:28:56 +0530
Subject: [PATCH 17/25] feat(gui): add label about the multi select
 implementation

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../nc-gui/components/virtual-cell/components/ListItems.vue  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/packages/nc-gui/components/virtual-cell/components/ListItems.vue b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
index b779171289..6122bf9357 100644
--- a/packages/nc-gui/components/virtual-cell/components/ListItems.vue
+++ b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
@@ -246,8 +246,13 @@ watch(vModel, (nextVal) => {
             show-less-items
           />
         </div>
+
+        <div class="text-xs text-gray-400 text-center px-2 mt-4 pb-0">
+          * Use <kbd>ALT</kbd> / <kbd>OPT</kbd> + <kbd>Click</kbd> to select multiple records
+        </div>
       </template>
 
+
       <a-empty v-else class="my-10" :image="Empty.PRESENTED_IMAGE_SIMPLE" />
 
       <Suspense>

From 746398bb7e2dffdd97b2f2cebf11b2356b0a57e8 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 21:13:04 +0800
Subject: [PATCH 18/25] fix(nocodb): add missing id

---
 packages/nocodb/src/lib/models/SelectOption.ts | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/packages/nocodb/src/lib/models/SelectOption.ts b/packages/nocodb/src/lib/models/SelectOption.ts
index baf25e2672..fa24d06e50 100644
--- a/packages/nocodb/src/lib/models/SelectOption.ts
+++ b/packages/nocodb/src/lib/models/SelectOption.ts
@@ -17,12 +17,8 @@ export default class SelectOption {
     data: Partial<SelectOption>,
     ncMeta = Noco.ncMeta
   ) {
-    // title: string;
-    // fk_column_id: string;
-    // color: string;
-    // order: number;
-
     const insertObj = extractProps(data, [
+      'id',
       'title',
       'fk_column_id',
       'color',

From 2bee197f29c1396ec56c965541839575fbe3cd1c Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 22:13:26 +0800
Subject: [PATCH 19/25] fix(nocodb): apply !!insertObj.mm even undefined

---
 packages/nocodb/src/lib/models/Model.ts | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/packages/nocodb/src/lib/models/Model.ts b/packages/nocodb/src/lib/models/Model.ts
index 4e937d14b6..2412d6259d 100644
--- a/packages/nocodb/src/lib/models/Model.ts
+++ b/packages/nocodb/src/lib/models/Model.ts
@@ -114,9 +114,7 @@ export default class Model implements TableType {
       'id',
     ]);
 
-    if (insertObj.mm) {
-      insertObj.mm = !!insertObj.mm;
-    }
+    insertObj.mm = !!insertObj.mm;
 
     if (!insertObj.order) {
       insertObj.order = await ncMeta.metaGetNextOrder(

From 0141c5024ed632929692d6187cfa4114031a66c4 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 23:42:20 +0800
Subject: [PATCH 20/25] fix(nocodb): add missing meta in extractProps

---
 packages/nocodb/src/lib/models/FormView.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/nocodb/src/lib/models/FormView.ts b/packages/nocodb/src/lib/models/FormView.ts
index 5f10ce1803..ad155f4434 100644
--- a/packages/nocodb/src/lib/models/FormView.ts
+++ b/packages/nocodb/src/lib/models/FormView.ts
@@ -68,6 +68,7 @@ export default class FormView implements FormType {
       'logo_url',
       'submit_another_form',
       'show_blank_form',
+      'meta',
     ]);
     if (insertObj.meta) {
       insertObj.meta = serializeJSON(insertObj.meta);
@@ -103,6 +104,7 @@ export default class FormView implements FormType {
       'show_blank_form',
       'meta',
     ]);
+
     if (o) {
       o = { ...o, ...updateObj };
       // set cache

From 360631e2aab2064b94c33b6e0f2bbb1e4ddd1f9f Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 23:45:28 +0800
Subject: [PATCH 21/25] refactor(nocodb): stringify if input is object

---
 packages/nocodb/src/lib/models/KanbanView.ts | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/packages/nocodb/src/lib/models/KanbanView.ts b/packages/nocodb/src/lib/models/KanbanView.ts
index d72a1b46f8..6ac1b65d09 100644
--- a/packages/nocodb/src/lib/models/KanbanView.ts
+++ b/packages/nocodb/src/lib/models/KanbanView.ts
@@ -108,11 +108,8 @@ export default class KanbanView implements KanbanType {
       'meta',
     ]);
 
-    if (updateObj.meta) {
-      updateObj.meta =
-        typeof updateObj.meta === 'string'
-          ? updateObj.meta
-          : JSON.stringify(updateObj.meta ?? {});
+    if (updateObj.meta && typeof updateObj.meta === 'object') {
+      updateObj.meta = JSON.stringify(updateObj.meta ?? {});
     }
 
     if (o) {

From fc4ac7a157b02ed526c7776e216dc6bbdf524bf5 Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Mon, 20 Feb 2023 23:46:53 +0800
Subject: [PATCH 22/25] fix(nocodb): add missing type props

---
 packages/nocodb/src/lib/models/Model.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/nocodb/src/lib/models/Model.ts b/packages/nocodb/src/lib/models/Model.ts
index 2412d6259d..43b5c7a77f 100644
--- a/packages/nocodb/src/lib/models/Model.ts
+++ b/packages/nocodb/src/lib/models/Model.ts
@@ -109,6 +109,7 @@ export default class Model implements TableType {
       'title',
       'mm',
       'order',
+      'type',
       'created_at',
       'updated_at',
       'id',

From 9c6d2af45a71d43f142386607bfa12d66b3a6a24 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Tue, 21 Feb 2023 11:31:27 +0530
Subject: [PATCH 23/25] chore: lint

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../components/virtual-cell/components/ListItems.vue  |  1 -
 packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts          | 11 ++++++++++-
 .../src/lib/db/sql-migrator/lib/KnexMigratorv2.ts     |  6 +++---
 .../src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts | 11 ++++++++++-
 .../nocodb/src/lib/meta/api/helpers/populateMeta.ts   |  2 +-
 .../src/lib/meta/helpers/mapDefaultDisplayValue.ts    |  8 ++++----
 .../lib/version-upgrader/ncStickyColumnUpgrader.ts    | 10 +++++-----
 7 files changed, 33 insertions(+), 16 deletions(-)

diff --git a/packages/nc-gui/components/virtual-cell/components/ListItems.vue b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
index 6122bf9357..2df6f5702d 100644
--- a/packages/nc-gui/components/virtual-cell/components/ListItems.vue
+++ b/packages/nc-gui/components/virtual-cell/components/ListItems.vue
@@ -252,7 +252,6 @@ watch(vModel, (nextVal) => {
         </div>
       </template>
 
-
       <a-empty v-else class="my-10" :image="Empty.PRESENTED_IMAGE_SIMPLE" />
 
       <Suspense>
diff --git a/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts b/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
index 02b5c4e343..3f6a85b9c2 100644
--- a/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
+++ b/packages/nocodb/src/lib/db/sql-mgr/SqlMgr.ts
@@ -345,7 +345,16 @@ export default class SqlMgr {
    * @returns
    * @memberof SqlMgr
    */
-  public async projectGetSqlClient(args): Promise<SnowflakeClient | MysqlClient | SqliteClient | MssqlClient | OracleClient | PGClient> {
+  public async projectGetSqlClient(
+    args
+  ): Promise<
+    | SnowflakeClient
+    | MysqlClient
+    | SqliteClient
+    | MssqlClient
+    | OracleClient
+    | PGClient
+  > {
     const func = this.projectGetSqlClient.name;
     log.api(`${func}:args:`, args);
 
diff --git a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
index cdf4d8f1f1..d32a40d1d0 100644
--- a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
+++ b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2.ts
@@ -1,7 +1,7 @@
 import fs from 'fs';
 import path from 'path';
 import { promisify } from 'util';
-import { Knex } from  'knex';
+import { Knex } from 'knex';
 
 import glob from 'glob';
 import SqlClientFactory from '../../sql-client/lib/SqlClientFactory';
@@ -754,7 +754,7 @@ export default class KnexMigratorv2 {
             const vm = this;
 
             const trx = sqlClient.knex.isTransaction
-              ? sqlClient.knex as Knex.Transaction
+              ? (sqlClient.knex as Knex.Transaction)
               : await sqlClient.knex.transaction();
             try {
               for (const query of upStatements) {
@@ -921,7 +921,7 @@ export default class KnexMigratorv2 {
           const vm = this;
 
           const trx = sqlClient.knex.isTransaction
-            ? sqlClient.knex as Knex.Transaction
+            ? (sqlClient.knex as Knex.Transaction)
             : await sqlClient.knex.transaction();
           try {
             for (const query of downStatements) {
diff --git a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
index 037cf6d559..87204964a0 100644
--- a/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
+++ b/packages/nocodb/src/lib/db/sql-migrator/lib/KnexMigratorv2Tans.ts
@@ -31,7 +31,16 @@ export default class KnexMigratorv2Tans extends KnexMigratorv2 {
   protected get metaDb(): XKnex {
     return this.ncMeta.knex || Noco.ncMeta.knex;
   }
-  protected async getSqlClient(base: Base): Promise<SnowflakeClient | MysqlClient | SqliteClient | MssqlClient | OracleClient | PGClient> {
+  protected async getSqlClient(
+    base: Base
+  ): Promise<
+    | SnowflakeClient
+    | MysqlClient
+    | SqliteClient
+    | MssqlClient
+    | OracleClient
+    | PGClient
+  > {
     return this.sqlClient || NcConnectionMgrv2.getSqlClient(base);
   }
 }
diff --git a/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts b/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
index 4fe72f70ac..ffe246a131 100644
--- a/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
+++ b/packages/nocodb/src/lib/meta/api/helpers/populateMeta.ts
@@ -228,7 +228,7 @@ export async function populateMeta(base: Base, project: Project): Promise<any> {
     return async () => {
       const columns = (await sqlClient.columnList({ tn: table.table_name }))
         ?.data?.list;
-      
+
       mapDefaultDisplayValue(columns);
 
       /* create nc_models and its rows if it doesn't exists  */
diff --git a/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts b/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
index 3fe911f447..a661f21818 100644
--- a/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
+++ b/packages/nocodb/src/lib/meta/helpers/mapDefaultDisplayValue.ts
@@ -4,7 +4,7 @@ export default function mapDefaultDisplayValue<T extends ColumnType>(
   columnsArr: Array<T>
 ): void | T {
   if (!columnsArr.some((column) => column.pv)) {
-    const pkIndex = columnsArr.findIndex((column) => column.pk)
+    const pkIndex = columnsArr.findIndex((column) => column.pk);
 
     // if PK is at the end of table
     if (pkIndex === columnsArr.length - 1) {
@@ -15,11 +15,11 @@ export default function mapDefaultDisplayValue<T extends ColumnType>(
         columnsArr[0].pv = true;
         return columnsArr[0];
       }
-    // pk is not at the end of table
-    } else if (pkIndex > -1) { 
+      // pk is not at the end of table
+    } else if (pkIndex > -1) {
       columnsArr[pkIndex + 1].pv = true;
       return columnsArr[pkIndex + 1];
-    //  no pk at all
+      //  no pk at all
     } else {
       if (columnsArr.length > 0) {
         columnsArr[0].pv = true;
diff --git a/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts b/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
index 0a6e16c21e..1fa222cd17 100644
--- a/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
+++ b/packages/nocodb/src/lib/version-upgrader/ncStickyColumnUpgrader.ts
@@ -40,8 +40,8 @@ export default async function ({ ncMeta }: NcUpgraderCtx) {
 
     // if no display value column is set
     if (!view_columns_meta.some((column) => column.pv)) {
-      const pkIndex = view_columns_meta.findIndex((column) => column.pk)
-  
+      const pkIndex = view_columns_meta.findIndex((column) => column.pk);
+
       // if PK is at the end of table
       if (pkIndex === view_columns_meta.length - 1) {
         if (pkIndex > 0) {
@@ -61,8 +61,8 @@ export default async function ({ ncMeta }: NcUpgraderCtx) {
             view_columns_meta[0].id
           );
         }
-      // pk is not at the end of table
-      } else if (pkIndex > -1) { 
+        // pk is not at the end of table
+      } else if (pkIndex > -1) {
         await ncMeta.metaUpdate(
           null,
           null,
@@ -70,7 +70,7 @@ export default async function ({ ncMeta }: NcUpgraderCtx) {
           { pv: true },
           view_columns_meta[pkIndex + 1].id
         );
-      //  no pk at all
+        //  no pk at all
       } else if (view_columns_meta.length > 0) {
         await ncMeta.metaUpdate(
           null,

From ffcb5841664b464a80dcd81dc52d9e2173b374f3 Mon Sep 17 00:00:00 2001
From: pranavxc <pranavxc@users.noreply.github.com>
Date: Tue, 21 Feb 2023 06:48:35 +0000
Subject: [PATCH 24/25] [create-pull-request] automated change

Signed-off-by: GitHub <noreply@github.com>
---
 packages/nc-gui/package-lock.json | 54 +++++++++++++++++++------------
 packages/nc-gui/package.json      |  2 +-
 packages/nc-lib-gui/package.json  |  2 +-
 packages/nocodb-sdk/package.json  |  2 +-
 packages/nocodb/package-lock.json | 50 +++++++++++++---------------
 packages/nocodb/package.json      |  8 ++---
 6 files changed, 63 insertions(+), 55 deletions(-)

diff --git a/packages/nc-gui/package-lock.json b/packages/nc-gui/package-lock.json
index 760ec39993..e52054f981 100644
--- a/packages/nc-gui/package-lock.json
+++ b/packages/nc-gui/package-lock.json
@@ -29,7 +29,7 @@
         "jwt-decode": "^3.1.2",
         "locale-codes": "^1.3.1",
         "monaco-editor": "^0.33.0",
-        "nocodb-sdk": "file:../nocodb-sdk",
+        "nocodb-sdk": "0.105.3",
         "papaparse": "^5.3.2",
         "qrcode": "^1.5.1",
         "socket.io-client": "^4.5.1",
@@ -97,7 +97,8 @@
       }
     },
     "../nocodb-sdk": {
-      "version": "0.105.2",
+      "version": "0.105.3",
+      "extraneous": true,
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "axios": "^0.21.1",
@@ -8554,7 +8555,6 @@
       "version": "1.15.1",
       "resolved": "https://registry.npmjs.org/follow-redirects/-/follow-redirects-1.15.1.tgz",
       "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA==",
-      "devOptional": true,
       "funding": [
         {
           "type": "individual",
@@ -11970,8 +11970,21 @@
       }
     },
     "node_modules/nocodb-sdk": {
-      "resolved": "../nocodb-sdk",
-      "link": true
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
+      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
+      "dependencies": {
+        "axios": "^0.21.1",
+        "jsep": "^1.3.6"
+      }
+    },
+    "node_modules/nocodb-sdk/node_modules/axios": {
+      "version": "0.21.4",
+      "resolved": "https://registry.npmjs.org/axios/-/axios-0.21.4.tgz",
+      "integrity": "sha512-ut5vewkiu8jjGBdqpM44XxjuCjq9LAKeHVmoVfHVzy8eHgxxq8SbAVQNovDA8mVi05kP0Ea/n/UzcSHcTJQfNg==",
+      "dependencies": {
+        "follow-redirects": "^1.14.0"
+      }
     },
     "node_modules/node-abi": {
       "version": "3.23.0",
@@ -23962,8 +23975,7 @@
     "follow-redirects": {
       "version": "1.15.1",
       "resolved": "https://registry.npmjs.org/follow-redirects/-/follow-redirects-1.15.1.tgz",
-      "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA==",
-      "devOptional": true
+      "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA=="
     },
     "form-data": {
       "version": "4.0.0",
@@ -26436,22 +26448,22 @@
       }
     },
     "nocodb-sdk": {
-      "version": "file:../nocodb-sdk",
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
+      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
       "requires": {
-        "@typescript-eslint/eslint-plugin": "^4.0.1",
-        "@typescript-eslint/parser": "^4.0.1",
         "axios": "^0.21.1",
-        "cspell": "^4.1.0",
-        "eslint": "^7.8.0",
-        "eslint-config-prettier": "^6.11.0",
-        "eslint-plugin-eslint-comments": "^3.2.0",
-        "eslint-plugin-functional": "^3.0.2",
-        "eslint-plugin-import": "^2.22.0",
-        "eslint-plugin-prettier": "^4.0.0",
-        "jsep": "^1.3.6",
-        "npm-run-all": "^4.1.5",
-        "prettier": "^2.1.1",
-        "typescript": "^4.0.2"
+        "jsep": "^1.3.6"
+      },
+      "dependencies": {
+        "axios": {
+          "version": "0.21.4",
+          "resolved": "https://registry.npmjs.org/axios/-/axios-0.21.4.tgz",
+          "integrity": "sha512-ut5vewkiu8jjGBdqpM44XxjuCjq9LAKeHVmoVfHVzy8eHgxxq8SbAVQNovDA8mVi05kP0Ea/n/UzcSHcTJQfNg==",
+          "requires": {
+            "follow-redirects": "^1.14.0"
+          }
+        }
       }
     },
     "node-abi": {
diff --git a/packages/nc-gui/package.json b/packages/nc-gui/package.json
index b37773df3f..eab13cc396 100644
--- a/packages/nc-gui/package.json
+++ b/packages/nc-gui/package.json
@@ -52,7 +52,7 @@
     "jwt-decode": "^3.1.2",
     "locale-codes": "^1.3.1",
     "monaco-editor": "^0.33.0",
-    "nocodb-sdk": "file:../nocodb-sdk",
+    "nocodb-sdk": "0.105.3",
     "papaparse": "^5.3.2",
     "qrcode": "^1.5.1",
     "socket.io-client": "^4.5.1",
diff --git a/packages/nc-lib-gui/package.json b/packages/nc-lib-gui/package.json
index 4b69138b3d..c8291f6ecc 100644
--- a/packages/nc-lib-gui/package.json
+++ b/packages/nc-lib-gui/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nc-lib-gui",
-  "version": "0.105.2",
+  "version": "0.105.3",
   "description": "NocoDB GUI",
   "author": {
     "name": "NocoDB",
diff --git a/packages/nocodb-sdk/package.json b/packages/nocodb-sdk/package.json
index 86e88d8bc1..ec22d9e676 100644
--- a/packages/nocodb-sdk/package.json
+++ b/packages/nocodb-sdk/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nocodb-sdk",
-  "version": "0.105.2",
+  "version": "0.105.3",
   "description": "NocoDB SDK",
   "main": "build/main/index.js",
   "typings": "build/main/index.d.ts",
diff --git a/packages/nocodb/package-lock.json b/packages/nocodb/package-lock.json
index fd244d0f9b..0bbe63aa54 100644
--- a/packages/nocodb/package-lock.json
+++ b/packages/nocodb/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "nocodb",
-  "version": "0.105.2",
+  "version": "0.105.3",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "nocodb",
-      "version": "0.105.2",
+      "version": "0.105.3",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "@google-cloud/storage": "^5.7.2",
@@ -65,10 +65,10 @@
         "mysql2": "^2.2.5",
         "nanoid": "^3.1.20",
         "nc-help": "0.2.85",
-        "nc-lib-gui": "0.105.2",
+        "nc-lib-gui": "0.105.3",
         "nc-plugin": "0.1.2",
         "ncp": "^2.0.0",
-        "nocodb-sdk": "file:../nocodb-sdk",
+        "nocodb-sdk": "0.105.3",
         "nodemailer": "^6.4.10",
         "object-hash": "^3.0.0",
         "os-locale": "^5.0.0",
@@ -154,7 +154,8 @@
       }
     },
     "../nocodb-sdk": {
-      "version": "0.105.2",
+      "version": "0.105.3",
+      "extraneous": true,
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "axios": "^0.21.1",
@@ -11257,9 +11258,9 @@
       }
     },
     "node_modules/nc-lib-gui": {
-      "version": "0.105.2",
-      "resolved": "https://registry.npmjs.org/nc-lib-gui/-/nc-lib-gui-0.105.2.tgz",
-      "integrity": "sha512-uycep2vme3XP6PwvFjJ6wE2fpCNiaJ/ptWR7ZHwRCu500eJGdtonWc00BQf01FitQ1vKq9Cpqqi2b0hoFBLFvQ==",
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nc-lib-gui/-/nc-lib-gui-0.105.3.tgz",
+      "integrity": "sha512-w07Y2+nBiUQYiUyURwH9nqvzzxXsz8MALU/MhmWyNe3Z0YhBvPPLLUxtG1WR3USLI925YT0BsAZTn+iSP3ooPw==",
       "dependencies": {
         "express": "^4.17.1"
       }
@@ -11318,8 +11319,13 @@
       "dev": true
     },
     "node_modules/nocodb-sdk": {
-      "resolved": "../nocodb-sdk",
-      "link": true
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
+      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
+      "dependencies": {
+        "axios": "^0.21.1",
+        "jsep": "^1.3.6"
+      }
     },
     "node_modules/node-abort-controller": {
       "version": "3.0.1",
@@ -27875,9 +27881,9 @@
       }
     },
     "nc-lib-gui": {
-      "version": "0.105.2",
-      "resolved": "https://registry.npmjs.org/nc-lib-gui/-/nc-lib-gui-0.105.2.tgz",
-      "integrity": "sha512-uycep2vme3XP6PwvFjJ6wE2fpCNiaJ/ptWR7ZHwRCu500eJGdtonWc00BQf01FitQ1vKq9Cpqqi2b0hoFBLFvQ==",
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nc-lib-gui/-/nc-lib-gui-0.105.3.tgz",
+      "integrity": "sha512-w07Y2+nBiUQYiUyURwH9nqvzzxXsz8MALU/MhmWyNe3Z0YhBvPPLLUxtG1WR3USLI925YT0BsAZTn+iSP3ooPw==",
       "requires": {
         "express": "^4.17.1"
       }
@@ -27924,22 +27930,12 @@
       "dev": true
     },
     "nocodb-sdk": {
-      "version": "file:../nocodb-sdk",
+      "version": "0.105.3",
+      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
+      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
       "requires": {
-        "@typescript-eslint/eslint-plugin": "^4.0.1",
-        "@typescript-eslint/parser": "^4.0.1",
         "axios": "^0.21.1",
-        "cspell": "^4.1.0",
-        "eslint": "^7.8.0",
-        "eslint-config-prettier": "^6.11.0",
-        "eslint-plugin-eslint-comments": "^3.2.0",
-        "eslint-plugin-functional": "^3.0.2",
-        "eslint-plugin-import": "^2.22.0",
-        "eslint-plugin-prettier": "^4.0.0",
-        "jsep": "^1.3.6",
-        "npm-run-all": "^4.1.5",
-        "prettier": "^2.1.1",
-        "typescript": "^4.0.2"
+        "jsep": "^1.3.6"
       }
     },
     "node-abort-controller": {
diff --git a/packages/nocodb/package.json b/packages/nocodb/package.json
index ebb9842020..3045d52b5a 100644
--- a/packages/nocodb/package.json
+++ b/packages/nocodb/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nocodb",
-  "version": "0.105.2",
+  "version": "0.105.3",
   "description": "NocoDB Backend",
   "main": "dist/bundle.js",
   "author": {
@@ -105,10 +105,10 @@
     "mysql2": "^2.2.5",
     "nanoid": "^3.1.20",
     "nc-help": "0.2.85",
-    "nc-lib-gui": "0.105.2",
+    "nc-lib-gui": "0.105.3",
     "nc-plugin": "0.1.2",
     "ncp": "^2.0.0",
-    "nocodb-sdk": "file:../nocodb-sdk",
+    "nocodb-sdk": "0.105.3",
     "nodemailer": "^6.4.10",
     "object-hash": "^3.0.0",
     "os-locale": "^5.0.0",
@@ -184,4 +184,4 @@
   "prettier": {
     "singleQuote": true
   }
-}
+}
\ No newline at end of file

From 259c7ee529de2c18f9d61d940ba035bf08fa42aa Mon Sep 17 00:00:00 2001
From: Wing-Kam Wong <wingkwong.code@gmail.com>
Date: Tue, 21 Feb 2023 16:01:13 +0800
Subject: [PATCH 25/25] chore: update sdk paths

---
 packages/nc-gui/package-lock.json     | 52 +++++++++++----------------
 packages/nc-gui/package.json          |  2 +-
 packages/nocodb-sdk/package-lock.json |  4 +--
 packages/nocodb/package-lock.json     | 30 +++++++++-------
 packages/nocodb/package.json          |  4 +--
 5 files changed, 42 insertions(+), 50 deletions(-)

diff --git a/packages/nc-gui/package-lock.json b/packages/nc-gui/package-lock.json
index e52054f981..f7ab5bfabf 100644
--- a/packages/nc-gui/package-lock.json
+++ b/packages/nc-gui/package-lock.json
@@ -29,7 +29,7 @@
         "jwt-decode": "^3.1.2",
         "locale-codes": "^1.3.1",
         "monaco-editor": "^0.33.0",
-        "nocodb-sdk": "0.105.3",
+        "nocodb-sdk": "file:../nocodb-sdk",
         "papaparse": "^5.3.2",
         "qrcode": "^1.5.1",
         "socket.io-client": "^4.5.1",
@@ -98,7 +98,6 @@
     },
     "../nocodb-sdk": {
       "version": "0.105.3",
-      "extraneous": true,
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "axios": "^0.21.1",
@@ -8555,6 +8554,7 @@
       "version": "1.15.1",
       "resolved": "https://registry.npmjs.org/follow-redirects/-/follow-redirects-1.15.1.tgz",
       "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA==",
+      "devOptional": true,
       "funding": [
         {
           "type": "individual",
@@ -11970,21 +11970,8 @@
       }
     },
     "node_modules/nocodb-sdk": {
-      "version": "0.105.3",
-      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
-      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
-      "dependencies": {
-        "axios": "^0.21.1",
-        "jsep": "^1.3.6"
-      }
-    },
-    "node_modules/nocodb-sdk/node_modules/axios": {
-      "version": "0.21.4",
-      "resolved": "https://registry.npmjs.org/axios/-/axios-0.21.4.tgz",
-      "integrity": "sha512-ut5vewkiu8jjGBdqpM44XxjuCjq9LAKeHVmoVfHVzy8eHgxxq8SbAVQNovDA8mVi05kP0Ea/n/UzcSHcTJQfNg==",
-      "dependencies": {
-        "follow-redirects": "^1.14.0"
-      }
+      "resolved": "../nocodb-sdk",
+      "link": true
     },
     "node_modules/node-abi": {
       "version": "3.23.0",
@@ -23975,7 +23962,8 @@
     "follow-redirects": {
       "version": "1.15.1",
       "resolved": "https://registry.npmjs.org/follow-redirects/-/follow-redirects-1.15.1.tgz",
-      "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA=="
+      "integrity": "sha512-yLAMQs+k0b2m7cVxpS1VKJVvoz7SS9Td1zss3XRwXj+ZDH00RJgnuLx7E44wx02kQLrdM3aOOy+FpzS7+8OizA==",
+      "devOptional": true
     },
     "form-data": {
       "version": "4.0.0",
@@ -26448,22 +26436,22 @@
       }
     },
     "nocodb-sdk": {
-      "version": "0.105.3",
-      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
-      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
+      "version": "file:../nocodb-sdk",
       "requires": {
+        "@typescript-eslint/eslint-plugin": "^4.0.1",
+        "@typescript-eslint/parser": "^4.0.1",
         "axios": "^0.21.1",
-        "jsep": "^1.3.6"
-      },
-      "dependencies": {
-        "axios": {
-          "version": "0.21.4",
-          "resolved": "https://registry.npmjs.org/axios/-/axios-0.21.4.tgz",
-          "integrity": "sha512-ut5vewkiu8jjGBdqpM44XxjuCjq9LAKeHVmoVfHVzy8eHgxxq8SbAVQNovDA8mVi05kP0Ea/n/UzcSHcTJQfNg==",
-          "requires": {
-            "follow-redirects": "^1.14.0"
-          }
-        }
+        "cspell": "^4.1.0",
+        "eslint": "^7.8.0",
+        "eslint-config-prettier": "^6.11.0",
+        "eslint-plugin-eslint-comments": "^3.2.0",
+        "eslint-plugin-functional": "^3.0.2",
+        "eslint-plugin-import": "^2.22.0",
+        "eslint-plugin-prettier": "^4.0.0",
+        "jsep": "^1.3.6",
+        "npm-run-all": "^4.1.5",
+        "prettier": "^2.1.1",
+        "typescript": "^4.0.2"
       }
     },
     "node-abi": {
diff --git a/packages/nc-gui/package.json b/packages/nc-gui/package.json
index eab13cc396..b37773df3f 100644
--- a/packages/nc-gui/package.json
+++ b/packages/nc-gui/package.json
@@ -52,7 +52,7 @@
     "jwt-decode": "^3.1.2",
     "locale-codes": "^1.3.1",
     "monaco-editor": "^0.33.0",
-    "nocodb-sdk": "0.105.3",
+    "nocodb-sdk": "file:../nocodb-sdk",
     "papaparse": "^5.3.2",
     "qrcode": "^1.5.1",
     "socket.io-client": "^4.5.1",
diff --git a/packages/nocodb-sdk/package-lock.json b/packages/nocodb-sdk/package-lock.json
index e9169032f3..8c1bda86b4 100644
--- a/packages/nocodb-sdk/package-lock.json
+++ b/packages/nocodb-sdk/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "nocodb-sdk",
-  "version": "0.105.2",
+  "version": "0.105.3",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "nocodb-sdk",
-      "version": "0.105.2",
+      "version": "0.105.3",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "axios": "^0.21.1",
diff --git a/packages/nocodb/package-lock.json b/packages/nocodb/package-lock.json
index 0bbe63aa54..c63e48dd18 100644
--- a/packages/nocodb/package-lock.json
+++ b/packages/nocodb/package-lock.json
@@ -68,7 +68,7 @@
         "nc-lib-gui": "0.105.3",
         "nc-plugin": "0.1.2",
         "ncp": "^2.0.0",
-        "nocodb-sdk": "0.105.3",
+        "nocodb-sdk": "file:../nocodb-sdk",
         "nodemailer": "^6.4.10",
         "object-hash": "^3.0.0",
         "os-locale": "^5.0.0",
@@ -155,7 +155,6 @@
     },
     "../nocodb-sdk": {
       "version": "0.105.3",
-      "extraneous": true,
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "axios": "^0.21.1",
@@ -11319,13 +11318,8 @@
       "dev": true
     },
     "node_modules/nocodb-sdk": {
-      "version": "0.105.3",
-      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
-      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
-      "dependencies": {
-        "axios": "^0.21.1",
-        "jsep": "^1.3.6"
-      }
+      "resolved": "../nocodb-sdk",
+      "link": true
     },
     "node_modules/node-abort-controller": {
       "version": "3.0.1",
@@ -27930,12 +27924,22 @@
       "dev": true
     },
     "nocodb-sdk": {
-      "version": "0.105.3",
-      "resolved": "https://registry.npmjs.org/nocodb-sdk/-/nocodb-sdk-0.105.3.tgz",
-      "integrity": "sha512-WLPIVlDIqLtRzhehAJE6t/51kk0tGU7EDOLJ64+/jvlh7gdjizWMyA1lvmKHl7extzn2ISLVxO6YtPJ/ufoY2A==",
+      "version": "file:../nocodb-sdk",
       "requires": {
+        "@typescript-eslint/eslint-plugin": "^4.0.1",
+        "@typescript-eslint/parser": "^4.0.1",
         "axios": "^0.21.1",
-        "jsep": "^1.3.6"
+        "cspell": "^4.1.0",
+        "eslint": "^7.8.0",
+        "eslint-config-prettier": "^6.11.0",
+        "eslint-plugin-eslint-comments": "^3.2.0",
+        "eslint-plugin-functional": "^3.0.2",
+        "eslint-plugin-import": "^2.22.0",
+        "eslint-plugin-prettier": "^4.0.0",
+        "jsep": "^1.3.6",
+        "npm-run-all": "^4.1.5",
+        "prettier": "^2.1.1",
+        "typescript": "^4.0.2"
       }
     },
     "node-abort-controller": {
diff --git a/packages/nocodb/package.json b/packages/nocodb/package.json
index 3045d52b5a..87bcff9e79 100644
--- a/packages/nocodb/package.json
+++ b/packages/nocodb/package.json
@@ -108,7 +108,7 @@
     "nc-lib-gui": "0.105.3",
     "nc-plugin": "0.1.2",
     "ncp": "^2.0.0",
-    "nocodb-sdk": "0.105.3",
+    "nocodb-sdk": "file:../nocodb-sdk",
     "nodemailer": "^6.4.10",
     "object-hash": "^3.0.0",
     "os-locale": "^5.0.0",
@@ -184,4 +184,4 @@
   "prettier": {
     "singleQuote": true
   }
-}
\ No newline at end of file
+}