[ADD] beesdoo_shift_website_unsubscribe: add module #305
Conversation
|
TODO: retarget on 12.0 once add-polln-shift is merged |
robinkeunen
force-pushed
the
12.0-add-polln-shift
branch
from
a4ee9c7 to
7db0a90
Compare
tfrancoi
force-pushed
the
12.0-unsubscribe
branch
from
dd99a36 to
d74b30c
Compare
|
|
||
| from werkzeug.exceptions import Forbidden | ||
|
|
||
| class WebsiteMacavracShiftController(http.Controller): |
| @@ -124,6 +124,58 @@ | |||
| <xpath expr="//p[hasclass('h_irreg_autoextension')]" position="replace" /> | |||
| </template> | |||
|
|
|||
| <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
robinkeunen
force-pushed
the
12.0-unsubscribe
branch
from
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 |
carmenbianca
force-pushed
the
12.0
branch
2 times, most recently
from
abc40d1 to
fa7679b
Compare
Add module to allow to unsubscribe from the portal
X hour before the shift starts