WIP: Affiche un warning si trop d'alcool consommé

lstephan commented

2020-06-10 22:36:21 +02:00

(Migrated from git.eleves.ens.fr)

Au delà de 4 UA (1 UA = 10g d'alcool pur) consommées en une soirée (hors LIQ), un warning (discret mais visible) apparaît sur K-Psul pour prévenir l'équipe.

Fixes #252

Au delà de 4 UA (1 UA = 10g d'alcool pur) consommées en une soirée (hors LIQ), un warning (discret mais visible) apparaît sur K-Psul pour prévenir l'équipe. Fixes #252

mpepin commented

2020-06-13 18:59:37 +02:00

(Migrated from git.eleves.ens.fr)

J'ai pas mal buggué sur le mot "unité", je pensais que tu parlais d'unité alcool (même si ce n'était pas très logique).

On peut peut-être trouver un nom moins ambigu ? "Volume d'une conso (en cL)" par exemple ?

J'ai pas mal buggué sur le mot "unité", je pensais que tu parlais d'unité alcool (même si ce n'était pas très logique). On peut peut-être trouver un nom moins ambigu ? "Volume d'une conso (en cL)" par exemple ?

mpepin commented

2020-06-13 19:06:08 +02:00

(Migrated from git.eleves.ens.fr)

Je suppose que XX = 21 ?

mpepin commented

2020-06-13 19:12:40 +02:00

(Migrated from git.eleves.ens.fr)

D'ailleurs ce 21 est hardcodé à plusieurs endroits, ça serait bien de le mettre dans une variable. Ça n'a pas besoin d'être configurable par la K-Fêt mais au moins qu'on puisse la changer facilement nous.

mpepin commented

2020-06-13 19:15:00 +02:00

(Migrated from git.eleves.ens.fr)

La K-Fêt a été consultée sur cette idée de ne compter les consos que sur la tranche "21h → 6h" ? Si ça n'a pas été fait, je confirmerais bien que ça correspond à leurs besoins avant de merge.

tbastian commented

2020-06-13 19:29:32 +02:00

(Migrated from git.eleves.ens.fr)

J'ai vu passer l'issue au pif sur Merle. Est-ce que ça serait pas carrément pertinent d'avoir ça tout le temps (hors soirées aussi) ? Genre si lors des X dernières heures, la personne a consommé plus de Y, warning ?

lstephan commented

2020-06-13 20:52:16 +02:00

(Migrated from git.eleves.ens.fr)

Ça serait possiblement pas con, ça m'éviterait de m'emmerder avec ces ******* de fuseaux horaires 😠

Ça serait possiblement pas con, ça m'éviterait de m'emmerder avec ces ******* de fuseaux horaires :angry:

mpepin commented

2020-06-13 21:29:00 +02:00

(Migrated from git.eleves.ens.fr)

Ça me semble mieux aussi 👍

Je crois qu'il y avait une objection du type "en journée les gens prennent des consos à plusieurs sur le même compte". Mais en fait y'a pas de raison que ce soit moins le cas en soirée donc je dirais osef.

Ça me semble mieux aussi :+1: Je crois qu'il y avait une objection du type "en journée les gens prennent des consos à plusieurs sur le même compte". Mais en fait y'a pas de raison que ce soit moins le cas en soirée donc je dirais osef.

mpepin commented

2020-06-13 21:58:40 +02:00

(Migrated from git.eleves.ens.fr)

Je pense qu'il faut afficher le degré et le volume des différents items sur la page /k-fet/articles/ sinon ça va être difficile pour l'équipe de suivre pour quels items iels ont déjà rentré ces infos ou pas.

Je pense qu'il faut afficher le degré et le volume des différents items sur la page `/k-fet/articles/` sinon ça va être difficile pour l'équipe de suivre pour quels items iels ont déjà rentré ces infos ou pas.

mpepin commented

2020-06-13 22:00:34 +02:00

(Migrated from git.eleves.ens.fr)

Je mettrais bien un message plus verbeux ici, par exemple "N unités dans la soirée" ou "N unités dans les 6 dernières heures" (selon la méthode de calcul choisie)

lstephan commented

2020-06-26 17:44:16 +02:00

(Migrated from git.eleves.ens.fr)

Après check auprès de la régence K-Fêt, il s'avère que la demande n'est plus d'actualité (enfin, qu'elle devrait passer par une réu au préalable)... Je passe la MR en wip, je changerai ptet 2-3 trucs pour virer ces histoires de fuseaux horaires à la con quand même =)

lstephan commented

2020-06-26 17:44:26 +02:00

(Migrated from git.eleves.ens.fr)

marked as a Work In Progress

marked as a **Work In Progress**

lstephan commented

2020-06-26 17:44:26 +02:00

(Migrated from git.eleves.ens.fr)

changed the description

mpepin commented

2020-06-28 23:14:36 +02:00

(Migrated from git.eleves.ens.fr)

Ça ne coûte rien d'avoir le logo, c'est pas très intrusif quand même…

This pull request has changes conflicting with the target branch.

kfet/static/kfet/js/account.js
kfet/views.py

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin Aufinal/alcoolémie:Aufinal/alcoolémie

git checkout Aufinal/alcoolémie

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master

git merge --no-ff Aufinal/alcoolémie

git checkout Aufinal/alcoolémie

git rebase master

git checkout master

git merge --ff-only Aufinal/alcoolémie

git checkout Aufinal/alcoolémie

git rebase master

git checkout master

git merge --no-ff Aufinal/alcoolémie

git checkout master

git merge --squash Aufinal/alcoolémie

git checkout master

git merge --ff-only Aufinal/alcoolémie

git checkout master

git merge Aufinal/alcoolémie

git push origin master

WIP: Affiche un warning si trop d'alcool consommé #727

Checkout

Merge