codegouvfr / codegouv-web-poc

0 stars 0 forks source link

Corrections Catalogue #29

Open hjonin opened 1 year ago

hjonin commented 1 year ago

Corrections bloquantes

Corrections majeures

Améliorations

Nice to have

mineraux commented 1 year ago

@hjonin En ce qui concerne les valeurs le Autorité de régulation, je pense que l'erreur est en back.

Pour débuger, j'ai mapper les repos et tester le nom de leur organisation. Quand on selectionne l'administration "Autorité de régulation", l'organisation associée est bien ARCEP [Autorité de Régulation des Communications Électroniques et des Postes]. Or aucun repo n'a de nom incluant les premieres lettres :

    repos.map(repo => {
        if (repo.organisation_name.includes("arce")) {
            console.log(repo) => ne log rien
        }
    })

Pour Analyse-Image_APC, je pense que l'erreur est aussi en back. Voici un log des organisations des repos, trié par ordre alphabétique. Cette valeur n'apparait pas. Capture d’écran du 2023-05-04 14-47-31

mineraux commented 1 year ago

Pour le filtre des dépendances, on a besoin de la clé repository_urls, le nom n'est pas suffisant

hjonin commented 1 year ago

https://github.com/codegouvfr/codegouv-web/issues/29#issuecomment-1534738200 tu as raison, j'avais oublié ce fonctionnement. C'est moi qui avais créé un endpoint (getDependencyNames) inutile, je le supprime. :)

hjonin commented 1 year ago

https://github.com/codegouvfr/codegouv-web/issues/29#issuecomment-1534730721 merci beaucoup d'avoir regardé, je prends la suite côté back du coup !

hjonin commented 1 year ago

A voir côté backend, certaines organisations n'ont pas de nom, certaines organistaions en double.

hjonin commented 1 year ago

Pour le merge :

hjonin commented 1 year ago

Nice to have :