♻️ Réduire les utilisations de ConsulterProjet #2764
Conversation
benjlevesque
force-pushed
the
benjlevesque/pot-540-repasser-sur-les-usages-de-consulterprojet-et-retirer-ceux
branch
from
65f5592 to
ba0687c
Compare
.../app/laureats/[identifiant]/garanties-financieres/actuelles:enregistrer-attestation/page.tsx
Show resolved
Hide resolved
...ions/ssr/src/app/laureats/[identifiant]/garanties-financieres/actuelles:enregistrer/page.tsx
Show resolved
Hide resolved
| @@ -25,20 +25,13 @@ export const metadata: Metadata = { | |||
| export default async function Page({ params: { identifiant } }: IdentifiantParameter) { | |||
| return PageWithErrorHandling(async () => { | |||
| const identifiantProjet = decodeParameter(identifiant); | |||
|
|
|||
| const candidature = await mediator.send<Candidature.ConsulterProjetQuery>({ | |||
There was a problem hiding this comment.
On fait ça pour vérifier que la candidature existe j'imagine, est ce qu'il faut garder ?
There was a problem hiding this comment.
pour les 2 précédents je suis d'accord, pour celui-là il y a déjà un notFound dans la page, je pense que ca suffit ?
| }); | ||
|
|
||
| if (Option.isNone(candidature)) { |
There was a problem hiding this comment.
On fait ça pour vérifier que la candidature existe j'imagine, est ce qu'il faut garder ?
There was a problem hiding this comment.
il y a déjà un notFound, ca devrait suffire ?
...plications/ssr/src/app/laureats/[identifiant]/garanties-financieres/depot:soumettre/page.tsx
Show resolved
Hide resolved
packages/applications/ssr/src/app/laureats/[identifiant]/garanties-financieres/page.tsx
Show resolved
Hide resolved
| statut: projet.statut, | ||
| message: "Vous ne pouvez pas consulter le raccordement d'un projet éliminé ou abandonné", | ||
| }); | ||
| await récupérerLauréatNonAbandonné(identifiantProjet.formatter()); |
There was a problem hiding this comment.
Pour tes cas d'usage, le message d'erreur sera moins clair. Après pas très grave IMO
There was a problem hiding this comment.
effectivement. Mais pour moi il faut sortir du paradigme projet; un éliminé et un lauréat sont 2 choses différentes dans Potentiel. C'est normal d'avoir "non trouvé" si on cherche un lauréat avec un ID d'éliminé je trouve
Après à nous de faire en sorte qu'il n'y ait pas moyen d'atterrir sur une page spécifique à un lauréat avec un ID d'éliminé :)
benjlevesque
force-pushed
the
benjlevesque/pot-540-repasser-sur-les-usages-de-consulterprojet-et-retirer-ceux
branch
from
b956bd4 to
929740b
Compare
Non traité :
récupérerProjetAdapterdans les notifications et projections