[ADD] website: mail groups #6048
Conversation
auva-odoo
force-pushed
the
15.0-website-mailing-lists-auva
branch
2 times, most recently
from
5df13d0 to
bfe42bf
Compare
auva-odoo
force-pushed
the
15.0-website-mailing-lists-auva
branch
from
bfe42bf to
741bf69
Compare
toaa-odoo
force-pushed
the
15.0-website-mailing-lists-auva
branch
from
741bf69 to
f2c861e
Compare
toaa-odoo
left a comment
toaa-odoo
left a comment
There was a problem hiding this comment.
Hey @auva-odoo, great work! I only moved a line, the rest was all good :)
xpl-odoo
left a comment
xpl-odoo
left a comment
There was a problem hiding this comment.
Great job @auva-odoo ! Almost all comments are nitpicks, as usual ;)
Note: do we really want to have this page at the level of the Pages/Configuration etc. sections? It feels like we give this feature too much importance maybe?
task-2845181
auva-odoo
force-pushed
the
15.0-website-mailing-lists-auva
branch
from
f2c861e to
14621a3
Compare
|
Hey @xpl-odoo thank you for the review! I implemented all of the suggested changes, except the one regarding weather we really want to have this page at the level of the Pages/Configuration etc. sections? I would argue that although I see your point, it's the solution that makes the most sense to me at this time. We can discuss this if you want :-) |
No need to discuss this, we can @robodoo r+ :) |
|
@robodoo r+ |
task-2845181 closes #6048 Signed-off-by: Xavier Platteau (xpl) <xpl@odoo.com>
task-2845181