Fix variables field type to support all JSON-compatible values

- Replace restrictive Record<string, any> with flexible JSONValue type for variables field
- Add JSONValue type alias that matches Payload's JSON field type specification
- Support string, number, boolean, objects, arrays, null, and undefined for variables
- Update both BaseEmailDocument and QueuedEmail interfaces consistently
- Update documentation to reflect JSONValue support

Fixes type constraint error where customer Email.variables field type
(string | number | boolean | {...} | unknown[] | null | undefined)
was not assignable to Record<string, any>.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
2025-09-13 23:38:46 +02:00
parent ecc0b0a73e
commit 4dcbc1446a
2 changed files with 9 additions and 3 deletions

View File

@@ -2,6 +2,9 @@ import { Payload } from 'payload'
import type { CollectionConfig, RichTextField } from 'payload'
import { Transporter } from 'nodemailer'
// JSON value type that matches Payload's JSON field type
export type JSONValue = string | number | boolean | { [k: string]: unknown } | unknown[] | null | undefined
// Generic base interfaces that work with any ID type and null values
export interface BaseEmailDocument {
id: string | number
@@ -14,7 +17,7 @@ export interface BaseEmailDocument {
subject: string
html: string
text?: string | null
variables?: Record<string, any> | null
variables?: JSONValue
scheduledAt?: string | null
sentAt?: string | null
status?: 'pending' | 'processing' | 'sent' | 'failed' | null
@@ -84,7 +87,7 @@ export interface QueuedEmail {
subject: string
html: string
text?: string | null
variables?: Record<string, any> | null
variables?: JSONValue
scheduledAt?: string | null
sentAt?: string | null
status: 'pending' | 'processing' | 'sent' | 'failed'