[17.0][ADD]mail_notify_employee_leave#156
Conversation
GuillermoNforgeflow
changed the title
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
2 times, most recently
from
a07e0f3 to
ed43a5c
Compare
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
2 times, most recently
from
acdda41 to
e90ac57
Compare
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
4 times, most recently
from
63cde2e to
aeced1b
Compare
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
2 times, most recently
from
9d30768 to
12ac186
Compare
GuillemCForgeFlow
left a comment
GuillemCForgeFlow
left a comment
There was a problem hiding this comment.
it doesn't seem to work for me:
- I create a time off for marc demo starting from today (24/4) and finishing next tuesday.
- I validate the time off
- I send a message to marc demo
- I don't receive any reponse
am i doing something wrong?
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
from
12ac186 to
c75c785
Compare
OCA-git-bot
added
series:17.0
mod:mail_notify_employee_leave
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
from
c75c785 to
74aa648
Compare
GuillemCForgeFlow
left a comment
GuillemCForgeFlow
left a comment
There was a problem hiding this comment.
lgtm 👍🏿 functional and code review
|
Hi @pedrobaeza, could we perhaps merge this one? |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-156-by-pedrobaeza-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_noify_employee_leave
branch
from
74aa648 to
34e2caf
Compare
|
Hi @pedrobaeza, could we merge this one again? |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at ef89f0b. Thanks a lot for contributing to OCA. ❤️ |
5 participants
This module automatically notifies users when they mention or assign a colleague who is out of office. The notification includes the expected return date of the absent user and is sent only once per day per user.