-
-
Notifications
You must be signed in to change notification settings - Fork 606
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
[FIX] mail_debrand: name of overridden method #209
[FIX] mail_debrand: name of overridden method #209
Conversation
200b7b5
to
ab479eb
Compare
There was a previous attempt at #176, but it misses the tests fix. |
Please change module version number |
ab479eb
to
0360e94
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.
Indeed the code fix is quite obvious, but please make Travis happy 😊
0360e94
to
01acc44
Compare
@yajo I fixed the lint error related to this change but the odoo/odoo build fails just like on the main branch (and I don't know how to fix timeouts in http tests anyway) |
Agh, same problem everywhere out of Odoo itself. There's a general problem with Travis + wkhtmltopdf, which reproduces the same way. Don't know if this one is the same. |
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.
Besides, there's an error in another addon.
OK, not blocking then
As per https://github.com/odoo/odoo/blob/8.0/addons/mail/mail_followers.py#L138.
@yajo, this one is a bit of a mystery. I could not find any trace of this method name with the preceding underscore.