[19.0][MIG] portal_invitation_by_website: Migration to 19.0 - #1216
Conversation
Currently translated at 100.0% (6 of 6 strings) Translation: website-17.0/website-17.0-portal_invitation_by_website Translate-URL: https://translation.odoo-community.org/projects/website-17-0/website-17-0-portal_invitation_by_website/it/
…tation footer Use the website domain in the portal invitation email footer instead of always using the company website. A single company can manage multiple websites, so using the company website may display the default website URL instead of the website from which the portal invitation is being sent. TT61856
Currently translated at 100.0% (11 of 11 strings) Translation: website-17.0/website-17.0-portal_invitation_by_website Translate-URL: https://translation.odoo-community.org/projects/website-17-0/website-17-0-portal_invitation_by_website/it/
OCA-git-bot
added
series:19.0
mod:portal_invitation_by_website
dc9fac7 to
5b37819
Compare
|
/ocabot migration portal_invitation_by_website Isn't the new core feature "Share users between websites" (unchecking it) aiding with this showing the possible websites in the invitation wizard, or this is still needed? |
|
The "Shared Customer Accounts" setting (website.specific_user_account) doesn't cover this. It only acts on the frontend signup: res.users._signup_create_user() stamps website_id = current website when a visitor registers from that website (and only when there's a request). What core adds to the wizard (https://github.com/odoo/odoo/blob/638f9477ee7a6c301e79e65e33816f700520f535/addons/website/wizard/portal_wizard.py) is just reading partner_id.website_id to compute if there is any duplicated email, but there is no website selector anywhere in the wizard. So with plain core you have to open each contact form and set its Website field before launching the wizard; once that's set, the signup URL already points to the right domain thanks to get_base_url(). 
And I think that the powerfull feature that this module provides is choosing the website from the wizard itself. 
|
|
Perfect, thanks for the explanation. Then it makes sense to continue having this module. |
| "website": "https://github.com/OCA/website", | ||
| "license": "AGPL-3", | ||
| "category": "Website", | ||
| "depends": ["auth_signup", "portal", "website"], |
There was a problem hiding this comment.
| "depends": ["auth_signup", "portal", "website"], | |
| "depends": ["portal", "website"], |
5b37819 to
de64aea
Compare
|
Change done @pilarvargas-tecnativa |
cristina-hidalgo-tecnativa
left a comment
cristina-hidalgo-tecnativa
left a comment
There was a problem hiding this comment.
Code reviewed and tested locally and on runbot, LGTM!
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at f74af4a. Thanks a lot for contributing to OCA. ❤️ |
6 participants
cc @Tecnativa TT64114
ping @cristina-hidalgo-tecnativa @pilarvargas-tecnativa