@jmruthers/pace-core 0.5.126 → 0.5.128

package/docs/api/interfaces/StorageUrlOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / StorageUrlOptions
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / StorageUrlOptions
 
 # Interface: StorageUrlOptions
 

package/docs/api/interfaces/StyleImport.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / StyleImport
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / StyleImport
 
 # Interface: StyleImport
 

package/docs/api/interfaces/SwitchProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / SwitchProps
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / SwitchProps
 
 # Interface: SwitchProps
 

package/docs/api/interfaces/ToastActionElement.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / ToastActionElement
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / ToastActionElement
 
 # Interface: ToastActionElement
 

package/docs/api/interfaces/ToastProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / ToastProps
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / ToastProps
 
 # Interface: ToastProps
 

package/docs/api/interfaces/UnifiedAuthContextType.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UnifiedAuthContextType
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UnifiedAuthContextType
 
 # Interface: UnifiedAuthContextType
 

package/docs/api/interfaces/UnifiedAuthProviderProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UnifiedAuthProviderProps
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UnifiedAuthProviderProps
 
 # Interface: UnifiedAuthProviderProps
 

package/docs/api/interfaces/UseInactivityTrackerOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseInactivityTrackerOptions
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UseInactivityTrackerOptions
 
 # Interface: UseInactivityTrackerOptions
 

package/docs/api/interfaces/UseInactivityTrackerReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseInactivityTrackerReturn
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UseInactivityTrackerReturn
 
 # Interface: UseInactivityTrackerReturn
 

package/docs/api/interfaces/UsePublicEventOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicEventOptions
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UsePublicEventOptions
 
 # Interface: UsePublicEventOptions
 

package/docs/api/interfaces/UsePublicEventReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicEventReturn
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UsePublicEventReturn
 
 # Interface: UsePublicEventReturn
 

package/docs/api/interfaces/UsePublicFileDisplayOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayOptions
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayOptions
 
 # Interface: UsePublicFileDisplayOptions
 

package/docs/api/interfaces/UsePublicFileDisplayReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayReturn
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayReturn
 
 # Interface: UsePublicFileDisplayReturn
 

package/docs/api/interfaces/UsePublicRouteParamsReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicRouteParamsReturn
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UsePublicRouteParamsReturn
 
 # Interface: UsePublicRouteParamsReturn
 

package/docs/api/interfaces/UseResolvedScopeOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseResolvedScopeOptions
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UseResolvedScopeOptions
 
 # Interface: UseResolvedScopeOptions
 

package/docs/api/interfaces/UseResolvedScopeReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseResolvedScopeReturn
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UseResolvedScopeReturn
 
 # Interface: UseResolvedScopeReturn
 

package/docs/api/interfaces/UserEventAccess.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserEventAccess
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UserEventAccess
 
 # Interface: UserEventAccess
 

package/docs/api/interfaces/UserMenuProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserMenuProps
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UserMenuProps
 
 # Interface: UserMenuProps
 

package/docs/api/interfaces/UserProfile.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserProfile
+[@jmruthers/pace-core - v0.5.128](../README.md) / [Exports](../modules.md) / UserProfile
 
 # Interface: UserProfile
 

package/docs/api/modules.md

@@ -1,6 +1,6 @@
-[@jmruthers/pace-core - v0.5.126](README.md) / Exports
+[@jmruthers/pace-core - v0.5.128](README.md) / Exports
 
-# @jmruthers/pace-core - v0.5.126
+# @jmruthers/pace-core - v0.5.128
 
 **`File`**
 
@@ -1558,7 +1558,7 @@ JSX.Element - The dialog footer container using semantic <footer> element
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:690](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L690)
+[packages/core/src/components/Dialog/Dialog.tsx:677](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L677)
 
 ___
 
@@ -1578,7 +1578,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:707](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L707)
+[packages/core/src/components/Dialog/Dialog.tsx:694](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L694)
 
 ___
 
@@ -1598,7 +1598,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:742](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L742)
+[packages/core/src/components/Dialog/Dialog.tsx:729](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L729)
 
 ___
 
