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

Compte épargne : affichage basique #769

Merged
merged 2 commits into from
Mar 2, 2023
Merged

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Feb 26, 2023

Quoi ?

Modifications apportées :

  • afficher le compteur épargne dans le dashboard du membre
  • afficher les événements du compte épargne dans l'onglet "Compteur de temps" (en bleu) : coté membre et coté admin
  • onglet "compteur de temps" : nouvelle colonne "Créneau" pour simplifier la colonne "Motif"

Capture d'écran

Page Image
Membre > dashboard Screenshot from 2023-02-26 23-35-18
Membre > Gérer mon compte > Compteur de temps (cas d'un créneau validé, puis d'un créneau annulé) Screenshot from 2023-03-02 17-13-41

Info supplémentaire

J'ai dû changer l'ordering des timeLogs, car j'ai l'impression que Doctrine stock les DateTime sans les miliseconds ?? du coup impossible de les ordonner par date lorsque les 3 time logs sont créés quasi en même temps (cf doctrine/dbal#2873)

@raphodn raphodn force-pushed the raphodn/time-log-saving-display branch from 4caa923 to ef5b7e6 Compare February 26, 2023 22:23
@raphodn raphodn changed the base branch from raphodn/time-log-saving-events to raphodn/time-log-saving-feature February 26, 2023 22:23
@raphodn raphodn linked an issue Feb 26, 2023 that may be closed by this pull request
@raphodn raphodn changed the title Compte épargne : affichage Compte épargne : affichage basique Feb 26, 2023
@raphodn raphodn marked this pull request as ready for review February 26, 2023 23:10
@raphodn raphodn self-assigned this Feb 26, 2023
@petitalb
Copy link
Collaborator

Pour la liste des logs de temps, je me demande s’il ne faudrait pas faire deux affichages distinct ?

Comme à la banque, tu as deux comptes ?

Base automatically changed from raphodn/time-log-saving-feature to master February 27, 2023 17:54
@raphodn raphodn force-pushed the raphodn/time-log-saving-display branch from c05586c to 3a4aa49 Compare March 2, 2023 15:43
@raphodn raphodn merged commit 5c4e602 into master Mar 2, 2023
@raphodn raphodn deleted the raphodn/time-log-saving-display branch March 2, 2023 16:16
quot17 pushed a commit to quot17/gestion-compte that referenced this pull request Mar 28, 2023
* Display time-saving in member time_logs

* Display time-saving in dashboard
quot17 pushed a commit to quot17/gestion-compte that referenced this pull request Mar 28, 2023
@raphodn
Copy link
Member Author

raphodn commented Jul 4, 2023

Quelques fixes sur l'affichage (ou pas) des détail du créneau des timelog : b582df4 & 0a7ff4d

Il existait par exemple un cas particulier : si le créneau était annulé, et ensuite re-réservé par un autre membre, le nom de cet autre membre apparaissait dans le compteur temps du premier membre..

OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
* Display time-saving in member time_logs

* Display time-saving in dashboard
OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
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.

Création d'un deuxième compteur "épargne"
2 participants