Comments (7)
J'ajouterais une 3e option, car le but est uniquement de détecter les 404 :
3. uniquement fail le build si status === 404, ignorer la condition sur datagouv sinon.
from api.gouv.fr.
Facile a implémenter
from api.gouv.fr.
Pour moi ici c'était vraiment un problème temporaire de data.gouv, la 404 à priori c'est "permanent" (et je suis d'accord ça devrait être géré).
Je ne suis pas sûr que gérer le 404 résolve ce problème ici btw.
from api.gouv.fr.
Ah yes me suis mal exprimé.
En gros je propose :
- 404 : permanent = lien mort, c'est un peu voulu que le build pete c'est a ca que servait ce check
- 2xx ou 5xx peut importe, on peu ignorer
from api.gouv.fr.
N'ayant pas la connaissance métier je ne peux pas m'exprimer sur le 5XX peu importe, on peut ignorer
, mais du coup de ce que je comprends de ton message c'est que l'on n'a pas besoin des infos de data.gouv pour build ? Dans ce cas là pourquoi on les fetch ..?
from api.gouv.fr.
L'info qu'on souhaite obtenir de datagouv, c'est est ce que cette ressource existe encore.
En effet certain(e) FD ont l'habitude de supprimer et republier les jeux de donnée.
A l'époque c'était ce que j'avais mis en place pour être certain de ne pas déployer des liens morts. Si datagouv retourne 404, c'est que tu dois mettre des liens à jours.
Mais :
- à l'époque j'étais bien plus impliqué sur apigouv et j'acceptais cette situation un peu bancale (la dépendance a datagouv ne devrait pas pouvoir péter le build) car je savais que je pourrais la contourner si nécessaire
- datagouv a peut etre introduit des liens permanents depuis, qui éviteraient pas mal d'emm***
from api.gouv.fr.
Dispo pour en parler de vive voix ou au tel si besoin
from api.gouv.fr.
Related Issues (20)
- Bug parcours utilisateur HOT 1
- Informer des scopes API Statut étudiant / statut étudiant boursier FranceConnecté
- [Doc] Remplacer "France connectée" par "FranceConnectée"
- Ipi
- Spelling mistake by searching « private data »
- API Géorisques - Absence de la Famille de Risque 'Retrait Gonflement des argiles'
- Empty keyword
- Démonstration API SIREN retourne code 410
- La recherche devrait se propager dans la barre d'adresse
- Erreur 400 lors de la recherche des numéros de rue compris entre 10 à 99. HOT 2
- Mise à jour de swagger UI en version 5 (support OpenAPI 3.1.0)
- Modification FAQ api.gouv.fr
- API RNCS n’existe pas/plus (erreur 404 sur le site) HOT 1
- Bug Paiement - demarches-plaisance.gouv.fr
- API2-80
- API2-80
- typo, doublons de les
- typo : https://api.gouv.fr/les-api/api-particulier HOT 1
- Suppression de l'API Sirene données ouvertes HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from api.gouv.fr.