@@ -1668,7 +1668,7 @@ React element with file display
 
 #### Defined in
 
-[packages/core/src/components/FileDisplay/FileDisplay.tsx:727](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/FileDisplay/FileDisplay.tsx#L727)
+[packages/core/src/components/FileDisplay/FileDisplay.tsx:803](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/FileDisplay/FileDisplay.tsx#L803)
 
 ___
 
@@ -1767,6 +1767,11 @@ and customizable branding support.
 A flexible header component that supports various configurations including custom logos,
 navigation menus, user authentication, event selection, and custom actions.
 
+**Logo Display:** When used via PaceAppLayout, the logo URL is automatically constructed
+from the appName prop as `/${appName.toLowerCase()}_logo_wide.svg`. The appName should
+come from an APP_NAME constant declared in your App.tsx file to ensure consistency across
+authenticated and public pages.
+
 Features:
 - Customizable logo (URL or custom component)
 - Clickable logo that automatically routes to dashboard (configurable via logoHref)
@@ -1872,7 +1877,7 @@ function MinimalHeader() {
 
 #### Defined in
 
-[packages/core/src/components/Header/Header.tsx:229](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Header/Header.tsx#L229)
+[packages/core/src/components/Header/Header.tsx:234](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Header/Header.tsx#L234)
 
 ___
 
@@ -2304,6 +2309,17 @@ This component is designed to work with React Router's nested routing pattern us
 Outlet to render child routes. It provides integrated authentication, navigation,
 and user management functionality.
 
+**Super Admin Access:** When `enforcePermissions={true}`, PaceAppLayout automatically
+checks if the user is a super admin before enforcing permissions. Super admins bypass
+all permission checks and can access any route without violations. The component extracts
+base page names from route paths (e.g., `/organisation/scouts-victoria` → `"organisation"`)
+for permission checking, which can be overridden using `pageIdMapping`.
+
+**Important:** The appName prop should use an APP_NAME constant declared in your App.tsx
+file. This ensures consistency with public pages (via PublicPageProvider) which should
+also receive the same APP_NAME constant. The logo URL is automatically constructed as
+`/${appName.toLowerCase()}_logo_wide.svg` from the public folder.
+
 Features:
 - React Router v6 integration with nested routing
 - Unified authentication integration
@@ -2315,6 +2331,7 @@ Features:
 - Layout-level permission enforcement
 - Permission-based navigation filtering
 - Automatic page permission validation
+- Super admin bypass (super admins automatically bypass all permission checks)
 
 #### Parameters
 
@@ -2334,17 +2351,20 @@ Basic React Router setup with permission enforcement (RECOMMENDED):
 ```tsx
 import { BrowserRouter as Router, Routes, Route } from 'react-router-dom';
 import { UnifiedAuthProvider } from '@jmruthers/pace-core/providers';
-import { PaceAppLayout, PaceLoginPage } from '@jmruthers/pace-core';
+import { PaceAppLayout, PaceLoginPage, PublicPageApp } from '@jmruthers/pace-core';
+
+const APP_NAME = 'CORE';
 
 function App() {
   return (
-    <UnifiedAuthProvider supabaseClient={supabase} appName="My App">
+    <UnifiedAuthProvider supabaseClient={supabase} appName={APP_NAME}>
       <Router>
         <Routes>
-          <Route path="/login" element={<PaceLoginPage appName="My App" />} />
+          <Route path="/login" element={<PaceLoginPage appName={APP_NAME} />} />
+          <Route path="/events/*" element={<PublicPageApp appName={APP_NAME} />} />
           <Route path="/" element={
             <PaceAppLayout 
-              appName="My Application"
+              appName={APP_NAME}
               enforcePermissions={true}
               defaultPermission="read"
             />
@@ -2442,7 +2462,7 @@ function AdminApp() {
 
 #### Defined in
 
-[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:328](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L328)
+[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:344](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L344)
 
 ___
 
@@ -2823,14 +2843,18 @@ Header component for public pages with event-specific branding
 This component displays the app logo, event logo, and event information
 in a clean, accessible layout suitable for public pages.
 
-Logo handling follows a priority order:
-1. customAppLogo prop (if provided) - highest priority
-2. logoUrl prop (if provided) - direct URL override
-3. Auto-generated path from appName via useAppConfig() - convention-based
-4. Default SVG fallback - lowest priority
+The app logo is automatically generated from the appName using the pattern:
+`/{appName.toLowerCase()}_logo_wide.svg` from the public folder.
+
+**Important:** The appName is obtained from PublicPageProvider context, which should
+receive the APP_NAME constant from your App.tsx file. This ensures consistency with
+authenticated pages that use the same APP_NAME constant.
 
-The logo can be made clickable by providing the logoHref prop, which will
-wrap the logo in a Link component for navigation.
+**Event Logo Requirements:**
+- Event logo files must be marked as `is_public = true` in the `file_references` table
+- The RPC function `data_file_reference_by_category_list` must be accessible to the anonymous/public role
+- Storage bucket must allow public read access for logo files
+- If no public logo is available, a fallback UI with event initials will be displayed
 
 #### Parameters
 
@@ -2846,7 +2870,7 @@ React element with public page header
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageHeader.tsx:124](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageHeader.tsx#L124)
+[packages/core/src/components/PublicLayout/PublicPageHeader.tsx:103](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageHeader.tsx#L103)
 
 ___
 
@@ -2922,6 +2946,7 @@ This provider:
 - Provides environment variables for public data access
 - Includes error boundary for graceful error handling
 - Is completely separate from the main app context
+- Provides appName for consistent logo display
 
 #### Parameters
 
@@ -2935,7 +2960,7 @@ This provider:
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:64](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L64)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:70](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L70)
 
 ___
 
@@ -2953,7 +2978,7 @@ Public page context with environment variables
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:119](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L119)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:128](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L128)
 
 ___
 
@@ -2971,7 +2996,7 @@ True if we're in a public page context
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:134](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L134)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:143](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L143)
 
 ___
 
@@ -3858,7 +3883,7 @@ Useful for testing or when you need to force refresh all data
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicFileDisplay.ts:373](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L373)
+[packages/core/src/hooks/public/usePublicFileDisplay.ts:542](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L542)
 
 ___
 
@@ -3879,7 +3904,7 @@ Get cache statistics for debugging
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicFileDisplay.ts:384](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L384)
+[packages/core/src/hooks/public/usePublicFileDisplay.ts:553](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L553)
 
 ___
 
@@ -3935,7 +3960,7 @@ Useful when you only need the event code and will fetch data separately
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:224](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L224)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:212](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L212)
 
 ___
 
@@ -3958,7 +3983,7 @@ Utility function to generate public route paths
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:260](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L260)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:248](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L248)
 
 ___
 
@@ -3981,7 +4006,7 @@ Supports 2-50 character event codes
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:275](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L275)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:263](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L263)
 
 ___
 

package/docs/api-reference/components.md

@@ -123,6 +123,12 @@ interface DataTableProps<TData extends DataRecord> {
   onDeleteRow?: (row: TData) => void;
   onCreateRow?: (data: Partial<TData>) => void;
   onImport?: (data: TData[]) => void | Promise<void>;
+  /** 
+   * Custom export handler - allows full control over export columns and filename
+   * Receives ExportOptions with tableRows, allColumns, visibleColumns, and more
+   * If not provided, defaults to exporting all visible columns
+   */
+  onExport?: (options: ExportOptions<TData>) => void | Promise<void>;
   onDeleteSelected?: (selectedRows: Record<string, boolean>) => void;
   
   // Performance
@@ -276,6 +282,24 @@ const columns: DataTableColumn<User>[] = [
   onEditRow={handleEdit}
   onDeleteRow={handleDelete}
   onCreateRow={handleCreate}
+  onExport={async (options) => {
+    // Custom export with specific columns
+    const exportColumns = options.allColumns
+      .filter(col => ['name', 'email'].includes(col.accessorKey || ''))
+      .map(col => ({
+        header: col.header || col.accessorKey,
+        id: col.id || col.accessorKey,
+        accessorKey: col.accessorKey,
+        accessorFn: 'accessorFn' in col ? col.accessorFn : undefined,
+      }));
+    
+    await exportToCSVWithTableRows(
+      options.tableRows,
+      exportColumns,
+      options.columnIdToTableColumn,
+      'users-export.csv'
+    );
+  }}
 />
 ```
 

package/docs/api-reference/types.md

@@ -334,6 +334,33 @@ interface ButtonProps {
 }
 ```
 
+### ExportOptions
+
+Options provided to the `onExport` handler for custom export functionality.
+
+```typescript
+interface ExportOptions<TData extends DataRecord> {
+  /** Filtered table rows with getValue() method for proper accessorFn evaluation */
+  tableRows: Array<{
+    original: TData;
+    getValue: (columnId: string) => any;
+    id: string;
+  }>;
+  /** All column definitions passed to the DataTable */
+  allColumns: DataTableColumn<TData>[];
+  /** Currently visible columns in the table */
+  visibleColumns: DataTableColumn<TData>[];
+  /** Mapping of column IDs to TanStack table column instances (for getValue() calls) */
+  columnIdToTableColumn: Map<string, any>;
+  /** Raw data array (unfiltered) */
+  data: TData[];
+  /** Default filename generated from table title */
+  filename: string;
+  /** TanStack table instance for advanced operations */
+  table: any;
+}
+```
+
 ### DataTableProps
 
 Props for the DataTable component.
@@ -948,6 +975,7 @@ export type {
   // UI Components
   ButtonProps,
   DataTableProps,
+  ExportOptions,
   ColumnDef,
   PaginationConfig,
   SortingConfig,

package/docs/architecture/rpc-function-standards.md

@@ -75,8 +75,9 @@ get_cake_meals                    -- Wrong: wrong prefix order
 
 ```sql
 CREATE OR REPLACE FUNCTION public.<family>_<domain>_<verb>(
-    p_param1 TYPE,
-    p_param2 TYPE DEFAULT NULL,
+    p_param1 TEXT,  -- ⚠️ Use TEXT for strings, NOT CHARACTER VARYING
+    p_param2 INTEGER DEFAULT NULL,  -- Use INTEGER, DOUBLE PRECISION, SMALLINT for numbers
+    p_param3 BOOLEAN DEFAULT NULL,  -- Use BOOLEAN for boolean values
     p_user_id UUID DEFAULT auth.uid(),
     p_organisation_id UUID DEFAULT NULL
 )
@@ -150,11 +151,23 @@ $$;
 
 2. **Always set `SET search_path TO 'public'`** - Prevents SQL injection attacks
 
-3. **Parameter Naming**:
+3. **Parameter Naming and Types**:
    - Prefix all parameters with `p_` (e.g., `p_event_id`, `p_user_id`)
    - Use `p_user_id UUID DEFAULT auth.uid()` for user context
    - Use `p_organisation_id UUID DEFAULT NULL` for organisation context
    - Use descriptive names
+   - **⚠️ CRITICAL: Parameter Type Requirements**:
+     - **String parameters MUST use `TEXT`**, NOT `CHARACTER VARYING` or `VARCHAR`
+     - **Numeric parameters**: Use `DOUBLE PRECISION` for decimal numbers, `INTEGER` for whole numbers, `SMALLINT` for small integers
+     - **Boolean parameters**: Use `BOOLEAN`
+     - **Date/Time parameters**: Use `DATE`, `TIMESTAMP`, or `TIMESTAMPTZ` as appropriate
+     - **UUID parameters**: Use `UUID`
+     - **Why TEXT instead of CHARACTER VARYING?**
+       - Application code calls functions with TEXT parameters
+       - PostgreSQL cannot automatically choose between TEXT and CHARACTER VARYING versions
+       - Having both creates ambiguity errors: "Could not choose the best candidate function"
+       - TEXT is the standard PostgreSQL type for string parameters in RPC functions
+       - Even though database columns may use CHARACTER VARYING, function parameters should use TEXT
 
 4. **Variable Naming**:
    - Prefix local variables with `v_` (e.g., `v_is_super_admin`, `v_event_exists`)
@@ -710,7 +723,22 @@ rbac_check_permission_simplified(
 
 3. **Missing page_id parameter**: Always provide the page name as the last parameter (e.g., `'dishes'`, `'meals'`, `'distribution'`)
 
-4. **Ambiguous column references**: 
+4. **Using CHARACTER VARYING instead of TEXT for string parameters**:
+   - ❌ **WRONG**: `p_unit_id CHARACTER VARYING` or `p_dish_name VARCHAR`
+   - ✅ **CORRECT**: `p_unit_id TEXT` or `p_dish_name TEXT`
+   - **CRITICAL**: All string parameters MUST use `TEXT`, NOT `CHARACTER VARYING` or `VARCHAR`
+   - Having both TEXT and CHARACTER VARYING versions causes "Could not choose the best candidate function" errors
+   - Application code calls functions with TEXT parameters, so functions must accept TEXT
+   - Even though database columns may use CHARACTER VARYING, function parameters should use TEXT
+
+5. **Creating duplicate function versions**:
+   - ❌ **WRONG**: Creating both `app_cake_unit_update(TEXT, ...)` and `app_cake_unit_update(CHARACTER VARYING, ...)`
+   - ✅ **CORRECT**: Only create `app_cake_unit_update(TEXT, ...)`
+   - Before creating a new function, check if a version already exists
+   - If updating a function, use `CREATE OR REPLACE FUNCTION` to update the existing version
+   - If you need to change parameter types, drop the old version first: `DROP FUNCTION IF EXISTS ... CASCADE;`
+
+6. **Ambiguous column references**: 
    - ❌ **WRONG**: `SELECT mealtype_name FROM cake_mealtype`
    - ✅ **CORRECT**: `SELECT cake_mealtype.mealtype_name FROM cake_mealtype`
    - Always fully qualify table names when there might be ambiguity
@@ -718,7 +746,7 @@ rbac_check_permission_simplified(
    - Common ambiguous columns: `organisation_id`, `created_by`, `updated_by`, `created_at`, `updated_at`
    - See "Avoiding Ambiguous Column References" section below for detailed guidance
 
-5. **Not checking existing migrations**: Before creating a new migration, check if a similar one already exists to avoid duplicates
+7. **Not checking existing migrations**: Before creating a new migration, check if a similar one already exists to avoid duplicates
 
 ### Updating Existing Functions
 
@@ -1046,6 +1074,12 @@ When creating or reviewing an RPC function, ensure:
 - ✅ Includes `GRANT EXECUTE` statement
 - ✅ Tested with various scenarios
 
+### ⚠️ Critical Parameter Type Requirements
+- ✅ **String parameters use TEXT**: All string parameters MUST use `TEXT`, NOT `CHARACTER VARYING` or `VARCHAR`
+- ✅ **No duplicate function versions**: Only one version of each function exists (TEXT version, not CHARACTER VARYING)
+- ✅ **Numeric parameters use correct types**: `DOUBLE PRECISION` for decimals, `INTEGER` for whole numbers, `SMALLINT` for small integers
+- ✅ **Why TEXT?**: Application code calls with TEXT parameters; having both TEXT and CHARACTER VARYING versions causes ambiguity errors
+
 ### ⚠️ Critical Permission Format
 - ✅ **Permission format is correct**: `'operation:page.{pageName}'` (e.g., `'create:page.dishes'`, `'read:page.meals'`)
 - ✅ **NOT using wrong format**: `'operation:resource'` (e.g., `'create:dishes'`, `'read:meals'`)

package/docs/implementation-guides/data-tables.md

@@ -1746,35 +1746,80 @@ The import automatically recognizes and maps:
 />
 ```
 
-### Custom Export Filename
+### Custom Export
 
 The export filename is automatically generated from the table `title` prop:
 
 - If `title` is provided: `{title}_{date}.csv` (e.g., `meals_2024-04-11.csv`)
 - If no `title`: `data_export_{date}.csv`
 
-To customize the filename, you can provide a custom `onExport` handler:
+By default, the DataTable exports all **visible columns**. To customize the export (different columns, custom filename, etc.), you can provide a custom `onExport` handler:
 
 ```tsx
+import { DataTable, type ExportOptions, exportToCSVWithTableRows, type ExportColumn } from '@jmruthers/pace-core';
+
 <DataTable
   title="Meals"
   columns={columns}
   features={{ export: true }}
-  onExport={async () => {
-    // Custom export logic
-    const data = getFilteredData();
-    await exportToCSV(data, columns, 'custom-export-name.csv');
+  onExport={async (options: ExportOptions<Meal>) => {
+    // Export only specific columns (different from visible columns)
+    const exportColumns: ExportColumn[] = options.allColumns
+      .filter(col => ['name', 'email', 'role'].includes(col.accessorKey || ''))
+      .map(col => ({
+        header: col.header || col.accessorKey,
+        id: col.id || col.accessorKey,
+        accessorKey: col.accessorKey,
+        accessorFn: 'accessorFn' in col ? col.accessorFn : undefined,
+      }));
+    
+    // Use the provided table rows and column mapping for proper accessorFn evaluation
+    await exportToCSVWithTableRows(
+      options.tableRows,
+      exportColumns,
+      options.columnIdToTableColumn,
+      'custom-export-name.csv'
+    );
   }}
 />
 ```
 
+### Export Options
+
+The `onExport` handler receives an `ExportOptions` object with:
+
+- `tableRows` - Filtered table rows with `getValue()` method for proper `accessorFn` evaluation
+- `allColumns` - All column definitions passed to the DataTable
+- `visibleColumns` - Currently visible columns in the table
+- `columnIdToTableColumn` - Mapping of column IDs to TanStack table column instances (for `getValue()` calls)
+- `data` - Raw data array (unfiltered)
+- `filename` - Default filename generated from table title
+- `table` - TanStack table instance for advanced operations
+
 ### Export Column Selection
 
-**Important**: Only **visible columns** are exported. If a column is hidden (via column visibility controls), it will not be included in the export.
+**Default Behavior**: Only **visible columns** are exported. If a column is hidden (via column visibility controls), it will not be included in the export.
+
+**Custom Export**: Use the `onExport` handler to export different columns than what's visible in the table:
 
-To ensure all columns are exported:
-- Make sure all desired columns are visible before exporting
-- Or use a custom `onExport` handler to export specific columns
+```tsx
+onExport={async (options) => {
+  // Export all columns (including hidden ones)
+  const allExportColumns: ExportColumn[] = options.allColumns.map(col => ({
+    header: col.header || col.accessorKey,
+    id: col.id || col.accessorKey,
+    accessorKey: col.accessorKey,
+    accessorFn: 'accessorFn' in col ? col.accessorFn : undefined,
+  }));
+  
+  await exportToCSVWithTableRows(
+    options.tableRows,
+    allExportColumns,
+    options.columnIdToTableColumn,
+    'all-columns-export.csv'
+  );
+}}
+```
 
 ### Import Column Mapping
 

package/docs/implementation-guides/permission-enforcement.md

@@ -53,6 +53,8 @@ setupRBAC(supabase);
 
 Layout-level enforcement provides a safety net that prevents consuming apps from forgetting to implement permission checks. It ensures that every route is automatically validated for permissions.
 
+**Note:** Super admins automatically bypass all permission checks in PaceAppLayout. The component checks super admin status before enforcing permissions, ensuring super admins can access any route without violations.
+
 ### Basic Implementation
 
 ```tsx
@@ -107,6 +109,8 @@ Configure different permissions for different routes:
 />
 ```
 
+**Page Name Mapping:** PaceAppLayout automatically extracts the base page name from route paths. For nested routes like `/organisation/scouts-victoria`, it uses `"organisation"` (the first path segment) as the page name. You can override this using `pageIdMapping` to provide custom page IDs for specific routes.
+
 ### Navigation Filtering
 
 Filter navigation items based on user permissions: