feat:[LAR-60] Add send mail for decline article and fix unsend mail f… #156
Conversation
|
Corrige les soucis de phpstan |
There was a problem hiding this comment.
Il vient d'ou ce fichier? Et ou est-il utilisé dans la PR?
There was a problem hiding this comment.
Dans la modal tu rejet c'est l'input du motif de rejet
There was a problem hiding this comment.
Pourquoi avoir créer ce composant ?
There was a problem hiding this comment.
Pourquoi la création de ce composant ?
There was a problem hiding this comment.
Vu que sur la plupart des projets nous partons sur la base des composants, je me suis permis d'en créer un pour chaque élément label, textaerea et input dans le dossier forms
There was a problem hiding this comment.
Tu es entrain de bosser dans un site qui existe, qui a des formulaires qui a des inputs, tu ne cherches pas a fouiller une page (login / register) pour voir quels inputs sont utilisé. Tu pars prendre des inputs qui n'ont pas le même style le même design que ce qui est déjà utilisé
|
Ou est le test de cette fonctionnalité? |
cybersoldattech
deleted the
feature/lar-60-notification-apres-validation-ou-rejet-de-larticle
branch
…or approve article