Some checks failed
Auto Tag Develop / tag (push) Has been cancelled
Suite à l'arrivée des workflows, correction des régressions UI et améliorations UX mail/modales (reviews Lucile Schnödt, Tristan Schnödtin). **Specs & décisions :** `docs/superpowers/specs/2026-05-20-workflow-ui-fixes-design.md` **Plan d'implémentation :** `docs/superpowers/plans/2026-05-21-workflow-ui-fixes.md` Cette PR contient désormais **les specs ET l'implémentation complète**. ## Chantiers livrés | # | Chantier | Détail | |---|----------|--------| | 2 | Sélecteur de statut filtré par workflow | `statusOptions` dérivé de `project.workflow.statuses`, statut courant conservé s'il est hors workflow | | 1 | Drag & drop « Mes tâches » | handlers `@dragover/@drop` ; résolution par workflow/catégorie (0→refus, 1→PATCH, ≥2→popover `StatusPickerPopover`) | | 4 | Couleurs | (a) migration Doctrine remettant les hex classiques sur le workflow Standard ; (b) entêtes kanban teintées via `STATUS_CATEGORY_COLOR` + contraste auto ; (c) couleur par défaut par catégorie dans `WorkflowDrawer` | | 5 | Suppression du bouton « Lier un mail » | + retrait de `MailPickerModal` et i18n associée | | 6 | Création de tâche depuis un mail | back : `assigneeId` + `statusId` (défaut = 1er statut du workflow), priorité retirée (TDD) ; front : `MailCreateTaskModal` sur `AppModal` + sélecteurs user/statut | | 7 | Modale réutilisable | nouveau `components/ui/AppModal.vue` (footer sticky) ; footer de `TaskModal` sorti du form scrollable | | 3 | Cartes responsive | badges en `flex-wrap` pleine taille (plus aucun débordement) | | 8 | (dette) Sélecteur de catégorie en `MalioSelect` | la lib supporte les valeurs `string` ; note CLAUDE.md corrigée | ## Vérifications - Build frontend OK ; PHPUnit **34 tests verts** (nouveau test fonctionnel TDD sur `create-task`). - Vérif navigateur (Chrome MCP) sur **données prod importées en local** : #2, #3, #4, #5, #6, #7 confirmés. - Revue de code finale : **APPROVED_WITH_NITS**. ## À noter - ⚠️ **#1 (D&D)** : le drag & drop HTML5 natif n'est pas auto-testable → **test manuel requis**. - 🗄️ **#4 (migration)** : `migrations/Version20260521094948.php` s'appliquera en **prod au prochain `make migration-migrate`**. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Matthieu <mtholot19@gmail.com> Reviewed-on: #6
145 lines
5.6 KiB
Vue
145 lines
5.6 KiB
Vue
<script setup lang="ts">
|
|
import type { MailMessageDetailDto } from '~/services/dto/mail'
|
|
import type { Task } from '~/services/dto/task'
|
|
import type { Project } from '~/services/dto/project'
|
|
import type { TaskGroup } from '~/services/dto/task-group'
|
|
import type { UserData } from '~/services/dto/user-data'
|
|
import { useMailService } from '~/services/mail'
|
|
import { useProjectService } from '~/services/projects'
|
|
import { useTaskGroupService } from '~/services/task-groups'
|
|
import { useUserService } from '~/services/users'
|
|
import { useAuthStore } from '~/stores/auth'
|
|
|
|
const props = defineProps<{
|
|
modelValue: boolean
|
|
messageId: number
|
|
messageDetail: MailMessageDetailDto | null
|
|
}>()
|
|
const emit = defineEmits<{
|
|
'update:modelValue': [value: boolean]
|
|
created: [task: Task]
|
|
}>()
|
|
|
|
const { t } = useI18n()
|
|
const auth = useAuthStore()
|
|
const mailService = useMailService()
|
|
const projectService = useProjectService()
|
|
const taskGroupService = useTaskGroupService()
|
|
const userService = useUserService()
|
|
|
|
const projectId = ref<number | null>(null)
|
|
const taskGroupId = ref<number | null>(null)
|
|
const assigneeId = ref<number | null>(null)
|
|
const statusId = ref<number | null>(null)
|
|
const isSubmitting = ref(false)
|
|
const touchedProject = ref(false)
|
|
|
|
const projects = ref<Project[]>([])
|
|
const groups = ref<TaskGroup[]>([])
|
|
const users = ref<UserData[]>([])
|
|
const loadingGroups = ref(false)
|
|
|
|
const projectOptions = computed(() => projects.value.map(p => ({ label: p.name, value: p.id })))
|
|
const groupOptions = computed(() => groups.value.filter(g => !g.archived).map(g => ({ label: g.title, value: g.id })))
|
|
const userOptions = computed(() => users.value.map(u => ({ label: u.username, value: u.id })))
|
|
|
|
const selectedProject = computed(() => projects.value.find(p => p.id === projectId.value) ?? null)
|
|
const statusOptions = computed(() =>
|
|
(selectedProject.value?.workflow?.statuses ?? []).map(s => ({ label: s.label, value: s.id })),
|
|
)
|
|
|
|
onMounted(async () => {
|
|
const [projs, us] = await Promise.all([
|
|
projectService.getAll({ archived: false }),
|
|
userService.getAll(),
|
|
])
|
|
projects.value = projs
|
|
users.value = us
|
|
})
|
|
|
|
watch(projectId, async (pid) => {
|
|
taskGroupId.value = null
|
|
statusId.value = selectedProject.value?.workflow?.statuses?.[0]?.id ?? null
|
|
groups.value = []
|
|
if (!pid) return
|
|
loadingGroups.value = true
|
|
try {
|
|
groups.value = await taskGroupService.getByProject(pid)
|
|
} finally {
|
|
loadingGroups.value = false
|
|
}
|
|
})
|
|
|
|
watch(() => props.modelValue, (open) => {
|
|
if (open) {
|
|
projectId.value = null
|
|
taskGroupId.value = null
|
|
statusId.value = null
|
|
assigneeId.value = auth.user?.id ?? null
|
|
touchedProject.value = false
|
|
}
|
|
})
|
|
|
|
function close(): void {
|
|
emit('update:modelValue', false)
|
|
}
|
|
|
|
async function handleSubmit(): Promise<void> {
|
|
touchedProject.value = true
|
|
if (!projectId.value) return
|
|
isSubmitting.value = true
|
|
try {
|
|
const task = await mailService.createTaskFromMail(props.messageId, {
|
|
projectId: projectId.value,
|
|
taskGroupId: taskGroupId.value ?? undefined,
|
|
assigneeId: assigneeId.value ?? undefined,
|
|
statusId: statusId.value ?? undefined,
|
|
})
|
|
emit('created', task)
|
|
close()
|
|
} finally {
|
|
isSubmitting.value = false
|
|
}
|
|
}
|
|
</script>
|
|
|
|
<template>
|
|
<AppModal
|
|
:model-value="modelValue"
|
|
width="lg"
|
|
:title="t('mail.createTaskModal.title')"
|
|
@update:model-value="emit('update:modelValue', $event)"
|
|
>
|
|
<div class="space-y-5">
|
|
<div v-if="messageDetail" class="rounded-lg border border-neutral-200 bg-neutral-50 px-4 py-3 text-sm">
|
|
<p class="truncate font-medium text-neutral-800">{{ messageDetail.header.subject ?? t('mail.noSubject') }}</p>
|
|
<p class="mt-0.5 truncate text-xs text-neutral-500">{{ messageDetail.header.fromName ?? messageDetail.header.fromEmail }}</p>
|
|
<p class="mt-2 text-xs italic text-neutral-400">{{ t('mail.createTaskModal.titleHint') }}</p>
|
|
<p class="text-xs italic text-neutral-400">{{ t('mail.createTaskModal.descriptionHint') }}</p>
|
|
</div>
|
|
|
|
<div>
|
|
<MalioSelect v-model="projectId" :options="projectOptions" :label="t('mail.createTaskModal.projectLabel')" :empty-option-label="t('mail.createTaskModal.projectPlaceholder')" min-width="w-full" />
|
|
<p v-if="touchedProject && !projectId" class="mt-1 text-xs text-red-500">{{ t('mail.createTaskModal.projectLabel').replace(' *', '') }} requis</p>
|
|
</div>
|
|
|
|
<div v-if="projectId">
|
|
<MalioSelect v-model="taskGroupId" :options="groupOptions" :label="t('mail.createTaskModal.groupLabel')" :empty-option-label="t('mail.createTaskModal.groupPlaceholder')" min-width="w-full" :disabled="loadingGroups" />
|
|
</div>
|
|
|
|
<div v-if="projectId">
|
|
<MalioSelect v-model="statusId" :options="statusOptions" :label="t('mail.createTaskModal.statusLabel')" min-width="w-full" />
|
|
</div>
|
|
|
|
<div>
|
|
<MalioSelect v-model="assigneeId" :options="userOptions" :label="t('mail.createTaskModal.assigneeLabel')" :empty-option-label="t('mail.createTaskModal.assigneePlaceholder')" min-width="w-full" />
|
|
</div>
|
|
</div>
|
|
|
|
<template #footer>
|
|
<MalioButton variant="tertiary" label="Annuler" button-class="w-auto px-4" @click="close" />
|
|
<MalioButton :label="t('mail.createTaskModal.submit')" button-class="w-auto px-6" :disabled="isSubmitting" @click="handleSubmit" />
|
|
</template>
|
|
</AppModal>
|
|
</template>
|