-
-
Notifications
You must be signed in to change notification settings - Fork 585
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][FIX] mail_debrand issues #939
Conversation
Hi @pedrobaeza, @joao-p-marques, |
775bebf
to
33687cb
Compare
Thanks! I see a duplicate PR has come in in the mean time. Do you want to check if it contains additional improvements that you want to take? #968 |
33687cb
to
7aace9d
Compare
…of type 'NoneType' has no len()
7aace9d
to
67f0465
Compare
@StefanRijnhart, incorporated the changes of #968 |
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.
Thanks!
This is a rough temporary fix while we wait for PR OCA#939 to be merged in upstream.
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at b4b1bd0. Thanks a lot for contributing to OCA. ❤️ |
Is this applicable to 16.0? |
Yes, but it's already picked in https://github.com/OCA/social/pull/1021/commits. |
This PR fixes the following issues: