-
-
Notifications
You must be signed in to change notification settings - Fork 612
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
[15.0][ADD] mail_forward: new module to forward messages from the chatter #1455
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
f1cce86
to
b61ff47
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on runboat and working. Is there any other FA icon that represents better the forwarding (double arrow?)
It can be done with social/mail_tracking/static/src/components/discuss/discuss.xml Lines 78 to 100 in 92640d7
|
I meant stacking your original arrows. Altough the fa-forward could fit |
I think stacking the arrow seems better |
b61ff47
to
2f3ae9a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge nobump
On my way to merge this fine PR! |
Congratulations, your PR was merged at 48c6002. Thanks a lot for contributing to OCA. ❤️ |
TT51020
@Tecnativa @pedrobaeza @chienandalu @victoralmau could you please review this