-
Notifications
You must be signed in to change notification settings - Fork 41
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
12.0 add shift unsubscribe #395
Conversation
Extension of this PR : #377 |
c10ec0e
to
82b6df8
Compare
Codecov Report
@@ Coverage Diff @@
## 12.0 #395 +/- ##
==========================================
- Coverage 69.19% 68.87% -0.32%
==========================================
Files 109 109
Lines 3447 3476 +29
Branches 604 608 +4
==========================================
+ Hits 2385 2394 +9
- Misses 948 968 +20
Partials 114 114
Continue to review full report at Codecov.
|
@MatteoLdx Sucessfully tested on lepedalo-test.
One suggestion : add One question : I thought it was impossible to subscribe to a shift after the attendance sheet is generated, but I might be wrong. Is there something similar to |
@MatteoLdx Autre suggestion : ça n'est pas une foncitonnalité voulue, mais par cohérence ça aurait du sens que les réguliers puissent s'inscrire à des compensations manuellement s'ils peuvent s'en désinscrire. C'est pas vraiment une demande puisque ce n'est pas voulu, mais c'est peut-être utile de le noter quelque part. |
C'est déjà fait ! Tout est rassemblé ici : #309 |
@MatteoLdx Ah super, sorry j'ai pas du tout de vue sur ce que tu as développé ! Hâte de voir tout ça demain. |
Merged with #309 |
Description
Enable unsubscription for irregular workers and for compensation shifts.
Checklist before approval