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

salutcesmoileo #4

Merged
tonitch merged 10 commits from salutcesmoileo into master 2023-12-14 14:37:27 +01:00
Conversation 0 Commits 10 Files Changed 1 +28 -33
1 changed files with 28 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 74e6d18f97 - Show all commits

39
Documents/proto/general/openapi.yaml
View File

@ -374,6 +374,7 @@ paths:
summary: Give an array of every requests of the specified type summary: Give an array of every requests of the specified type
tags : tags :
- Requests - Requests
- SI
- Ext (Student Registration) - Ext (Student Registration)
responses : responses :
'200': '200':
@ -383,24 +384,17 @@ paths:
schema : schema :
type : array type : array
items : items :
oneOf: allOf:
- type : object - type : object
properties : properties :
id : id :
type : integer type : integer
Firstname : - oneOf :
type : string
Name :
type : string
Date :
type : string
description : Follow the iso 8601 ("YYYY-MM-DD")
State :
type : string
- $ref : '#/components/schemas/RRRequest' - $ref : '#/components/schemas/RRRequest'
tonitch marked this conversation as resolved Outdated
Maxime commented 2023-12-11 12:05:12 +01:00
Outdated
Review
Copy Link

PAS BIEN VAIS TE FRAPPER AVEC MA GROSSSE BATTE

PAS BIEN VAIS TE FRAPPER AVEC MA GROSSSE BATTE
👀 1
- $ref : '#/components/schemas/SSRequest' - $ref : '#/components/schemas/SSRequest'
- $ref : '#/components/schemas/UnRegRequest' - $ref : '#/components/schemas/UnRegRequest'
- $ref : '#/components/schemas/ExempRequest' - $ref : '#/components/schemas/ExempRequest'
- $ref : '#/components/schemas/RegRequest'
discriminator: discriminator:
propertyName : type propertyName : type
@ -416,6 +410,8 @@ paths:
summary: create a request summary: create a request
tags : tags :
- Requests - Requests
- Guest
tonitch commented 2023-12-12 22:40:33 +01:00
Outdated
Review
Copy Link

tu peux ajouter la ligne
enum: blop1, blop2, blop3, ...
en dessous et au même niveau que type pour spécifier les différent paramètres disponible.

tu peux ajouter la ligne `enum: blop1, blop2, blop3, ...` en dessous et au même niveau que type pour spécifier les différent paramètres disponible.
- Student
- Ext (Student Registration) - Ext (Student Registration)
tonitch commented 2023-12-12 22:42:25 +01:00
Outdated
Review
Copy Link

si tu appliques les précédents commentaire tu as soit le choix de laisser le post dans /Requests/{type} soit de le mettre avec /Requests et dans ce cas il suffit de mettre le type en paramètre dans requestBody ou dans query (je pense que requestBody serait mieux dans le cas présent)

si tu appliques les précédents commentaire tu as soit le choix de laisser le post dans `/Requests/{type}` soit de le mettre avec `/Requests` et dans ce cas il suffit de mettre le type en paramètre dans `requestBody` ou dans `query` (je pense que requestBody serait mieux dans le cas présent)
security: security:
tonitch marked this conversation as resolved Outdated
Maxime commented 2023-12-11 12:05:53 +01:00
Outdated
Review
Copy Link

PAS studentRegistrattion

PAS studentRegistrattion
- bearer: [] - bearer: []
@ -438,7 +434,7 @@ paths:
'401': '401':
$ref: '#/components/responses/UnauthorizedError' $ref: '#/components/responses/UnauthorizedError'
/Requests/{type}{id}: /Requests/{type}/{id}:
parameters: parameters:
- name: type - name: type
in: path in: path
@ -458,6 +454,8 @@ paths:
summary : Get all the data composing a request summary : Get all the data composing a request
tonitch marked this conversation as resolved Outdated
tonitch commented 2023-12-11 23:27:27 +01:00
Outdated
Review
Copy Link

Est-ce vraiment nécéssaire de faire des ré-ré register ? est-ce que ca ne serais pas une sous-classe de register avec peut être un bool pour dire si c'est la première fois ou non ?

Est-ce vraiment nécéssaire de faire des ré-ré register ? est-ce que ca ne serais pas une sous-classe de register avec peut être un bool pour dire si c'est la première fois ou non ?
LeoMoulin commented 2023-12-12 10:33:18 +01:00
Outdated
Review
Copy Link

Non, cela impliquerait une redondance d'informations. Quand un étudiant se réinscrit on a pas besoin de restocker son nom, son prénom, son adresse, ...
On les a déja avec le matricule.

Non, cela impliquerait une redondance d'informations. Quand un étudiant se réinscrit on a pas besoin de restocker son nom, son prénom, son adresse, ... On les a déja avec le matricule.
tonitch commented 2023-12-12 22:20:18 +01:00
Outdated
Review
Copy Link

Le fait de metre des end-points n'a rien à voir avec le stockage d'information tho. ton end point peut être clair et concis tandis que ta base de données et ton backend s'occupent de chercher les infos ou tu veux. en clair tu ne dois pas suivre ce qui est fait dans class et db

Le fait de metre des end-points n'a rien à voir avec le stockage d'information tho. ton end point peut être clair et concis tandis que ta base de données et ton backend s'occupent de chercher les infos ou tu veux. en clair tu ne dois pas suivre ce qui est fait dans class et db
tags : tags :
- Requests - Requests
- SI
- Teacher
- Ext (Student Registration) - Ext (Student Registration)
responses : responses :
'200': '200':
@ -465,7 +463,12 @@ paths:
content : content :
application/json : application/json :
schema : schema :
oneOf: allOf:
- type : object
properties :
id :
type : integer
- oneOf :
- $ref : '#/components/schemas/RRRequest' - $ref : '#/components/schemas/RRRequest'
- $ref : '#/components/schemas/SSRequest' - $ref : '#/components/schemas/SSRequest'
- $ref : '#/components/schemas/UnRegRequest' - $ref : '#/components/schemas/UnRegRequest'
@ -478,6 +481,8 @@ paths:
summary: Change the state of request summary: Change the state of request
tags: tags:
- Requests - Requests
- SI
- Teacher
- Ext (Student Registration) - Ext (Student Registration)
security: security:
- bearer: [] - bearer: []
@ -597,8 +602,6 @@ components:
RegRequest: RegRequest:
type : object type : object
properties: properties:
id:
type : integer
Name : Name :
type: string type: string
Firstname : Firstname :
@ -636,8 +639,6 @@ components:
RRRequest: RRRequest:
type : object type : object
properties : properties :
id :
type : integer
RegNo : RegNo :
type : integer type : integer
NewCursusid : NewCursusid :
@ -649,8 +650,6 @@ components:
SSRequest: SSRequest:
type : object type : object
properties : properties :
id :
type : integer
RegNo : RegNo :
type : integer type : integer
Amount : Amount :
@ -662,8 +661,6 @@ components:
UnRegRequest : UnRegRequest :
type : object type : object
properties : properties :
id :
type : integer
RegNo : RegNo :
type : integer type : integer
State : State :
@ -672,8 +669,6 @@ components:
ExempRequest : ExempRequest :
type : object type : object
properties : properties :
id:
type : integer
RegNo : RegNo :
type : integer type : integer
Courseid : Courseid :