Liste Saisie Notes #178

Closed
opened 2021-11-03 08:56:47 +01:00 by pascal.bouron · 8 comments
Contributor

Bonjour,
Afin de pouvoir exploiter plus facilement le "Lister toutes les saisies de notes" , est il possible d'afficher l'id de l'étudiant ?
/Scolarite/Notes/formsemestre_list_saisies_notes?formsemestre_id=1

(Et éventuellement pour la lisibilité, le nom de l'enseignant en plus de son id )

Bonjour, Afin de pouvoir exploiter plus facilement le "Lister toutes les saisies de notes" , est il possible d'afficher l'id de l'étudiant ? **/Scolarite/Notes/formsemestre_list_saisies_notes?formsemestre_id=1** (Et éventuellement pour la lisibilité, le nom de l'enseignant en plus de son id )
Author
Contributor

Bonjour,
Je me suis lancé à regarder le code.
Je propose la modification suivante du fichier sco_undo_notes.py

Dans formsemestre_list_saisies_notes

  1. Ligne 181, ajout de i.id as etudid, dans le SELECT
  2. Ligne 195, ajout de "etudid", dans le columns_ids
  3. Ligne 205, ajout de "etudid": "EID", dans le titles

Cela permettrait de cloturer ce ticket.

Bonjour, Je me suis lancé à regarder le code. Je propose la modification suivante du fichier **sco_undo_notes.py** Dans **formsemestre_list_saisies_notes** 1. Ligne 181, ajout de **i.id as etudid,** dans le SELECT 2. Ligne 195, ajout de **"etudid",** dans le columns_ids 3. Ligne 205, ajout de **"etudid": "EID",** dans le titles Cela permettrait de cloturer ce ticket.
Owner

merci. Tu n'as pas de fork, donc pas de PR. Cela complique l'intégration mais je le fais quand même (pour cette fois seulement ;-) )

merci. Tu n'as pas de fork, donc pas de PR. Cela complique l'intégration mais je le fais quand même (pour cette fois seulement ;-) )
Owner

merci. Tu n'as pas de fork, donc pas de PR. Cela complique l'intégration mais je le fais quand même (pour cette fois seulement ;-) )

merci. Tu n'as pas de fork, donc pas de PR. Cela complique l'intégration mais je le fais quand même (pour cette fois seulement ;-) )
Author
Contributor

Merci ! Je vais essayer de faire mieux la prochaine fois.

Merci ! Je vais essayer de faire mieux la prochaine fois.
Owner

A quoi sert cette modif ?

Afficher un id interne sur une page destinée à un humain n'est pas souhaitable.
Les id internes ne devraient servir qu'à l'API.

Est-ce que le NIP ou l'INE ne seraient pas plus utiles ?

Eventuellement seulement dans la version Excel ?

A quoi sert cette modif ? Afficher un id interne sur une page destinée à un humain n'est pas souhaitable. Les id internes ne devraient servir qu'à l'API. Est-ce que le NIP ou l'INE ne seraient pas plus utiles ? Eventuellement seulement dans la version Excel ?
Owner

donc on met le NIP dans une nouvelel colonne.

donc on met le NIP dans une nouvelel colonne.
Author
Contributor

Oui, très bien.

Oui, très bien.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 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: ScoDoc/ScoDoc#178
No description provided.