-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
[IMP] auth_signup: Imporve odoo connect template to with new layout. #24576
[IMP] auth_signup: Imporve odoo connect template to with new layout. #24576
Conversation
0e83bce
to
73a77c0
Compare
73a77c0
to
e80351d
Compare
e80351d
to
8abbb6d
Compare
8abbb6d
to
be12a1a
Compare
be12a1a
to
255e7bc
Compare
255e7bc
to
9da6b58
Compare
LGTM |
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.
Technical review
</span> | ||
</td> | ||
<td valign="middle" align="right"> | ||
<img src="/web/static/src/img/logo.png" style="padding: 0px; margin: 0px; height: auto; width: 80px;"/> |
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.
Shouldn't we use company-specific logo ?
<field name="subject"><![CDATA[${object.create_uid.name} has invited you to connect to Odoo]]></field> | ||
<field name="body_html" type="html"> | ||
<table border="0" cellpadding="0" style="padding-top: 16px; background-color: #F1F1F1; font-family:Verdana, Arial,sans-serif; color: #454748; width: 100%; border-collapse:separate;"> | ||
% set website_url = object.env['ir.config_parameter'].get_param('web.base.url') |
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.
Whut ? Why are you using website url as "odoo domain" ?
<a href="${object.signup_url}" style="padding: 12px; color: #FFFFFF; border-color:#875A7B; text-decoration: none; display: inline-block; text-align: center; cursor: pointer; background-image: none; background-color: #875A7B; border: 1px solid #875A7B; border-radius:3px;">Accept invitation</a> | ||
<br/> | ||
<div style="margin-top:15px; margin-bottom:15px;"> | ||
Your Odoo domain is: <br/> |
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.
What is an Odoo domain ?
9da6b58
to
883443e
Compare
883443e
to
62bbf30
Compare
cdbda1e
to
3fe6d8b
Compare
3fe6d8b
to
a7b9f03
Compare
a7b9f03
to
92d0325
Compare
92d0325
to
344eefd
Compare
344eefd
to
aa54457
Compare
Even completely valid simple email like alfred+toto@example.com was not considered as valid. As email may contain a lot of characters and strange stuff let us be less strict on validation. This commit is related to task ID 1839609. Closes odoo#24576.
Task: https://www.odoo.com/web?#id=1839609&view_type=form&model=project.task&action=333&active_id=131&menu_id=4720
PAD: https://pad.odoo.com/p/r.0660d07aa08b13c7da20d84d941a826d