-
Notifications
You must be signed in to change notification settings - Fork 6
Refacto #2
Conversation
src/index.js
Outdated
const cors = require("cors"); | ||
const fs = require("fs"); | ||
|
||
const parseWeez = require("./parseWeez"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weez ça me fait rigoler comme nom mais bon, apparement c'est pas nous qui avons choisi !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😂
src/index.js
Outdated
|
||
app.get("/api/v1/:siret", (req, res) => { | ||
const siret = req.params.siret; | ||
if (!siret || siret === "" || siret.length != 14) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
le chaine vide est considérée comme false il me semble, pas besoin de rechecker ça après le !siret
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tout à fait !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je te fais confiance pour la parti deployement, je ne comprends pas grand chose là dessus 😁
c'est le moment de poser des questions :) |
haha il y en aurait trop ! |
## [1.7.2](v1.7.1...v1.7.2) (2023-09-18) ### Bug Fixes * upgrade docker image ([#2](#2)) ([28561c4](28561c4))
Version minimale de l'api qui ne fait que siret2idcc avec la dernière version du fichier + pipeline