-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Problème avec les champs requis #70
Comments
Champs requis 👍 Acronyme (nom court) ; Code Unité (avec étoile) ; Nom (avec étoile) ; Numéro national de structure ; Lien RNSR ; Fiche du centre dans ScanR ; Wikidata ; Lien Wikipedia ; Directeur·rice (avec étoile) ; Courriel Direction ; Année de création (avec étoile) ; Historique ; Effectif total (avec étoile) ; Personnels permanents (avec étoile) ; Personnels non permanents (avec étoile) ; Lien vers la page Il semble également que certains champs bel et bien obligatoires, et qui ont une étoile sur le BO en prod, ont perdu l'étoile sur la maj du BO en pprd. Exemple avec le champ "Bibliothèques utilisées" |
Pas évident à corriger car je ne reproduis pas ces comportements en local, pour le moment j'ai donc pushé deux choses :
|
Ping @jri-sp si tu peux déployer en preprod afin que @MG95SP puisse re-tester dans l'après-midi car pour l'instant on en est toujours à essayer de deviner :/ Parmi les choses "étranges" : 100% des champs semblent required lors de la création d'un nouveau centre, chose que je n'ai pas du tout en local. |
Hello! |
…g in fields-list, trying to fix some weird unidentified side effet (refs #70)
Je pense avoir trouvé un build qui fonctionne : a priori (à valider après le prochain déploiement pour être sûr que j'ai la bonne solution tout de même) c'était uniquement un problème de dépendance. Une des dépendances du projet a du évoluer dans les 30 derniers jours et comme le Dockerfile ne se base pas sur le "package-lock" il n'a pas récupéré les "bonnes" versions. |
Ping @jri-sp pour un déploiement en préprod demain si possible, ça devrait être le bon :) |
Hello, |
La cause "dépendance" est confirmé, en dév local après un "npm update" je reproduis bien le même comportement. Je vais maintenant procéder dépendance par dépendance pour isoler la fautive. Merci @jri-sp :) |
La dépendance fautive a été isolée, c'est "angular" en toute simplicité :') |
Afin de rendre tout ça le plus robuste possible, j'ai figé les dépendances après les avoir mises à jour au maximum (tout en respectant les contraintes posées par l'existant). Ça devrait assurer que ce genre d'incident ne se reproduise pas à long terme. |
Je pense que le code coupable est identifié : angular/angular.js#16814 |
Résolu en pprd ! |
Ok en prod |
Avec la nouvelle mise à jour du pprd, de nombreux champs pourtant non-obligatoires sont déormais requis dans le formulaire en BO.
Par exemple, un centre TEST pour lequel j'avais rempli l'ensemble des champs requis précédemment me demande désormais 24 champs de plus : "Ce formulaire contient 24 erreur(s) de validation"
L’ensemble des champs ajoutés suite à l'issue 67 (Numero National de Structure ; Lien RNSR ; Fiche du centre dans ScanR ; Wikidata ; Lien Wikipedia ) sont en requis alors que seul le numéro national de structure est censé l'être.
Il y a également des champs sur l'ensemble des autres onglets qui sont passés en champs requis.
Les champs requis sont pourtant normalement très peu nombreux : cf colonne H (required) dans le CartoScPo - Metadata schema.
The text was updated successfully, but these errors were encountered: