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

12.0 add shift unsubscribe #395

Closed
wants to merge 4 commits into from
Closed

Conversation

MatteoLdx
Copy link
Collaborator

Description

Enable unsubscription for irregular workers and for compensation shifts.

Checklist before approval

  • Tests are present (or not needed).
  • Credits/copyright have been changed correctly.
  • (If a new module) Moving this to OCA has been considered.

@MatteoLdx
Copy link
Collaborator Author

Extension of this PR : #377

@MatteoLdx MatteoLdx force-pushed the 12.0-add-shift-unsubscribe branch from c10ec0e to 82b6df8 Compare May 30, 2022 08:48
@codecov-commenter
Copy link

Codecov Report

Merging #395 (0b7b1a1) into 12.0 (365d53e) will decrease coverage by 0.31%.
The diff coverage is 31.03%.

@@            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              
Impacted Files Coverage Δ
beesdoo_shift/models/task.py 73.00% <18.18%> (-3.97%) ⬇️
beesdoo_website_shift/controllers/main.py 17.11% <21.42%> (+0.34%) ⬆️
beesdoo_website_shift/models/res_config.py 100.00% <100.00%> (ø)
beesdoo_website_shift/models/website.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 365d53e...0b7b1a1. Read the comment docs.

@MatteoLdx MatteoLdx marked this pull request as ready for review June 7, 2022 08:49
@polchampion
Copy link
Collaborator

@MatteoLdx Sucessfully tested on lepedalo-test.

  • Irregular can unsubscribe : OK
  • Regular can unsubscribe from compensation : OK
  • min_hours_to_unsubscribe can be adjusted in system parameters (impossible to unsubscribe within this interval before the shift): OK
  • enable_unsubscribe_compensation in website parameters (if not, unsubscribe not available): OK
  • irregular_enable_unsubscribe in website parameters (if not, unsubscribe not available): OK

One suggestion : add min_hours_to_unsubscribe to website_shift settings

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 min_hours_to_unsubscribe for subscription that corresponds to attendance_sheet_generation_interval ?

@polchampion
Copy link
Collaborator

@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.

@MatteoLdx
Copy link
Collaborator Author

@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

@polchampion
Copy link
Collaborator

@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.

@MatteoLdx
Copy link
Collaborator Author

Merged with #309

@MatteoLdx MatteoLdx closed this Jun 20, 2022
@MatteoLdx MatteoLdx deleted the 12.0-add-shift-unsubscribe branch July 15, 2022 08:59
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