[ADD] beesdoo_shift_website_unsubscribe: add module#305
[ADD] beesdoo_shift_website_unsubscribe: add module#305
Conversation
|
TODO: retarget on 12.0 once add-polln-shift is merged |
a4ee9c7 to
7db0a90
Compare
dd99a36 to
d74b30c
Compare
robinkeunen
left a comment
There was a problem hiding this comment.
comment + move to beesdoo_shift
|
|
||
| from werkzeug.exceptions import Forbidden | ||
|
|
||
| class WebsiteMacavracShiftController(http.Controller): |
|
|
||
| <template id="my_shift" name="Shifts for Irregular Workers" | ||
| inherit_id="beesdoo_website_shift.my_shift_next_shifts"> | ||
| <xpath expr="//t[@t-esc='shift.task_type_id.name']" position="replace"> |
There was a problem hiding this comment.
Instead of replace, could you rather
- submit a PR to beesdoo_shift to replace
<t t-esc="shift.task_type_id.name" />by<t t-esc="shift.task_template_id.name or shift.task_type_id.name" /> - than adding the button with a
<xpath ... position="after">
| </button> | ||
| </xpath> | ||
| <xpath expr="//div[@t-if='shift.super_coop_id.name']/.." position="after"> | ||
| <t t-foreach="subscribed_shifts" t-as="shift"> |
There was a problem hiding this comment.
TIL modals for confirmation popups: nice 🙏 @remytms fyi
Add module to allow to unsubscribe from the portal X hour before the shift starts
d74b30c to
3e2c14a
Compare
Codecov Report
@@ Coverage Diff @@
## 12.0 #305 +/- ##
==========================================
- Coverage 68.53% 68.37% -0.17%
==========================================
Files 107 107
Lines 3366 3377 +11
Branches 589 591 +2
==========================================
+ Hits 2307 2309 +2
- Misses 951 960 +9
Partials 108 108
Continue to review full report at Codecov.
|
|
I rater prefer that this will be merge into the |
|
|
Virg et Pol : intéret probable de l'intercoop, Virg sonde @remytms Peux-tu estimer environ le temps nécessaire ici pour séparer dans un nouveau module et clore les reviews ? |
|
@robinkeunen @vdewulf Pour suivi. |
|
@polchampion Matteo a fait cet ajout dans #395 et c'est en test. |
|
@vdewulf Pour info, testé et documenté |
|
@polchampion super, merci :) |
|
@polchampion une autre petite question : est-ce qu'il y a une limite de temps pour pouvoir se désinscrire qui est codée qqpart (par exemple impossible de se désinscrire 24h à l'avance via l'interface) ? |
|
@polchampion ah purée je suis trop à la masse, j'ai vu que c'était bien le cas et documenté ;) abort le message précédent |
@vdewulf Oui je fais ça cette aprèm. Et malheureusement, Nimbus capture n'aide pas vraiment à faire des jolis arcs de cercle, c'est mission impossible à la main donc j'ai arrêté d'essayer... |
|
Unsubscription has been added to beesdoo_shift -> #309 |
abc40d1 to
fa7679b
Compare
Add module to allow to unsubscribe from the portal
X hour before the shift starts