Ajoute un lien vers le site DGCL dans le cadre d'information
Cette MR :
- Ajoute un renvoi vers le site DGCL de référence en cette période de notification des dotations (notifiée le 31 mars en fin de journée). En gris comme évoqué oralement. cc @Dorine
- Ajoute un lien vers le site Dotations Locales (Incubateur des Territoires de l'ANCT) pour l'historique des montants. En gris également.
- Ajoute une cible
clean
pour retirer les dépendances installées avecnpm run clean
.
Pour quel rendu ?
Quelle couleur de hover sur les liens ?
Le seul lien bleu qui subsiste est celui du formulaire (a priori, pour le mettre en avant puisque l'on veut recueillir des avis). Mais la couleur en hover est à confirmer. Par défaut, elle est du bleu turquoise (cf. Avant cette MR ci-dessous). Cette MR teste l'application d'une couleur plus claire, celle du cadre.
Mais l'association des bleus n'est peut-être pas heureuse ?

Autre option identifiée (pas de screenshot) : appliquer couleur plus foncée similaire à celle du hover sur bouton Estimer.
Merge request reports
Activity
requested review from @Dorine
assigned to @sandcha
Par contre, il est clair qu'on a besoin de faire attention aux versions de librairies que nous voulons.
@benoit-cty j'ai tenté de retirer le
package-lock.json
et de faire unnom install
. Que ce soit en node v12.21 (celle que j'avais par défaut en v12) ou la plus récente v12.22 (ne parlons pas de la LTS en v18 actuellement ), on réduit drastiquement les alertes sur l'obsolescence des dépendances mais obtient une erreur à l'exécution de ce type (où j'avais renommé le composantMatomo
enLexImpactMatomo
pour le test) :Edited by sandcha@sandcha top merci bcp, cette version me convient très bien :
J'aurai retiré le saut de ligne entre les deux premiers paragraphes, quitte à garder le retour à la ligne. :)
added 1 commit
- 1d180563 - Retire une ligne vide entre l'alerte et le lien DGCL
enabled an automatic merge when the pipeline for 1d180563 succeeds
mentioned in commit 5961571f
mentioned in issue openfisca-france-dotations-locales#4 (moved)
mentioned in issue leximpact-server#61 (closed)