fix(mail) : aligne le contrat front/back pour le listing et le détail des messages
Le service appelait GET /mail/messages?folder=X (404) au lieu de la vraie route
/mail/folders/{path}/messages. Ajoute aussi une couche de mapping backend→DTO
(messages→items, fromAddress→fromEmail, toAddresses→toRecipients, détail plat→imbriqué)
pour réconcilier la dérive de contrat entre Phase 3 (API) et Phase 4 (DTOs front).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -14,6 +14,47 @@ import type {
|
|||||||
} from './dto/mail'
|
} from './dto/mail'
|
||||||
import type { Task } from './dto/task'
|
import type { Task } from './dto/task'
|
||||||
|
|
||||||
|
type BackendMailMessage = {
|
||||||
|
id: number
|
||||||
|
messageId: string
|
||||||
|
uid: number
|
||||||
|
folderPath?: string
|
||||||
|
subject: string | null
|
||||||
|
fromAddress: string | null
|
||||||
|
fromName: string | null
|
||||||
|
toAddresses: string[] | null
|
||||||
|
ccAddresses: string[] | null
|
||||||
|
sentAt: string | null
|
||||||
|
isRead: boolean
|
||||||
|
isFlagged: boolean
|
||||||
|
hasAttachments: boolean
|
||||||
|
snippet?: string | null
|
||||||
|
linkedTaskIds?: number[]
|
||||||
|
}
|
||||||
|
|
||||||
|
function toAddressList(values: string[] | null | undefined): { email: string; name: string | null }[] {
|
||||||
|
return (values ?? []).map((email) => ({ email, name: null }))
|
||||||
|
}
|
||||||
|
|
||||||
|
function mapHeader(m: BackendMailMessage, fallbackFolderPath = ''): MailMessageHeaderDto {
|
||||||
|
return {
|
||||||
|
id: m.id,
|
||||||
|
messageId: m.messageId,
|
||||||
|
folderPath: m.folderPath ?? fallbackFolderPath,
|
||||||
|
subject: m.subject,
|
||||||
|
fromName: m.fromName,
|
||||||
|
fromEmail: m.fromAddress,
|
||||||
|
toRecipients: toAddressList(m.toAddresses),
|
||||||
|
ccRecipients: toAddressList(m.ccAddresses),
|
||||||
|
sentAt: m.sentAt,
|
||||||
|
receivedAt: m.sentAt ?? '',
|
||||||
|
isRead: m.isRead,
|
||||||
|
isFlagged: m.isFlagged,
|
||||||
|
hasAttachments: m.hasAttachments,
|
||||||
|
linkedTaskIds: m.linkedTaskIds ?? [],
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
export function useMailService() {
|
export function useMailService() {
|
||||||
const api = useApi()
|
const api = useApi()
|
||||||
|
|
||||||
@@ -74,10 +115,19 @@ export function useMailService() {
|
|||||||
cursor?: string,
|
cursor?: string,
|
||||||
limit?: number,
|
limit?: number,
|
||||||
): Promise<MailMessagesPageDto> {
|
): Promise<MailMessagesPageDto> {
|
||||||
const query: Record<string, unknown> = { folder: folderPath }
|
const query: Record<string, unknown> = {}
|
||||||
if (cursor) query.cursor = cursor
|
if (cursor) query.cursor = cursor
|
||||||
if (limit) query.limit = limit
|
if (limit) query.limit = limit
|
||||||
return api.get<MailMessagesPageDto>('/mail/messages', query)
|
const path = `/mail/folders/${encodeURIComponent(folderPath)}/messages`
|
||||||
|
const response = await api.get<{ messages: BackendMailMessage[]; nextCursor: string | null }>(
|
||||||
|
path,
|
||||||
|
query,
|
||||||
|
)
|
||||||
|
return {
|
||||||
|
items: response.messages.map((m) => mapHeader(m, folderPath)),
|
||||||
|
nextCursor: response.nextCursor,
|
||||||
|
total: response.messages.length,
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -85,7 +135,19 @@ export function useMailService() {
|
|||||||
* @param id - ID BDD du message (MailMessage.id)
|
* @param id - ID BDD du message (MailMessage.id)
|
||||||
*/
|
*/
|
||||||
async function getMessage(id: number): Promise<MailMessageDetailDto> {
|
async function getMessage(id: number): Promise<MailMessageDetailDto> {
|
||||||
return api.get<MailMessageDetailDto>(`/mail/messages/${id}`)
|
const response = await api.get<
|
||||||
|
BackendMailMessage & {
|
||||||
|
bodyHtml: string | null
|
||||||
|
bodyText: string | null
|
||||||
|
attachments: MailMessageDetailDto['attachments']
|
||||||
|
}
|
||||||
|
>(`/mail/messages/${id}`)
|
||||||
|
return {
|
||||||
|
header: mapHeader(response),
|
||||||
|
bodyHtml: response.bodyHtml,
|
||||||
|
bodyText: response.bodyText,
|
||||||
|
attachments: response.attachments,
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// ─── Actions sur les messages ─────────────────────────────────────────────
|
// ─── Actions sur les messages ─────────────────────────────────────────────
|
||||||
|
|||||||
Reference in New Issue
Block a user