update des values de quasiment tout #19
Conversation
|
et aussi @Vexcited tu penses c est quand la prochaine release dcp ? sa mérite carrément être la 1.8 si c est merge |
JyhuKo
changed the title
|
c est bon j'ai tout fini j espere que c est pas mal et que j ai pas fais n importe quoi |
|
pas de review ? :( |
src/core/request.ts
Outdated
|
|
||
| public addVersionURL(): Request { | ||
| this.url.searchParams.set("v", "6.17.0"); | ||
| this.url.searchParams.set("v", "4.69.0"); |
There was a problem hiding this comment.
C'est la dernière version de l'application mobile ?
J'ai besoin d'une personne qui a des accès EcoleDirecte élève qui review vu tout les changements effectués. |
|
ah oui mb , mais c est pas possible pour l instant de push une release pour le coef des matieres ? |
camarm-dev
left a comment
camarm-dev
left a comment
There was a problem hiding this comment.
J'ai jeté un petit coup d'oeuil à ta PR c'est pas mal tu as rajouté beaucoup de propriétés aux modèles ! Pour les exemples c'est bien mieux aussi, tu pourrais peut-être utiliser la date du jour si aucune date est spécifiée ?
Par contre j'ai remarqué que t'avais rajouté pas mal d'espaces à la fin des lignes ou sur certains commentaires qui ont donc une indentation bizarre.
Faudrait que tu regardes aussi si le linter et typecheck passe...
PS: J'ai pas (encore) testé j'ai juste lu assez rapidement 😄
| const Date = prompt("Enter date in the format YYYY-MM-DD:"); | ||
| const assignmentsDate = Date ? Date : '2025-01-14'; |
There was a problem hiding this comment.
| const Date = prompt("Enter date in the format YYYY-MM-DD:"); | |
| const assignmentsDate = Date ? Date : '2025-01-14'; | |
| const date = prompt("Enter date in the format YYYY-MM-DD:"); | |
| const assignmentsDate = date || '2025-01-14'; |
C'est mieux de ne pas utiliser le nom d'un objet qui existe (Date). Aussi la condition ternaire peut être simplifiée 😄
| const StartDate = prompt("Enter date in the format YYYY-MM-DD:"); | ||
| const CheckStartDate = StartDate ? new Date(StartDate) : new Date('2025-01-14'); |
There was a problem hiding this comment.
| const StartDate = prompt("Enter date in the format YYYY-MM-DD:"); | |
| const CheckStartDate = StartDate ? new Date(StartDate) : new Date('2025-01-14'); | |
| const startDate = prompt("Enter date in the format YYYY-MM-DD:"); | |
| const checkStartDate = startDate ? new Date(startDate) : new Date('2025-01-14'); |
Juste pour respecter les conventions de nommage du projet (et de typescript).
| const CheckStartDate = StartDate ? new Date(StartDate) : new Date('2025-01-14'); | ||
|
|
||
| const timetable = await studentTimetable(session, account, startDate); | ||
| const timetable = await studentTimetable(session, account, CheckStartDate); |
There was a problem hiding this comment.
| const timetable = await studentTimetable(session, account, CheckStartDate); | |
| const timetable = await studentTimetable(session, account, checkStartDate); |
| import type { APIReceivedMessageType } from "./message"; | ||
|
|
||
| export type APIReceivedMessagesList = Readonly<{ | ||
| export type APIReceivedMesagesList = Readonly<{ |
There was a problem hiding this comment.
| export type APIReceivedMesagesList = Readonly<{ | |
| export type APIReceivedMessagesList = Readonly<{ |
Erreur d'orthographe
| classSize?: number; // effectif | ||
| rank?: number; //rang | ||
| teachers?: { id: number; name: string }[]; // profs | ||
| comments?: string; // appreciations |
There was a problem hiding this comment.
Commentaires inutiles ici...
3 participants
ajout de nouvelles keys et values
amelioration de l'obtention des messages en changeant le systeme d'année scolaire
changement d un truc qui avait un s en trop