Pierre TOP - TOPI

Results 27 issues of Pierre TOP - TOPI

## :unicorn: Problème Certaines assertions chai sont toujours vraies, car leur syntaxe est incorrecte. Cela peut entraîner des faux négatifs: - une régression est introduite dans l'implémentation: - le test...

:eyes: Tech Review Needed
cross-team

## :unicorn: Problème L'image utilisée pour instancier la BDD locale n'est pas celle de Scalingo. Des écarts de comportement peuvent apparaître. ## :robot: Solution Utiliser l'image Scalingo https://hub.docker.com/r/scalingo/postgresql/tags ## :rainbow:...

:eyes: Tech Review Needed
cross-team

## :unicorn: Problème Des erreurs ont été remontées par les nouveaux arrivants lors de l'installation. De plus, la procédure d'installation n'est pas facile à localiser. Et elle est longue en...

Tech Review OK
cross-team

## :unicorn: Problème L'image utilisée pour instancier le serveur redis local n'est pas celle de Scalingo. Des écarts de comportement peuvent apparaître. ## :robot: Solution Utiliser l'image Scalingo https://hub.docker.com/r/scalingo/redis ##...

:eyes: Tech Review Needed
cross-team

## :unicorn: Problème L'arrêt du serveur web sur `SIGINT` dans `index.js` comprend un arrêt forcé du processus (`process.exit(0)`). Il sert à arrêter le processus en présence de connexions redis actives....

:eyes: Tech Review Needed
cross-team

La v4 du client redis retourne des promesses https://github.com/redis/node-redis/blob/master/CHANGELOG.md#v400---24-nov-2021 On peut supprimer l'ajout de promisify dans api/lib/infrastructure/utils/RedisClient.js

La doc est https://github.com/1024pix/pix/blob/dev/INSTALLATION.md La commande suivante sort en erreur en local sur plusieurs configurations `npm exec run-p start:api start:mon-pix`

## :unicorn: Problème Le logger devient complexe: - activation:désactivation en fonction de l'environnement - affichage prettyPrint - suppression de données sensibles Des [évolutions](https://github.com/1024pix/pix/pull/4167) récentes complexifient encore le fonctionnement. Or, il...

Development in progress
cross-team

Le logger de l'API n'est pas testé, en conséquence: - il faut des tests manuels à chaque modification (et il y a beacoup de scénarios) sinon risque de régression sur...

Afin d'isoler le domaine des détails d'implémentation (Clean Architecture), le domaine doit faire référence à l'infrastructure de manière indirecte (injection de dépendances via les argument du use-case). Il ne devrait...