mission-apprentissage / referentiel

GNU Affero General Public License v3.0
3 stars 1 forks source link

Page accueil du site #198

Closed FlavieSauvebois closed 4 weeks ago

FlavieSauvebois commented 3 months ago

Est-ce que les données chiffrées sont mises à jour en temps réel ? Il contient 97% d’organismes validés sur le territoire national. Les 3974 organismes référencés dans le référentiel sont :

Est-ce que la partie sur l'intégration des données ACCE ne serait pas à supprimer en attendant que la récupération des données soit réellement effective ?

FlavieSauvebois commented 3 months ago

J'ai quand même un doute sur : "Les 3974 organismes référencés dans le référentiel sont "_ Est-ce que qq peut vérifier que ce chiffre est bien mise à jour en fonction des données car si je consulte le Référentiel et que j'applique les filtres pour faire ressortir les OF remontant en expertise (siret valide, nature déclarée et Qualiopi oui) j'ai un chiffre qui tourne autour des 5000 OF et non 3974.

Par ailleurs dans le paragraphe du dessous il est écrit : _Les 3976 organismes référencés dans le référentiel sont :

gasilber commented 3 months ago

Je pense qu'effectivement on peut enlever la mention de la base ACCE dans la description. Pour la modalité de calcul, je laisse @ldaverio répondre (ingénieur en informatique aux mines et qui nous rejoint sur ce projet).

ldaverio commented 3 months ago

Bonjour à tous, je suis très heureux de participer au projet.

Les données en question (nombre d'organismes référencés, et pourcentage d'organismes validés) sont récupérés via un appel à l'API dans le frontend :

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/ui/src/accueil/AccueilPage.jsx#L21

Cet appel récupère un objet statsavec deux clés, total et valides :

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/ui/src/accueil/AccueilPage.jsx#L43-L44

et :

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/ui/src/accueil/AccueilPage.jsx#L50

Côté serveur, les données sont calculées comme suit :

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/server/src/http/routes/statsRoutes.js#L38-L42

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/server/src/http/routes/statsRoutes.js#L44-L50

Pour l'instant, je ne peux pas en dire plus, juste identifier le cheminement ayant conduit à ces résultats.

FlavieSauvebois commented 3 months ago

Bonjour Laurent,

Bienvenue dans l'équipe, nous sommes ravis de t'accueillir. Merci pour tes explications qui me permettent d'y voir plus clair et de comprendre pourquoi les données chiffrées affichées sur la page d'accueil sont différentes de celles que j'avais en termes de volume.

Déjà on peut confirmer que la mise à jour des chiffres se fait en temps réels (ou du moins chaque jour) car le nombre d'OF en expertise affichés a bien évolué entre nos deux posts.

Par ailleurs, le chiffre actuel qui est affiché (Les 3979 organismes référencés dans le référentiel sont ) : comptabilise uniquement les OF responsable et responsable formateur qui sont Qualiopi OUI avec un SIRET actif. Pour que le chiffre soit juste : il faut simplement ajouter les OF dont la nature est formateur uniquement. On aura alors la volumétrie suivante pour aujourd'hui : 5009 OF Et il s'agira de la volumétrie exacte car les OF formateurs ont été ajouté dans un second à l'expertise (la correction n'avait donc pas été faite dans le code).

Pour le calcul du % d'OF c'est OK

Nous pourrons reparler de tout ça jeudi lors de notre visio qui a lieu tous les 15 jours. @ldaverio peux-tu me donner ton mail afin que te transfère l’invitation ?

Bonne journée

ldaverio commented 3 months ago

Bonjour Flavie,

Par ailleurs, le chiffre actuel qui est affiché (Les 3979 organismes référencés dans le référentiel sont ) : comptabilise uniquement les OF responsable et responsable formateur qui sont Qualiopi OUI avec un SIRET actif. Pour que le chiffre soit juste : il faut simplement ajouter les OF dont la nature est formateur uniquement.

Effectivement, si je modifie le code en :

      const validationQuery = {
        etat_administratif: "actif",
        qualiopi: true,
        nature: { $in: ["formateur", "responsable", "responsable_formateur"] },
      };

J'obtiens bien le nombre correct.

Mon adresse e-mail est laurent.daverio@minesparis.psl.eu. Je peux me rendre disponible jeudi pour la réunion.

Bonne journée :)

ldaverio commented 3 months ago

J'obtiens bien le nombre correct.

Par contre, sur la page "organismes", en sélectionnant les mêmes critères, je n'obtiens pas la même valeur : 5222 organismes au lieu de 5008 sur mes données de test.

Screenshot 2024-06-24 at 11-35-50 Référentiel

FlavieSauvebois commented 3 months ago

C'est normal car sur le site tu ne peux pas faire de filtre sur l'état administratif des OF (actif ou fermé). Seuls ceux qui sont actifs remontent en expertise et sont donc comptabilisés.

ldaverio commented 3 months ago

J'ai voulu faire le commit correspondant à la modification demandée. Malheureusement, un test échoue sur ma configuration (MacOS, installation sans Docker) :

https://github.com/mission-apprentissage/referentiel/blob/be34381322b7f3a9c6da3c3dc3f1d0c23390e505/server/tests/http/organismesRoutes-test.js#L213-L218

Le test précédent (recherche des organismes actifs) passe sans problème, la différence est le "é" dans l'URL, ligne 218. Si je modifie cette ligne en :

const response = await httpClient.get(`/api/v1/organismes?etat_administratif=${encodeURI("fermé")}`);

le test passe.

Étant donné que le problème ne semble pas exister sur la configuration "dockerisée" standard, que me suggérez-vous de faire ? Merci d'avance :)

ldaverio commented 3 months ago

Si je mets à jour le paquet NPM axiosist de la version 0.1.0 (une version 0.x ne devrait pas être utilisée en production, au passage) en version 1.2.0 (la dernière), il y a toujours le même message d'erreur pendant les tests, mais le test réussit. Par conséquent, je peux faire mon commit :)

gasilber commented 3 months ago

Modifié et déployé en développement

gasilber commented 2 months ago

Déployé en développement: https://dev-referentiel.apprentissage.onisep.fr

FlavieSauvebois commented 2 months ago

Ok pour mise en prod