Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Refacto #2

Merged
merged 11 commits into from
Aug 19, 2019
Merged

Refacto #2

merged 11 commits into from
Aug 19, 2019

Conversation

revolunet
Copy link
Member

@revolunet revolunet commented Jul 30, 2019

Version minimale de l'api qui ne fait que siret2idcc avec la dernière version du fichier + pipeline

@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 14:23 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 14:30 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 14:34 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 14:39 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 14:52 Inactive
@SocialGroovyBot SocialGroovyBot requested a deployment to dev.factory July 30, 2019 14:58 Abandoned
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 15:11 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 15:16 Inactive
@SocialGroovyBot SocialGroovyBot requested a deployment to dev.factory July 30, 2019 15:23 Abandoned
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 30, 2019 15:26 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 31, 2019 08:22 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 31, 2019 08:29 Inactive
@revolunet revolunet marked this pull request as ready for review July 31, 2019 08:30
src/index.js Outdated
const cors = require("cors");
const fs = require("fs");

const parseWeez = require("./parseWeez");

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 !

Copy link
Member Author

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) {
Copy link

@UnbearableBear UnbearableBear Jul 31, 2019

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

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tout à fait !

Copy link

@UnbearableBear UnbearableBear left a 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 😁

@revolunet
Copy link
Member Author

c'est le moment de poser des questions :)

@UnbearableBear
Copy link

haha il y en aurait trop !

@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory July 31, 2019 09:50 Inactive
@SocialGroovyBot SocialGroovyBot temporarily deployed to dev.factory August 19, 2019 07:30 Inactive
@revolunet revolunet merged commit f7c04d2 into master Aug 19, 2019
@lionelB lionelB deleted the refacto branch December 6, 2019 10:32
SocialGroovyBot pushed a commit that referenced this pull request Sep 18, 2023
SocialGroovyBot added a commit that referenced this pull request Sep 18, 2023
## [1.7.2](v1.7.1...v1.7.2) (2023-09-18)

### Bug Fixes

* upgrade docker image ([#2](#2)) ([28561c4](28561c4))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants