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

GestionHoraire #6

Merged
Wal merged 5 commits from GestionHoraire into master 2023-12-13 15:00:49 +01:00
Conversation 0 Commits 5 Files Changed 1 +314 -3
Wal commented 2023-12-11 10:34:09 +01:00
Owner
Copy Link
No description provided.
Wal added 1 commit 2023-12-11 10:34:10 +01:00
Wal added 1 commit 2023-12-11 11:02:55 +01:00
Wal added 1 commit 2023-12-11 11:32:14 +01:00
Wal changed title from WIP: GestionHoraire to GestionHoraire 2023-12-11 11:32:45 +01:00
LeoMoulin approved these changes 2023-12-11 14:12:04 +01:00
Wal added 1 commit 2023-12-11 14:13:39 +01:00
Maxime requested changes 2023-12-11 23:27:17 +01:00
Maxime left a comment
Owner
Copy Link

quelques oubli dans les patch/post des profs et du secretariat et changer Request -> ShceduleRequest mais à part ça seems fine

quelques oubli dans les patch/post des profs et du secretariat et changer Request -> ShceduleRequest mais à part ça seems fine
Documents/proto/general/openapi.yaml
@ -368,0 +389,4 @@
type: object
properties:
date:
type: string
Maxime commented 2023-12-11 23:19:35 +01:00
Owner
Copy Link

Tu ne précises pas qui donne le cours, jsp si c'est un problème mais du coup il y a plusieurs personnes qui peuvent le donner comme prof référant ou assistant.

Tu ne précises pas qui donne le cours, jsp si c'est un problème mais du coup il y a plusieurs personnes qui peuvent le donner comme prof référant ou assistant.
tonitch commented 2023-12-11 23:42:56 +01:00
Owner
Copy Link

Vu qu'il stock IDcourse (que j'aurais personellement nommé CourseID) l'enseignant est sensé être dedans non ?

Vu qu'il stock IDcourse (que j'aurais personellement nommé CourseID) l'enseignant est sensé être dedans non ?
Maxime commented 2023-12-11 23:48:57 +01:00
Owner
Copy Link

alors oui le courseId donne accès à un array de profs mais du coup on sait pas spécifier lequel donne le cours dans son implémentation

alors oui le courseId donne accès à un array de profs mais du coup on sait pas spécifier lequel donne le cours dans son implémentation
👍 1
Wal commented 2023-12-12 08:13:35 +01:00
Author
Owner
Copy Link

Jcomptais mettre les teachers quand cetait theorique et examen et les assistants quand c'est tp mais apres jpeux spécifier meme si sur l'hyperplannng et tout il y a la liste des profs qui peuvent donner cours

Jcomptais mettre les teachers quand cetait theorique et examen et les assistants quand c'est tp mais apres jpeux spécifier meme si sur l'hyperplannng et tout il y a la liste des profs qui peuvent donner cours
tonitch marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +435,4 @@
content:
application/x-www-form-urlencoded:
schema:
type: object
Maxime commented 2023-12-11 23:21:13 +01:00
Owner
Copy Link

Du coup, est-ce qu'il y aurrait pas moyen qu'un professeur demande de mettre à l'horaire que finalement ça sera son assistant qui donnera le cours ? (c'est des détails en vrai mais autant y réfléchir un peu)

Du coup, est-ce qu'il y aurrait pas moyen qu'un professeur demande de mettre à l'horaire que finalement ça sera son assistant qui donnera le cours ? (c'est des détails en vrai mais autant y réfléchir un peu)
Wal commented 2023-12-12 08:16:42 +01:00
Author
Owner
Copy Link

Imo ca devient trop spécifique et les assistants peuvent donner que les tps selon moi et au pire le prof peut demander. et ca peux meme aller dans la gestion de cours qui est pas mon dos comme cest une autre extension perso

Imo ca devient trop spécifique et les assistants peuvent donner que les tps selon moi et au pire le prof peut demander. et ca peux meme aller dans la gestion de cours qui est pas mon dos comme cest une autre extension perso
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +470,4 @@
summary: Create a new request
tags:
- Teacher
- request
Maxime commented 2023-12-11 23:22:21 +01:00
Owner
Copy Link

tu as laissé request ici peut-être le changer en ScheduleRequest ?

tu as laissé request ici peut-être le changer en ScheduleRequest ?
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +477,4 @@
content:
application/x-www-form-urlencoded:
schema:
type: object
Maxime commented 2023-12-11 23:23:10 +01:00
Owner
Copy Link

ici tu as oublié de changer, c'est encore l'ancienne version où le prof n'as pas le choix de quand son cours est déplacé ni où.

ici tu as oublié de changer, c'est encore l'ancienne version où le prof n'as pas le choix de quand son cours est déplacé ni où.
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +500,4 @@
tags:
- Teacher
- Secretariat
- request
Maxime commented 2023-12-11 23:23:45 +01:00
Owner
Copy Link

de nouveau request -> ScheduleRequest ?

de nouveau request -> ScheduleRequest ?
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +513,4 @@
summary : Modify request
tags:
- Teacher
- request
Maxime commented 2023-12-11 23:24:04 +01:00
Owner
Copy Link

de nouveau request -> ScheduleRequest ?

de nouveau request -> ScheduleRequest ?
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +524,4 @@
properties:
id:
type: integer
requestType:
Maxime commented 2023-12-11 23:24:32 +01:00
Owner
Copy Link

ici aussi oubli de dernière version

ici aussi oubli de dernière version
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +536,4 @@
tags:
- Teacher
- Secretariat
- request
Maxime commented 2023-12-11 23:24:42 +01:00
Owner
Copy Link

