WIP - display drawer shadow when open

Add dynamic tenant-level related lists and page layout selection

backend/src/ai-assistant/ai-assistant.service.ts

@@ -694,6 +694,8 @@ export class AiAssistantService {
       type: field.type,
     }));
 
+    console.log('fields:',fields);
+
     const formatInstructions = parser.getFormatInstructions();
     const today = new Date().toISOString();
 

backend/src/object/object.service.ts

@@ -1284,23 +1284,10 @@ export class ObjectService {
     if (missingIds.length > 0) {
       throw new NotFoundException(`Records not found: ${missingIds.join(', ')}`);
     }
-
-    const deletableIds: string[] = [];
-    const deniedIds: string[] = [];
-
+    
+    // Check if user can delete each record
     for (const record of records) {
-      const canDelete = await this.authService.canPerformAction(
-        'delete',
-        objectDefModel,
-        record,
-        user,
-        knex,
-      );
-      if (canDelete) {
-        deletableIds.push(record.id);
-      } else {
-        deniedIds.push(record.id);
-      }
+      await this.authService.assertCanPerformAction('delete', objectDefModel, record, user, knex);
     }
 
     // Ensure model is registered
@@ -1308,23 +1295,14 @@ export class ObjectService {
 
     // Use Objection model
     const boundModel = await this.modelService.getBoundModel(resolvedTenantId, objectApiName);
-    if (deletableIds.length > 0) {
-      await boundModel.query().whereIn('id', deletableIds).delete();
-    }
+    await boundModel.query().whereIn('id', recordIds).delete();
 
     // Remove from search index
     await Promise.all(
-      deletableIds.map((id) =>
-        this.removeIndexedRecord(resolvedTenantId, objectApiName, id),
-      ),
+      recordIds.map((id) => this.removeIndexedRecord(resolvedTenantId, objectApiName, id)),
     );
     
-    return {
-      success: true,
-      deleted: deletableIds.length,
-      deletedIds: deletableIds,
-      deniedIds,
-    };
+    return { success: true, deleted: recordIds.length };
   }
 
   private async indexRecord(

frontend/components/ui/checkbox/Checkbox.vue

@@ -18,7 +18,7 @@ const forwarded = useForwardPropsEmits(delegatedProps, emits)
   <CheckboxRoot
     v-bind="forwarded"
     :class="
-      cn('grid place-content-center peer h-4 w-4 shrink-0 rounded-sm border border-primary shadow focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring disabled:cursor-not-allowed disabled:opacity-50',
+      cn('grid place-content-center peer h-4 w-4 shrink-0 rounded-sm border border-primary shadow focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring disabled:cursor-not-allowed disabled:opacity-50 data-[state=checked]:bg-primary data-[state=checked]:text-primary-foreground',
          props.class)"
   >
     <CheckboxIndicator class="grid place-content-center text-current">

frontend/components/views/ListView.vue

@@ -12,7 +12,6 @@ import { Button } from '@/components/ui/button'
 import { Input } from '@/components/ui/input'
 import { Badge } from '@/components/ui/badge'
 import { Checkbox } from '@/components/ui/checkbox'
-import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from '@/components/ui/select'
 import FieldRenderer from '@/components/fields/FieldRenderer.vue'
 import { ListViewConfig, ViewMode, FieldType } from '@/types/field-types'
 import { ChevronDown, ChevronUp, Search, Plus, Download, Trash2, Edit } from 'lucide-vue-next'
@@ -50,13 +49,11 @@ const emit = defineEmits<{
 }>()
 
 // State
-const normalizeId = (id: any) => String(id)
-const selectedRowIds = ref<string[]>([])
+const selectedRows = ref<Set<string>>(new Set())
 const searchQuery = ref('')
 const sortField = ref<string>('')
 const sortDirection = ref<'asc' | 'desc'>('asc')
 const currentPage = ref(1)
-const bulkAction = ref('delete')
 
 // Computed
 const visibleFields = computed(() => 
@@ -97,39 +94,27 @@ const showLoadMore = computed(() => (
 ))
 
 const allSelected = computed({
-  get: () => props.data.length > 0 && selectedRowIds.value.length === props.data.length,
+  get: () => props.data.length > 0 && selectedRows.value.size === props.data.length,
   set: (val: boolean) => {
     if (val) {
-      selectedRowIds.value = props.data.map(row => normalizeId(row.id))
+      selectedRows.value = new Set(props.data.map(row => row.id))
     } else {
-      selectedRowIds.value = []
+      selectedRows.value.clear()
     }
     emit('row-select', getSelectedRows())
   },
 })
 
 const getSelectedRows = () => {
-  const idSet = new Set(selectedRowIds.value)
-  return props.data.filter(row => idSet.has(normalizeId(row.id)))
+  return props.data.filter(row => selectedRows.value.has(row.id))
 }
 
 const toggleRowSelection = (rowId: string) => {
-  const normalizedId = normalizeId(rowId)
-  const nextSelection = new Set(selectedRowIds.value)
-  nextSelection.has(normalizedId) ? nextSelection.delete(normalizedId) : nextSelection.add(normalizedId)
-  selectedRowIds.value = Array.from(nextSelection)
-  emit('row-select', getSelectedRows())
-}
-
-const setRowSelection = (rowId: string, checked: boolean) => {
-  const normalizedId = normalizeId(rowId)
-  const nextSelection = new Set(selectedRowIds.value)
-  if (checked) {
-    nextSelection.add(normalizedId)
+  if (selectedRows.value.has(rowId)) {
+    selectedRows.value.delete(rowId)
   } else {
-    nextSelection.delete(normalizedId)
+    selectedRows.value.add(rowId)
   }
-  selectedRowIds.value = Array.from(nextSelection)
   emit('row-select', getSelectedRows())
 }
 
@@ -151,14 +136,6 @@ const handleAction = (actionId: string) => {
   emit('action', actionId, getSelectedRows())
 }
 
-const handleBulkAction = () => {
-  if (bulkAction.value === 'delete') {
-    emit('delete', getSelectedRows())
-    return
-  }
-  emit('action', bulkAction.value, getSelectedRows())
-}
-
 const goToPage = (page: number) => {
   const nextPage = Math.min(Math.max(page, 1), availablePages.value)
   if (nextPage !== currentPage.value) {
@@ -180,19 +157,6 @@ watch(
     }
   }
 )
-
-watch(
-  () => props.data,
-  (rows) => {
-    const rowIds = new Set(rows.map(row => normalizeId(row.id)))
-    const nextSelection = selectedRowIds.value.filter(id => rowIds.has(id))
-    if (nextSelection.length !== selectedRowIds.value.length) {
-      selectedRowIds.value = nextSelection
-      emit('row-select', getSelectedRows())
-    }
-  },
-  { deep: true }
-)
 </script>
 
 <template>
@@ -217,24 +181,14 @@ watch(
 
       <div class="flex items-center gap-2">
         <!-- Bulk Actions -->
-        <template v-if="selectedRowIds.length > 0">
+        <template v-if="selectedRows.size > 0">
           <Badge variant="secondary" class="px-3 py-1">
-            {{ selectedRowIds.length }} selected
+            {{ selectedRows.size }} selected
           </Badge>
-          <div class="flex items-center gap-2">
-            <Select v-model="bulkAction" @update:model-value="(value) => bulkAction = value">
-              <SelectTrigger class="h-8 w-[180px]">
-                <SelectValue placeholder="Select action" />
-              </SelectTrigger>
-              <SelectContent>
-                <SelectItem value="delete">Delete selected</SelectItem>
-              </SelectContent>
-            </Select>
-            <Button variant="outline" size="sm" @click="handleBulkAction">
-              <Trash2 class="h-4 w-4 mr-2" />
-              Run
-            </Button>
-          </div>
+          <Button variant="outline" size="sm" @click="emit('delete', getSelectedRows())">
+            <Trash2 class="h-4 w-4 mr-2" />
+            Delete
+          </Button>
         </template>
 
         <!-- Custom Actions -->
@@ -268,10 +222,7 @@ watch(
         <TableHeader>
           <TableRow>
             <TableHead v-if="selectable" class="w-12">
-              <Checkbox
-                :model-value="allSelected"
-                @update:model-value="(value: boolean) => (allSelected = value)"
-              />
+              <Checkbox v-model:checked="allSelected" />
             </TableHead>
             <TableHead
               v-for="field in visibleFields"
@@ -312,8 +263,8 @@ watch(
           >
             <TableCell v-if="selectable" @click.stop>
               <Checkbox
-                :model-value="selectedRowIds.includes(normalizeId(row.id))"
-                @update:model-value="(checked: boolean) => setRowSelection(normalizeId(row.id), checked)"
+                :checked="selectedRows.has(row.id)"
+                @update:checked="toggleRowSelection(row.id)"
               />
             </TableCell>
             <TableCell v-for="field in visibleFields" :key="field.id">

frontend/composables/useFieldViews.ts

@@ -330,25 +330,9 @@ export const useViewState = <T extends { id?: string }>(
     loading.value = true
     error.value = null
     try {
-      const useBulkEndpoint = apiEndpoint.includes('/runtime/objects/')
-      if (useBulkEndpoint) {
-        const response = await api.post(`${apiEndpoint}/bulk-delete`, { ids })
-        const deletedIds = Array.isArray(response?.deletedIds) ? response.deletedIds : ids
-        records.value = records.value.filter(r => !deletedIds.includes(r.id!))
-        totalCount.value = Math.max(0, totalCount.value - deletedIds.length)
-        return {
-          deletedIds,
-          deniedIds: Array.isArray(response?.deniedIds) ? response.deniedIds : [],
-        }
-      }
-
       await Promise.all(ids.map(id => api.delete(`${apiEndpoint}/${id}`)))
       records.value = records.value.filter(r => !ids.includes(r.id!))
       totalCount.value = Math.max(0, totalCount.value - ids.length)
-      return {
-        deletedIds: ids,
-        deniedIds: [],
-      }
     } catch (e: any) {
       error.value = e.message
       console.error('Failed to delete records:', e)

frontend/pages/[objectName]/[[recordId]]/[[view]].vue

@@ -6,14 +6,6 @@ import { useFields, useViewState } from '@/composables/useFieldViews'
 import ListView from '@/components/views/ListView.vue'
 import DetailView from '@/components/views/DetailViewEnhanced.vue'
 import EditView from '@/components/views/EditViewEnhanced.vue'
-import {
-  Dialog,
-  DialogContent,
-  DialogDescription,
-  DialogFooter,
-  DialogHeader,
-  DialogTitle,
-} from '@/components/ui/dialog'
 
 const route = useRoute()
 const router = useRouter()
@@ -156,14 +148,8 @@ const maxFrontendRecords = computed(() => listConfig.value?.maxFrontendRecords ?
 const searchQuery = ref('')
 const searchSummary = ref('')
 const searchLoading = ref(false)
-const deleteDialogOpen = ref(false)
-const deleteSubmitting = ref(false)
-const pendingDeleteRows = ref<any[]>([])
-const deleteSummary = ref<{ deletedIds: string[]; deniedIds: string[] } | null>(null)
 
 const isSearchActive = computed(() => searchQuery.value.trim().length > 0)
-const pendingDeleteCount = computed(() => pendingDeleteRows.value.length)
-const deniedDeleteCount = computed(() => deleteSummary.value?.deniedIds.length ?? 0)
 
 // Fetch object definition
 const fetchObjectDefinition = async () => {
@@ -208,42 +194,16 @@ const handleCreateRelated = (relatedObjectApiName: string, _parentId: string) =>
 }
 
 const handleDelete = async (rows: any[]) => {
-  pendingDeleteRows.value = rows
-  deleteSummary.value = null
-  deleteDialogOpen.value = true
-}
-
-const resetDeleteDialog = () => {
-  deleteDialogOpen.value = false
-  deleteSubmitting.value = false
-  pendingDeleteRows.value = []
-  deleteSummary.value = null
-}
-
-const confirmDelete = async () => {
-  if (pendingDeleteRows.value.length === 0) {
-    resetDeleteDialog()
-    return
-  }
-
-  deleteSubmitting.value = true
-  try {
-    const ids = pendingDeleteRows.value.map(r => r.id)
-    const result = await deleteRecords(ids)
-    const deletedIds = result?.deletedIds ?? []
-    const deniedIds = result?.deniedIds ?? []
-    deleteSummary.value = { deletedIds, deniedIds }
-
-    if (deniedIds.length === 0) {
-      resetDeleteDialog()
+  if (confirm(`Delete ${rows.length} record(s)? This action cannot be undone.`)) {
+    try {
+      const ids = rows.map(r => r.id)
+      await deleteRecords(ids)
       if (view.value !== 'list') {
         await router.push(`/${objectApiName.value.toLowerCase()}/`)
       }
+    } catch (e: any) {
+      error.value = e.message || 'Failed to delete records'
     }
-  } catch (e: any) {
-    error.value = e.message || 'Failed to delete records'
-  } finally {
-    deleteSubmitting.value = false
   }
 }
 
@@ -462,46 +422,6 @@ onMounted(async () => {
         @back="handleBack"
       />
     </div>
-
-    <Dialog v-model:open="deleteDialogOpen">
-      <DialogContent class="sm:max-w-[520px]">
-        <DialogHeader>
-          <DialogTitle>Delete records</DialogTitle>
-          <DialogDescription>
-            This action cannot be undone.
-          </DialogDescription>
-        </DialogHeader>
-
-        <div class="space-y-2 text-sm">
-          <p>
-            You are about to delete {{ pendingDeleteCount }} record<span v-if="pendingDeleteCount !== 1">s</span>.
-          </p>
-          <p v-if="deleteSummary" class="text-muted-foreground">
-            Deleted {{ deleteSummary.deletedIds.length }} record<span v-if="deleteSummary.deletedIds.length !== 1">s</span>.
-          </p>
-          <p v-if="deniedDeleteCount > 0" class="text-destructive">
-            {{ deniedDeleteCount }} record<span v-if="deniedDeleteCount !== 1">s</span> could not be deleted due to missing permissions.
-          </p>
-          <p v-if="!deleteSummary" class="text-muted-foreground">
-            Records you do not have permission to delete will be skipped.
-          </p>
-        </div>
-
-        <DialogFooter>
-          <Button variant="outline" @click="resetDeleteDialog" :disabled="deleteSubmitting">
-            {{ deleteSummary ? 'Close' : 'Cancel' }}
-          </Button>
-          <Button
-            v-if="!deleteSummary"
-            variant="destructive"
-            @click="confirmDelete"
-            :disabled="deleteSubmitting"
-          >
-            Delete
-          </Button>
-        </DialogFooter>
-      </DialogContent>
-    </Dialog>
   </NuxtLayout>
 </template>