Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Add attendance display #577

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jnguiot
Copy link
Member

@jnguiot jnguiot commented Nov 1, 2022

Voici ma proposition pour commencer à s'attaquer au problème des personnes qui se désinscrivent ou ne viennent pas aux sorties: dans un premier temps, mettre mieux en évidences les "incidents", notamment en attribuant des notes aux utilisateurs.

Les critères de notations pourront être modifiés par la suite, pour le moment j'ai mis:

    * A: No infamous status
    * B: Less than 5% infamous status.
    * C: One infamous status
    * D: Two infamous status
    * E: others

Les restrictions pourront être mises par la suite en place sur ce système.

La notation et les absences sont aussi beaucoup plus visibles sur le profil d'un utilisateur:
Screenshot 2022-11-01 at 21-37-39 Collectives

Et sur les évènements:
Screenshot 2022-11-01 at 21-37-51 Collectives

A noter que seuls les collectives et les formations comptent pour la rédaction du rapport et de la note.

@jnguiot jnguiot requested review from gdaviet and fberu November 1, 2022 21:00
@fberu
Copy link
Contributor

fberu commented Nov 2, 2022

@jnguiot merci pour la proposition.
De mon côté j'avais imaginé des "badges":

J'y propose notamment des "blocages" et impossibilité pour les personnes de s'inscrire dans certains cas

@jnguiot
Copy link
Member Author

jnguiot commented Nov 2, 2022

Je ne sais pas ce que tu en penses, mais je préfère une note au lieu d'un badge. Cependant, sur ce système de note, on pourra construire les badges. En tout cas, oui, un jour, il faudra faire les blocages.

Copy link
Contributor

@gdaviet gdaviet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sur le plan technique ça me parait bien, je laisse l'aspect politique à François :)

attendance = db.Column(db.Boolean(), nullable=False, default=True)
""" True if this event acts on the users attendance reports.

:type: Boolean"""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"track_attendance" peut être ? Histoire d'avoir un nom un peu plus explicite

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En effet, je vais modifier

cls.JustifiedAbsentee: "#005aa0",
cls.UnJustifiedAbsentee: "#ffad8f",
cls.ToBeDeleted: "",
cls.Waiting: "#e6f4f1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si possible ce serait pas mal de définir les couleurs au niveau des templates Jinja2

:rtype: list
"""
return [
cls.SelfUnregistered,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour moi SelfUnregistered ne devrait pas être infamous par défaut, je pense qu'il y a beaucoup d'encadrants qui ne mettent pas à jour le statut pour une absence justifiée. Si elle est vraiment non justifié ils peuvent changer le statut en UnjustifiedAbsentee a posteriori. Mieux vaut avoir des faux négatifs que des faux positifs à mon avis.

if infamous_strikes == 1:
return "C"
if infamous_strikes == 2:
return "D"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quelle est la motivation pour définir parfois en pourcentage et parfois en nombre absolu ? Tu as un exemple en tête ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants