salutcesmoileo #4

Merged
tonitch merged 10 commits from salutcesmoileo into master 2023-12-14 14:37:27 +01:00
Showing only changes of commit e5717e4c3d - Show all commits

View File

@ -374,9 +374,6 @@ paths:
$ref : '#/components/schemas/RegRequests'
/RegRequest/{id}:
get :
summary : Get all the data composing a register request
@ -423,7 +420,7 @@ paths:
/RegRequest:
post :
summary: Add a new student
summary: Add a new register request
tags :
- RegisterRequests
security:
@ -440,6 +437,295 @@ paths:
'401':
$ref: '#/components/responses/UnauthorizedError'
/ReRegRequest:
get :
tonitch marked this conversation as resolved
Review

Il manque un /
/Requests/{type}/{id}

Il manque un / `/Requests/{type}/{id}`
summary : Get an array of every reregister requests
tags :
- ReregisterRequests
responses :
'200':
description : OK
tonitch marked this conversation as resolved Outdated

Ici en utilisant le ref, l'id est présent. sauf que tu ne peux pas post avec un id vu qu'il est généré par le post.

Ici en utilisant le ref, l'id est présent. sauf que tu ne peux pas post avec un id vu qu'il est généré par le post.
content :
application/json :
schema :
tonitch marked this conversation as resolved Outdated

New Student created -> Register request created ?

New Student created -> Register request created ?
type : array
items :
$ref : '#/components/schemas/ReRegRequest'
post :
tonitch marked this conversation as resolved Outdated

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 ?

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.

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
summary: Add a new ReRegister request
tags :
- ReregisterRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
$ref: '#/components/schemas/ReRegRequest'
responses:
'201':
description: New ReRegister request created
'401':
$ref: '#/components/responses/UnauthorizedError'
/ReRegRequest/{id}:
parameters:
- name: id
in: path
description: id of a reregistrationrequest
required: true
schema:
type: integer
get:
summary : get the data of a single reregistration request
tags :
- ReregisterRequests
responses :
'200':
description : OK
content :
application/json :
schema :
$ref : '#/components/schemas/ReRegRequest'
patch:
summary: Change the state of a reregister request
tags:
- ReregisterRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
type: object
properties:
State:
type: string
responses:
'201':
description: State modified
'401':
$ref: '#/components/responses/UnauthorizedError'
/SSRequest:
get :
summary : Get an array of every scholarship requests
tags :
- ScholarshipRequests
responses :
'200':
description : OK
content :
application/json :
schema :
type : array
items :
$ref : '#/components/schemas/SSRequest'
post :
summary: Add a new scholarship request
tags :
- ScholarshipRequests
security:
tonitch marked this conversation as resolved Outdated

Same, est-ce que tu ne ferais pas un
(GET) /requests -> array of all requests
(GET) /requests/{type} -> array of all requests of

et pour les post c'est juste un paramètre ? c'est moins verbeux et plus lisible ?

Same, est-ce que tu ne ferais pas un (GET) /requests -> array of all requests (GET) /requests/{type} -> array of all requests of <type> et pour les post c'est juste un paramètre ? c'est moins verbeux et plus lisible ?

Je suis d'accord avec toi ! Mais comment spécifier le type de retour qui change en fonction du paramètre {type} ducoup ? Et je n'ai pas compris la deuxième partie de la question

Je suis d'accord avec toi ! Mais comment spécifier le type de retour qui change en fonction du paramètre {type} ducoup ? Et je n'ai pas compris la deuxième partie de la question

EDIT : c'est fait

EDIT : c'est fait
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
$ref: '#/components/schemas/SSRequest'
responses:
'201':
description: New scholarship request created
'401':
$ref: '#/components/responses/UnauthorizedError'
/SSRequest/{id}:
parameters:
- name: id

peut-être rajouter user dans les tags ?

peut-être rajouter user dans les tags ?

les tags sont fixs !

les tags sont fixs !
in: path
description: id of a scholarshiprequest
required: true
schema:
type: integer
get:
summary : get the data of a single scholarship request
tags :
- ScholarshipRequests
responses :
'200':
description : OK
content :
application/json :
schema :
$ref : '#/components/schemas/SSRequest'
patch:
summary: Change the state of a scholarship request
tags:
- ScholarshipRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
type: object
properties:
State:
type: string
responses:
'201':
description: State modified
'401':
$ref: '#/components/responses/UnauthorizedError'
/UnRegRequest:
get :
summary : Get an array of every unregister requests
tags :
- UnregisterRequests
responses :
'200':
description : OK
content :
application/json :
schema :
type : array
items :
$ref : '#/components/schemas/UnRegRequest'
post :
summary: Add a new unregister request
tags :
- UnregisterRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
$ref: '#/components/schemas/UnRegRequest'
responses:
'201':
description: New unregister request created
'401':
$ref: '#/components/responses/UnauthorizedError'
/UnRegRequest{id}:
parameters:
- name: id
in: path
description: id of a unregistrationrequest
required: true
schema:
type: integer
get:
summary : get the data of a single unregistration request
tags :
- UnregisterRequests
responses :
'200':
description : OK
content :
application/json :
schema :
$ref : '#/components/schemas/UnRegRequest'
patch:
summary: Change the state of a unregister request
tags:
- UnregisterRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
type: object
properties:
State:
type: string
responses:
'201':
description: State modified
'401':
$ref: '#/components/responses/UnauthorizedError'
/ExempRequest:
get :
summary : Get an array of every exemptions requests
tags :
- ExemptionRequests
responses :
'200':
description : OK
content :
application/json :
schema :
type : array
items :
$ref : '#/components/schemas/ExempRequest'
post :
summary: Add a new exemption request
tags :
- ExemptionRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
$ref: '#/components/schemas/ExempRequest'
responses:
'201':
description: New exemption request created
'401':
$ref: '#/components/responses/UnauthorizedError'
/ExempRequest/{id}:
parameters:
- name: id
in: path
description: id of a exemption request
required: true
schema:
type: integer
get:
summary : get the data of a single exemption request
tags :
- ExemptionRequests
responses :
'200':
description : OK
content :
application/json :
schema :
$ref : '#/components/schemas/ExempRequest'
patch:
summary: Change the state of a exemption request
tags:
- ExemptionRequests
security:
- bearer: []
requestBody:
required: true
content:
application/x-www-form-urlencoded:
schema:
type: object
properties:
State:
type: string
responses:
'201':
description: State modified
'401':
$ref: '#/components/responses/UnauthorizedError'
components:
securitySchemes:
bearer:
@ -598,7 +884,7 @@ components:
type : string
SSRequests:
SSRequest:
type : object
properties :
tonitch marked this conversation as resolved Outdated

à quoi sert RegRequests ?

à quoi sert RegRequest**s** ?

C'est une genre de sous partie de RegRequest servant un peu de "résumé", je l'ai crée pour palier au fait que quand on arrive sur le menu de séléction des demandes d'inscriptions, je veux les lister, ce faisant je n'ai pas besoin de load toutes les informations de toutes les demandes d'inscription, j'ai juste besoin de quelques attributs pour les lister.

C'est une genre de sous partie de RegRequest servant un peu de "résumé", je l'ai crée pour palier au fait que quand on arrive sur le menu de séléction des demandes d'inscriptions, je veux les lister, ce faisant je n'ai pas besoin de load toutes les informations de toutes les demandes d'inscription, j'ai juste besoin de quelques attributs pour les lister.
id :