de nouveau request -> ScheduleRequest ?

de nouveau request -> ScheduleRequest ?
tonitch commented 2023-12-11 23:54:36 +01:00
Owner
Copy Link

j'ai jamais vu un aussi gros copier-collé que depuis math podcast

j'ai jamais vu un aussi gros copier-collé que depuis math podcast
Wal marked this conversation as resolved
tonitch requested changes 2023-12-11 23:59:32 +01:00
tonitch left a comment
Owner
Copy Link

Franchement pas mal, j'ai fait beaucoup de commentaire mais c'est principalement des clarification sur la syntaxe. ça ne devrais pas être trop long à changer je pense.

Franchement pas mal, j'ai fait beaucoup de commentaire mais c'est principalement des clarification sur la syntaxe. ça ne devrais pas être trop long à changer je pense.
Documents/proto/general/openapi.yaml
@ -58,3 +58,3 @@
security:
- bearer: []
responses:
parameters:
tonitch commented 2023-12-11 23:37:51 +01:00
Owner
Copy Link

ce endpoint permet l'accés à "information about yourself" donc je ne vois pas comment il pourrais être paramètrisé par sa fonction ^^

ce endpoint permet l'accés à "information about yourself" donc je ne vois pas comment il pourrais être paramètrisé par sa fonction ^^
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +377,4 @@
summary: Create a new lesson
tags:
- Secretariat
- lesson
tonitch commented 2023-12-11 23:41:31 +01:00
Owner
Copy Link

Pour l'instant pour que ça fasse plus bô j'ai mis tout les tags en commençant avec une majuscule. c'est mineur mais ça serait cool ^^

Pour l'instant pour que ça fasse plus bô j'ai mis tout les tags en commençant avec une majuscule. c'est mineur mais ça serait cool ^^
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +465,4 @@
description: Success
'401':
$ref: '#/components/responses/UnauthorizedError'
/request:
tonitch commented 2023-12-11 23:46:14 +01:00
Owner
Copy Link

faut juste voir avec @LeoMoulin que ces endpoints ne soient pas en conflits ?

faut juste voir avec @LeoMoulin que ces endpoints ne soient pas en conflits ?
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml Outdated
@ -368,0 +487,4 @@
type: integer
responses:
'201':
description: Request created
tonitch commented 2023-12-11 23:50:10 +01:00
Owner
Copy Link

quand il y a un 'security' tag c'est toujours bien d'envisager l'erreur de permission:

        '401':
          $ref: '#/components/responses/UnauthorizedError'
quand il y a un 'security' tag c'est toujours bien d'envisager l'erreur de permission: ```yaml '401': $ref: '#/components/responses/UnauthorizedError' ```
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +522,4 @@
schema:
type: object
properties:
id:
tonitch commented 2023-12-11 23:51:43 +01:00
Owner
Copy Link

Je ne pense pas que tu puisse changer l'id d'un request.

Je ne pense pas que tu puisse changer l'id d'un request.
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +544,4 @@
/display:
post:
summary: Create a new display
tonitch commented 2023-12-11 23:55:48 +01:00
Owner
Copy Link

summary permet une courte description. il y'a également le tag description qui peut être pratique pour une plus grande description.

Dans ce cas ci ça pourrais peut être être utile ? display peut être ambigu ?

summary permet une courte description. il y'a également le tag `description` qui peut être pratique pour une plus grande description. Dans ce cas ci ça pourrais peut être être utile ? display peut être ambigu ?
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +555,4 @@
schema:
type: object
properties:
id:
tonitch commented 2023-12-11 23:56:13 +01:00
Owner
Copy Link

l'id ne pourrais pas

l'id ne pourrais pas
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -368,0 +600,4 @@
schema:
type: object
properties:
id:
tonitch commented 2023-12-11 23:57:05 +01:00
Owner
Copy Link

l'id ne pourrais pas être changé ici non plus.

l'id ne pourrais pas être changé ici non plus.
Wal marked this conversation as resolved
Documents/proto/general/openapi.yaml
@ -459,6 +705,80 @@ components:
"id": 42,
"courses": ['Math', 'Info']
}
Lesson :
tonitch commented 2023-12-11 23:58:51 +01:00
Owner
Copy Link

C'est sympa de faire des schema pour avoir une bonne description mais il ne sont jamais utilisé. pour les utiliser tu peux faire
$ref: <path/to/source>

par example pour utiliser Lesson tu peux faire $ref: '#/components/schemas/Lesson qui va être remplacé par tout ce qui est dans ce tag ci!

C'est sympa de faire des schema pour avoir une bonne description mais il ne sont jamais utilisé. pour les utiliser tu peux faire `$ref: <path/to/source>` par example pour utiliser Lesson tu peux faire `$ref: '#/components/schemas/Lesson` qui va être remplacé par tout ce qui est dans ce tag ci!
Wal marked this conversation as resolved
Wal added 1 commit 2023-12-12 11:39:20 +01:00
tonitch approved these changes 2023-12-13 14:59:51 +01:00
Maxime approved these changes 2023-12-13 15:00:22 +01:00
Maxime left a comment
Owner
Copy Link

appprobed

ssh-keygen -t rsa -b 4096 -C "your_email@example.com"

appprobed ssh-keygen -t rsa -b 4096 -C "your_email@example.com"
👀 1
Wal merged commit 9e299b39bd into master 2023-12-13 15:00:49 +01:00
tonitch deleted branch GestionHoraire 2023-12-13 16:59:37 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
LeoMoulin
tonitch
Maxime
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
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#6
No description provided.
Delete Branch "GestionHoraire"

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?