1489 permettre dajouter les schéma et plans pluriannuel

[emma11y]

closes #1489

Avant de merger la pull request, s’assurer que :

• Les checks GitHub passent (lint...).
• Les tests Cypress ont été lancés en local (dans le cas d’une correction de bug ou d’une nouvelle fonctionnalité).

[hissalht]

juste 2-3 détails mais sinon lgtm !

[bellangerq]

Je ne peux pas modifier ma déclaration si je supprime un des deux champs (schéma ou plan d'action).

{
  "message": [
    "schemaPluriannuelUrl must be an URL address",
    "planActionUrl must be an URL address"
  ],
  "error":"Bad Request","statusCode":400
}

Je crois que comme la valeur peut être une string vide (""), ça ne passe pas dans le DTO avec le décorateur IsUrl(). Je ne sais pas s'il est nécessaire comme on valide déjà côté front le format et il n'y a pas de risque de ne pas le faire côté back (c'est pas un login ou quoi).

Pour le contactFormUrl par exemple, on fait :

@IsString()
@IsOptional()
contactFormUrl?: string;

[bellangerq]

Du coup je sais pas si c'est lié à la PR mais quitte à supprimer le contactName, autant le supprimer aussi côté back ?

À moins qu'on ait prévu de s'en servir plus tard.

[emma11y]

Je ne peux pas modifier ma déclaration si je supprime un des deux champs (schéma ou plan d'action).

{
  "message": [
    "schemaPluriannuelUrl must be an URL address",
    "planActionUrl must be an URL address"
  ],
  "error":"Bad Request","statusCode":400
}

Je crois que comme la valeur peut être une string vide (""), ça ne passe pas dans le DTO avec le décorateur IsUrl(). Je ne sais pas s'il est nécessaire comme on valide déjà côté front le format et il n'y a pas de risque de ne pas le faire côté back (c'est pas un login ou quoi).

Pour le contactFormUrl par exemple, on fait :

@IsString()
@IsOptional()
contactFormUrl?: string;

Tu es en contradiction avec le commentaire d'Adrien : #1508 (comment)

[bellangerq]

Ajouter le changelog !