Revert "revert(zapier): make View a text field again — view list isn't OAuth-accessible"

This reverts commit 3b8c0d8.

Author: caoxing9

packages/zapier/src/authentication.ts

@@ -15,6 +15,7 @@ const SCOPES = [
   'base|read',
   'table|read',
   'field|read',
+  'view|read',
   'record|read',
   'record|create',
   'record|update',

packages/zapier/src/index.ts

@@ -7,6 +7,7 @@ import { rawInstance } from './lib/client';
 import bases from './triggers/bases';
 import tables from './triggers/tables';
 import fields from './triggers/fields';
+import views from './triggers/views';
 import newRecord from './triggers/new_record';
 import newOrUpdatedRecord from './triggers/new_or_updated_record';
 
@@ -73,6 +74,7 @@ const App = {
     bases,
     tables,
     fields,
+    views,
     new_record: newRecord,
     new_or_updated_record: newOrUpdatedRecord,
   },

packages/zapier/src/lib/types.ts

@@ -30,6 +30,12 @@ export interface TeableTable {
   name: string;
 }
 
+// A view as returned by GET /table/{tableId}/view.
+export interface TeableView {
+  id: string;
+  name: string;
+}
+
 // An { id, name } row used to populate Zapier dynamic dropdowns.
 export interface DropdownItem {
   id: string;

packages/zapier/src/triggers/new_or_updated_record.ts

@@ -46,14 +46,17 @@ export default {
         required: true,
         dynamic: 'tables.id.name',
         search: 'tables.id',
+        // Pass the chosen table down so the View dropdown (which depends on
+        // tableId) re-fetches with it in scope.
+        altersDynamicFields: true,
       },
       {
         key: 'viewId',
         label: 'View',
         type: 'string',
         required: false,
-        helpText:
-          'Optional. The `viw…` id of a view to limit records to (from the view URL). Leave blank for all records.',
+        dynamic: 'views.id.name',
+        helpText: 'Limit to records in a specific view. Leave blank for all records.',
       },
     ],
     perform,

packages/zapier/src/triggers/new_record.ts

@@ -36,14 +36,17 @@ export default {
         required: true,
         dynamic: 'tables.id.name',
         search: 'tables.id',
+        // Pass the chosen table down so the View dropdown (which depends on
+        // tableId) re-fetches with it in scope.
+        altersDynamicFields: true,
       },
       {
         key: 'viewId',
         label: 'View',
         type: 'string',
         required: false,
-        helpText:
-          'Optional. The `viw…` id of a view to limit records to (from the view URL). Leave blank for all records.',
+        dynamic: 'views.id.name',
+        helpText: 'Limit to records in a specific view. Leave blank for all records.',
       },
     ],
     perform,

packages/zapier/src/triggers/views.ts

@@ -0,0 +1,34 @@
+import type { ZObject, Bundle } from 'zapier-platform-core';
+import { apiUrl } from '../lib/client';
+import type { TeableView, DropdownItem } from '../lib/types';
+
+// Hidden trigger powering the "View" dropdown. Depends on a chosen tableId.
+const perform = async (z: ZObject, bundle: Bundle): Promise<DropdownItem[]> => {
+  const { tableId } = bundle.inputData;
+  z.console.log(`views dropdown: tableId=${tableId ?? '(none)'}`);
+  if (!tableId) return [];
+  const response = await z.request<TeableView[]>({
+    url: apiUrl(bundle, `/table/${tableId}/view`),
+  });
+  const items = (response.data || []).map((view) => ({
+    id: view.id,
+    name: view.name,
+  }));
+  z.console.log(`views dropdown: ${items.length} view(s) for table ${tableId}`);
+  return items;
+};
+
+export default {
+  key: 'views',
+  noun: 'View',
+  display: {
+    label: 'List Views',
+    description: 'Internal trigger used to populate the View dropdown.',
+    hidden: true,
+  },
+  operation: {
+    inputFields: [{ key: 'tableId', type: 'string', required: true }],
+    perform,
+    sample: { id: 'viwXXXXXXXXXXXX', name: 'Grid view' },
+  },
+};