PGL/Clyde
PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

openapi-dev #3

Merged
LeoMoulin merged 6 commits from openapi-dev into master 2023-12-11 10:23:15 +01:00
Conversation 2 Commits 6 Files Changed 1 +371 -65
tonitch commented 2023-12-07 23:39:34 +01:00
Owner
Copy Link

Salut @PGL/Owners ,

Voici la première version finale du rest api.

Je l'ai mise sur mon serveur pour que vous puissiez y accéder plus facilement: https://clyde.herisson.ovh/

N'hésitez pas à cliquer partout dans les moindres détails.

(Sur ce je vais dormir, à demain :p)

Salut @PGL/Owners , Voici la première version finale du rest api. Je l'ai mise sur mon serveur pour que vous puissiez y accéder plus facilement: https://clyde.herisson.ovh/ N'hésitez pas à cliquer partout dans les moindres détails. (Sur ce je vais dormir, à demain :p)
LeoMoulin was assigned by tonitch 2023-12-07 23:39:34 +01:00
tonitch self-assigned this 2023-12-07 23:39:34 +01:00
tonitch added 2 commits 2023-12-07 23:39:35 +01:00
Maxime added 1 commit 2023-12-08 13:35:58 +01:00 
ajout d à require
tonitch added 1 commit 2023-12-08 15:52:27 +01:00
tonitch commented 2023-12-08 15:53:50 +01:00
Author
Owner
Copy Link

Je viens de faire la traduction en anglais des endpoints et j'ai mis à jour sur clyde.herisson.ovh Si vous voulez bien y rejeter un oeuil :)

Je viens de faire la traduction en anglais des endpoints et j'ai mis à jour sur clyde.herisson.ovh Si vous voulez bien y rejeter un oeuil :)
tonitch added 2 commits 2023-12-10 21:27:17 +01:00
tonitch commented 2023-12-10 21:44:51 +01:00
Author
Owner
Copy Link

Je pense que l'api générale est finie. pouvez vous y jeter un dernier oeuil.
j'ai fait quelques corrections et j'ai ajouté tout les endpoints à général (donc vous devriez également ajouter tout vos endpoints dans un tag pour qu'on puisse facilement les distinguer dans le document final)

Je l'ai mis à jour sur https://clyde.herisson.ovh

Si vous êtes d'accord, et pour qu'on prenne des bonnes habitudes pour le projet je vous propose de sugérer les modifications (ou non) par le site. pour se faire il suffit de cliquer sur "files changed" (en haut de cette page) puis de faire les réviews. il suffit de cliquer à la gauche d'une ligne pour proposer une modification et puis une fois toutes les propositions faites vous pouvez valider ou non le changement en haut de la page sur le bouton vert.

Je vous met en reviewer. Une fois toutes les reviews faites la branche devrais être envoyé sur master par elle même. à partir de ce point vous pourrez créer un PR par extension et appliquer le même système automatiquement ^^

On peut voir à ça demain si vous le souhaitez

Je pense que l'api générale est finie. pouvez vous y jeter un dernier oeuil. j'ai fait quelques corrections et j'ai ajouté tout les endpoints à général (donc vous devriez également ajouter tout vos endpoints dans un tag pour qu'on puisse facilement les distinguer dans le document final) Je l'ai mis à jour sur https://clyde.herisson.ovh Si vous êtes d'accord, et pour qu'on prenne des bonnes habitudes pour le projet je vous propose de sugérer les modifications (ou non) par le site. pour se faire il suffit de cliquer sur "files changed" (en haut de cette page) puis de faire les réviews. il suffit de cliquer à la gauche d'une ligne pour proposer une modification et puis une fois toutes les propositions faites vous pouvez valider ou non le changement en haut de la page sur le bouton vert. Je vous met en reviewer. Une fois toutes les reviews faites la branche devrais être envoyé sur master par elle même. à partir de ce point vous pourrez créer un PR par extension et appliquer le même système automatiquement ^^ On peut voir à ça demain si vous le souhaitez
tonitch requested review from Maxime 2023-12-10 21:49:58 +01:00
tonitch requested review from LeoMoulin 2023-12-10 21:49:58 +01:00
tonitch requested review from Wal 2023-12-10 21:49:58 +01:00
LeoMoulin reviewed 2023-12-11 10:19:07 +01:00
Documents/proto/general/openapi.yaml
@ -23,3 +25,1 @@
/user:
put:
summary: create a new user
/users:
LeoMoulin commented 2023-12-11 10:19:07 +01:00
Owner
Copy Link

Le service d'inscriptions a juste besoin de lister les étudiants et non tous les utilisateurs.

Le service d'inscriptions a juste besoin de lister les étudiants et non tous les utilisateurs.
tonitch commented 2023-12-11 10:22:46 +01:00
Author
Owner
Copy Link

c'est filtrable ptdr t ki

c'est filtrable ptdr t ki
tonitch marked this conversation as resolved
LeoMoulin requested changes 2023-12-11 10:19:39 +01:00
LeoMoulin left a comment
Owner
Copy Link

Petite erreur au niveau du service d'inscriptions

Petite erreur au niveau du service d'inscriptions
LeoMoulin merged commit c2b6ad0693 into master 2023-12-11 10:23:15 +01:00
Maxime reviewed 2023-12-11 10:23:32 +01:00
Documents/proto/general/openapi.yaml
@ -5,2 +5,4 @@
This is the Clyde's api documentation. <br/>
Each endpoints are organised by endpoints **AND** by extensions making it easier to navigate.
version: 0.0.1
Maxime commented 2023-12-11 10:23:32 +01:00
Owner
Copy Link

good

good
tonitch marked this conversation as resolved
Wal approved these changes 2023-12-11 10:24:33 +01:00
Maxime reviewed 2023-12-11 10:24:45 +01:00
Documents/proto/general/openapi.yaml
@ -61,3 +139,3 @@
'200':
description: Cursus found
description: User found
content:
Maxime commented 2023-12-11 10:24:45 +01:00
Owner
Copy Link

why user found ?

why user found ?
tonitch commented 2023-12-11 10:28:00 +01:00
Author
Owner
Copy Link

parce que c'est plus déscriptif

parce que c'est plus déscriptif
tonitch marked this conversation as resolved
Maxime approved these changes 2023-12-11 10:27:47 +01:00
Maxime left a comment
Owner
Copy Link

good

good
LeoMoulin approved these changes 2023-12-11 10:29:09 +01:00
tonitch deleted branch openapi-dev 2023-12-11 10:30:30 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Wal
Maxime
LeoMoulin
Labels
Clear labels   
Bug

   
Done

finie, en attente de validation pour ou pret à la fermeture

  
Pas urgent

A faire que si on a le temps

  
Proposition

   
Question

   
TODO

   
Tests

   
URGENT BORDEL DE Q

   
Waiting for review

   
back

   
front
No Label
Bug
Done
Pas urgent
Proposition
Question
TODO
Tests
URGENT BORDEL DE Q
Waiting for review
back
front
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
tonitch
LeoMoulin
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PGL/Clyde#3
No description provided.
Delete Branch "openapi-dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?