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

Table swipe_card_log : Ajout du beneficiary_Id ou du numéro du badge #509

Closed
SabineJa opened this issue Sep 19, 2022 · 7 comments · Fixed by #559
Closed

Table swipe_card_log : Ajout du beneficiary_Id ou du numéro du badge #509

SabineJa opened this issue Sep 19, 2022 · 7 comments · Fixed by #559

Comments

@SabineJa
Copy link

Dans la table swipe_card_log, les scan de badge sont aujourd’hui anonymes. On a juste le compteur.
Serait-il possible d’avoir - de manière optionnelle dans la table des paramètres - le beneficiary_Id ou le numéro du badge afin de savoir qui vient faire ses courses au magasin ?

@raphodn
Copy link
Member

raphodn commented Sep 20, 2022

Salut Sabine 👋

en effet on a fait le choix (à l'Elefan) de rendre ça anonyme.
mais on peut réflechir ensemble pour rajouter ce champ, et rendre son remplissage optionnel en fonction d'un paramètre.

est-ce que tu serais en mesure de proposer une PR ? de notre coté on est pas mal occupé par d'autres tâches, peu de chances que cela soit fait rapidement...

Tu fais partie de quelle coop ?

Merci !

@SabineJa
Copy link
Author

SabineJa commented Sep 21, 2022 via email

@raphodn raphodn linked a pull request Oct 27, 2022 that will close this issue
@raphodn
Copy link
Member

raphodn commented Oct 29, 2022

@SabineJa la PR a été mergée sur master ! on fera une release sometime ce weekend, mais vous pouvez déjà tester et nous faire un retour.

  • j'ai rajouté la possibilité de stocker l'id de swipecard. mais il faut mettre la bonne valeur au paramètre swipe_card_logging_anonymous: false
  • c'est possible qu'il y ait des bugs... je n'ai pas testé

@raphodn
Copy link
Member

raphodn commented Oct 31, 2022

et un commit additionnel qui répare certains bugs : 38375f1

il faut prendre la release v1.37.2

@SabineJa
Copy link
Author

SabineJa commented Oct 31, 2022 via email

@SabineJa
Copy link
Author

SabineJa commented Nov 5, 2022 via email

@raphodn
Copy link
Member

raphodn commented Nov 5, 2022

Top ! C'était une bonne idée l'affichage des log de temps, on s'en sert aussi du coup :)

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 a pull request may close this issue.

2 participants