OCA / calendar

GNU Affero General Public License v3.0
24 stars 95 forks source link

[FIX] Avoid sending notifications when public user #60

Closed yajo closed 3 years ago

yajo commented 3 years ago

Public users usually don't have an email, so they hit this error.

No test added because anyways the tests in website_sale_resource_booking already fail without this patch, and it's hard to find a use case without it.

@Tecnativa TT31328

yajo commented 3 years ago

/ocabot merge patch

OCA-git-bot commented 3 years ago

On my way to merge this fine PR! Prepared branch 13.0-ocabot-merge-pr-60-by-Yajo-bump-patch, awaiting test results.

OCA-git-bot commented 3 years ago

Congratulations, your PR was merged at d3a0cb07b11a44b8333d7b0cb1d6b4f6b53dc480. Thanks a lot for contributing to OCA. ❤️