✨ Formulaire de modification d'une candidature #2300
Conversation
benjlevesque
commented
Oct 9, 2024
•
benjlevesque
commented
- retirer le menu "Corriger des candidats". On souhaite plutot utiliser le formulaire. Il est dispo dans la page "importer des candidats"
- fix sur la projection en cas de Correction
- nouvelle page détail Candidature
- nouvelle page corriger Candidature
benjlevesque
force-pushed
the
feat/ui-corriger-candidature
branch
3 times, most recently
from
462f4d0
to
c4b989b
Compare
benjlevesque
force-pushed
the
feat/ui-corriger-candidature
branch
from
c4b989b
to
14c39a8
Compare
...ges/applications/ssr/src/components/pages/candidature/importer/ImporterCandidatures.page.tsx
Outdated
Show resolved
Hide resolved
...ges/applications/ssr/src/components/pages/candidature/importer/ImporterCandidatures.page.tsx
Outdated
Show resolved
Hide resolved
...ges/applications/ssr/src/components/pages/candidature/importer/ImporterCandidatures.page.tsx
Show resolved
Hide resolved
| @@ -103,9 +103,6 @@ const MenuAdmin = (currentPage?: string) => ( | |||
| > | |||
| Notifier des candidats | |||
| </DropdownMenu.DropdownItem> | |||
| <DropdownMenu.DropdownItem href={Routes.Candidature.corriger}> | |||
There was a problem hiding this comment.
Je l'aurai gardée en mettant Corriger plusieurs candidatures (c'est très bizarre de corriger des candidats, et pas les candidatures, j'ai un problème avec ce wording)
There was a problem hiding this comment.
je pense vraiment qu'on doit retirer ce menu, et on en avait parlé avec Mathieu il me semble.
Ce n'est pas du tout quelque chose de standard de venir corriger par csv; c'est juste pour le cas ou tout le csv (ou la majorité des cas) a un problème.
Du coup je crois qu'il faut rendre ce menu moins visible, et disponible sur la page import et la page correction individuelle
packages/applications/ssr/src/components/pages/candidature/détails/DétailsCandidature.page.tsx
Outdated
Show resolved
Hide resolved
...ages/applications/ssr/src/components/pages/candidature/lister/CandidatureListItemActions.tsx
Outdated
Show resolved
Hide resolved
| }) | ||
| // le motif d'élimination est obligatoire si la candidature est éliminée | ||
| .superRefine(requiredFieldIfReferenceFieldEquals('motifElimination', 'statut', 'éliminé')); | ||
| // le type de GF est obligatoire si la candidature est classée |
There was a problem hiding this comment.
Du coup tu ne le mets pas ?
There was a problem hiding this comment.
on peut le rajouter effectivement, uniquement avant notification du coup.
...ages/applications/ssr/src/components/pages/candidature/corriger/CorrigerCandidature.form.tsx
Outdated
Show resolved
Hide resolved
| historiqueAbandonValue: previous.historiqueAbandon.formatter(), | ||
| technologieValue: previous.technologie.formatter(), | ||
|
|
||
| // TODO |
There was a problem hiding this comment.
ah j'attendais que tu aies mergé tes PR pour gérer les détails, je m'en occupe
| identifiantProjet: IdentifiantProjet.ValueType; | ||
| }; | ||
|
|
||
| const mapToActionComponents = ({ identifiantProjet, actions }: MapToActionsComponentsProps) => ( |
There was a problem hiding this comment.
Return null si pas d'action, en tant que porteur je vois juste le titre sans rien en dessous
There was a problem hiding this comment.
tu ne devrais pas avoir accès à cette page ETQ porteur, tu y es arrivé par URL ?
There was a problem hiding this comment.
ca a pas d'importance qu'il puisse y accéder, et du coup je pense que c'est OK qu'il n'y ait pas d'action
benjlevesque
force-pushed
the
feat/ui-corriger-candidature
branch
from
71cf446
to
5c67074
Compare
