Skip to content

[17.0][FIX] portal_invitation_by_website: use website name in portal invitation email#1185

Merged
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-portal_invitation_by_website-website_name_in_email
May 19, 2026
Merged

[17.0][FIX] portal_invitation_by_website: use website name in portal invitation email#1185
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-portal_invitation_by_website-website_name_in_email

Conversation

@cristina-hidalgo-tecnativa
Copy link
Copy Markdown

Override the portal invitation email template to display the website name instead of the company name when a website is selected in the wizard.

Falls back to the company name when no website is set.

@Tecnativa TT61856
@pedrobaeza @pilarvargas-tecnativa

@OCA-git-bot OCA-git-bot added series:17.0 mod:portal_invitation_by_website Module portal_invitation_by_website labels May 18, 2026
@pedrobaeza pedrobaeza added this to the 17.0 milestone May 18, 2026
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not valid, as translations won't be available, being an XML-ID of another module.

One thing you can do is to intercept def ref(..) method to return a different template from this module when getting the reference of portal.mail_template_data_portal_welcome.

This is still not the best, as you have to re-translate the whole template, but at least it's translatable.

@cristina-hidalgo-tecnativa cristina-hidalgo-tecnativa force-pushed the 17.0-fix-portal_invitation_by_website-website_name_in_email branch from 5b8056e to 101221b Compare May 18, 2026 13:09
Comment thread portal_invitation_by_website/models/ir_model_data.py Outdated
@cristina-hidalgo-tecnativa cristina-hidalgo-tecnativa force-pushed the 17.0-fix-portal_invitation_by_website-website_name_in_email branch from 101221b to d7c5bc1 Compare May 18, 2026 14:08
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the template doesn't appear in translations?

@pilarvargas-tecnativa
Copy link
Copy Markdown
Contributor

Why the template doesn't appear in translations?

ping @cristina-hidalgo-tecnativa

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 17.0-fix-portal_invitation_by_website-website_name_in_email branch from d7c5bc1 to 5dbfe51 Compare May 19, 2026 07:44
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-1185-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d53f7ad into OCA:17.0 May 19, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-fix-portal_invitation_by_website-website_name_in_email branch May 19, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants