Code Monkey home page Code Monkey logo

mon-service-securise's Issues

Problème de `query params` pour la liste d'identifiants de services lors de l'export CSV

Ce problème n'est pas simple à régler.
Il n'existe pas de version standard pour passer un tableau dans des query params.

La méthode qui semblait fonctionner avec notre stack serait :

Client
let params = new URLSearchParams();
params.append('p', 'val1');
params.append('p', 'val2');
Serveur
const { idsServices = [] } = requete.query;
if(!Array.isArray(idsServices)) throw new Error();

Le problème de cette approche est que express interprète un tableau de longueur 1 comme étant une string.

Lien vers le problème:

Mettre à jour la page « À propos »

La page « À propos » indique que la solution est construite par /ut7. Depuis l'attribution du nouveau marché, il serait intéressant de mentionner également Omnicité.

Éviter le recours à la description des données dans un JSON depuis les vues PUG

La généralisation des script(id = '…', type = 'application/json') !{JSON.stringify(homologation.XXX.YYY.toJSON()) }devient de plus en plus problématique. D'une part, la duplication n'est pas heureuse, d'autre part on génére de plus en plus de comportements liés à des variables globales.

À la place, on souhaite ne plus avoir besoin de transmettre ainsi les données du PUG vers le module JavaScript correspondant :

  • en passant plutôt par des data attributes
  • en gérant les index qui serviront d'identifiants depuis le PUG, ou depuis jQuery
  • etc.

Généraliser description des propriétés atomiques dans une méthode de classe de `InformationsHomologation`

Actuellement, dans la classe métier PointAcces, on a une méthode statique qui décrit les propriétés atomiques liées à cette classe. Il serait bien de…

  • appeler cette méthode statique pour instancier l'objet
  • généraliser ce principe aux autres classes filles de InformationsHomologation
  • en profiter pour peut-être revoir le filtrage des données de listes dans le routeur mss.js (ex., entitesExternes, etc.)
  • en profiter pour supprimer la duplication avec middleware.aseptise('liste.*.propriete')

Gérer les listes d'items à partir de l'index `0`

Actuellement, les pages client qui font intervenir la création / suppression d'items dans une liste (ex. points accès, entités externes, mesures spécifiques, etc.) transmettent un tableau dont le premier élément est null. Cela est dû au fait que la première zone de saisie créée quand on clique sur « ajouter un nouvel item » se base sur un index qui vaut 1 – alors qu'il devrait valoir 0.

Bien sûr, on pourrait décréter que la valeur initiale de l'index, avant première création (et donc avant première incrémentation) vaudrait -1 – mais le fait qu'il faille changer ce changement à plusieurs endroits suggère peut-être une duplication, et l'idée que la gestion de cet index et son incrément devrait avoir lieu à un même endroit, de manière encapsulée.

Ajout d’un `mtm_campaign` pour l’API de recherche d’entreprises

Nous souhaiterions tracker plus finement les utilisations de notre API Recherche d’Entreprises. Pourriez-vous ajouter mtm_campaign=mon-service-securise à la fin de l’url ?

'https://recherche-entreprises.api.gouv.fr/search',

Les stats apparaitront ici dans la partie Campagne :
https://stats.data.gouv.fr/index.php?module=CoreHome&action=index&idSite=294&period=range&date=previous30#?period=day&date=2024-04-30&idSite=294&category=Referrers_Referrers&subcategory=Referrers_WidgetGetAll

Duplication intempestive du dossier entre sa création et la saisie des mesures de sécurisation

Mon premier dossier a été aussitôt dupliqué : après avoir suivi le parcours création avec description du service puis saisie de l'état des mesures de sécurisation, j'ai obtenu deux dossiers avec le même nom de service, l'un avec la page "sécuriser" telle que je l'ai renseignée, l'autre sans (uniquement initiation du dossier avec sa description).

Ensuite, dans la page "décrire" de chaque dossier, la requête PUT du bouton "enregistrer" déclenchait une erreur 422 Unprocessable entity. dont la pile incluait un message "Le nom du service ... existe déjà pour une autre homologation".

Une fois modifié le nom d'un des dossiers, la page "décrire" se comporte de nouveau normalement.

Suggestions :

  • comprendre pourquoi le dossier a été dupliqué lors des premières étapes de sa création
  • permettre la suppression d'un dossier.

Supprimer duplications dans la page « Questions Fréquentes »

Les questions sont répétées, les styles sont répétés, les ancres sont répétées… Peut-être qu'on pourrait plus simplement décrire les questions fréquentes dans les données du référentiel. Reste à voir comment faire pour intégrer proprement des balises HTML dans ces données.

Gérer correctement les données facultatives dans `InformationsHomologation`

À l'heure actuelle, on déclare la saisie d'une donnée comme facultative en surchargeant proprieteSaisie dans la classe métier. Cela entraîne une anomalie toutefois, en ce sens que le statut de saisie ne peut plus être dans l'état « à saisir » – vu qu'un des attributs est considéré comme rempli.

Il convient de corriger cette anomalie en se dotant d'une déclaration plus propre des données facultatives, et en les gérant correctement dans le calcul de statutSaisie.

Le référentiel doit cloner les objets qu'il émet

Aujourd'hui la plupart des objets émis par le référentiel sont des adresses d'un même objet en mémoire. Si jamais ces objets sont modifiés, toutes les classes accédant à ces objets récupèrent des objets modifiés. Le référentiel doit rester un singleton, mais doit devenir immutable.

NB : quelque chose a déjà été fait concernant les mesures récupérées depuis le référentiel (cf PR #436). Il faudrait étendre ce mécanisme aux autres objets du référentiel.

Redistribuer ailleurs le code de `public/modules/soumetsHomologation.js`

Ce code se trouve couplé à public/homologation/formulaireInformationsGenerales.js (on doit notamment bouger le code si dans ce deuxième module on ajoute un bloc ElementsAjoutables). Par ailleurs, dans ce code a plusieurs responsabilités, à la fois de gérer le comportement de la pop-up modale d'alerte sur la criticité et la mise en forme des paramètres à transmettre au serveur lors du POST ou du PUT.

On doit pouvoir faire mieux 🙂 Peut-être en déplaçant une partie de ce code vers public/homologation/formulaireInformationsGenerales.js, une autre partie vers un nouveau module public/modules/modaleInformationsGenerales.js qui serait appelé depuis src/vues/fragments/modaleInformationsGenerales.pug ?

Supprimer duplications dans déclaration des règles de personnalisation

Suite à PR #253

La règle « si application mobile sélectionnée, alors ajouter mesure de téléchargements officiels » se retrouve spécifiée à deux endroits. Il serait bien de supprimer cette duplication.

Plusieurs pistes de réflexion :

  • cumuler les profils (et donc de chaîner les concat•filter) ?
  • composer les profils (et donc pouvoir ajouter dans les critères d'application d'un profil une règle de type profil: "autreProfil"?

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.