Skip to content

remarques-architecteBytel#2

Open
ekossent wants to merge 6 commits into
before-interop:mainfrom
ekossent:main
Open

remarques-architecteBytel#2
ekossent wants to merge 6 commits into
before-interop:mainfrom
ekossent:main

Conversation

@ekossent

@ekossent ekossent commented Mar 4, 2022

Copy link
Copy Markdown

Bonjour,
Voici les recommandations des architectes Bytel :
Flux 1 M1 : Renommage de l'intervention DO en intervention OI=>mise à jour de l'url en POST/interventions-oi
création de l'intervention OI iso intervention DO chez l'OI
Flux 1 M3 : création d'un service PUT/interventions-oi qui permet de modifier l'intervention OI en l'occurrence avec la date de fin de l'intervention
Flux 2 M2 et M4 : notifierInterventionOI => l'OI notifie l'OC de la création puis des mises à jour faites sur l'intervention OI
POST /interventions-oi/notification
Flux 3 : M5 déclarer les accès Hors service OC auprés de l'OI. Nous pensons qu'il s'agit plutôt de déclarer les accès hors service des clients. /interventions-oi/{refIntervOI}/acces-oc-hs
Gestion des erreurs HTTP :
Revision des codes HTTP et ajout d'une structure Body pour l'erreur exemple : {   "error": "INVALID_FORMAT",   "error_description": "CHAMPS_NON_RENSEIGNE",   "error_parameters": [ "name"  ] }

@ekossent

ekossent commented Mar 4, 2022

Copy link
Copy Markdown
Author

Je viens de rajouter un commit sur la partie sécurité des API : Protocole Oauth2 Client Credentials

@alexisthethe

alexisthethe commented Jan 30, 2024

Copy link
Copy Markdown

Avis d'Axione sur la proposition :

Nous préférons ne pas changer le nom des routes. Une convention a déjà été validée, et nous pensons qu'il est plus judicieux de se concentrer sur la spécification et l’implémentation des protocoles en cours et à venir (malfaçon, anomalie adresse, ...).

Modifier un protocole qui fonctionne bien actuellement ne semble pas nécessaire.

Comment thread E-Intervention-v1.0.0.swagger.json
Comment thread E-Intervention-v1.0.0.swagger.json
Comment thread E-Intervention-v1.0.0.swagger.json
Comment thread E-Intervention-v1.0.0.swagger.json
Comment thread E-Intervention-v1.0.0.swagger.json
@richard-olvera

Copy link
Copy Markdown

Sinon dernier point est ce que ca vous derange si on passe en YAML ?

c'est moins lourd à lire

@f-teyssier

Copy link
Copy Markdown
Contributor

a clôturer suite à l'ancienneté

@f-teyssier f-teyssier requested a review from Fat-slitane March 20, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants