Skip to content
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] add current website to context #802

Conversation

@rubenseijas
Copy link

commented Aug 8, 2019

If the portal wizard is used in a company with multi website the current website is needed in context to send the correct signup_url to the user becuase if not the wizard can send a signup_url for a incorrect website.
The only requisite is that the user who is sending the invitation have to select the correct website in the backend selector bar because is the website passed into the context

for wizard_user in self.sudo().with_context(active_test=False, website_id=self.env.user.backend_website_id.id):
group_portal = wizard_user.wizard_id.portal_id
if not group_portal.is_portal:
raise UserError(_('Group %s is not a portal') % group_portal.name)

This comment has been minimized.

Copy link
@hound

hound bot Aug 8, 2019

undefined name '_'

@yelizariev

This comment has been minimized.

Copy link
Member

commented Aug 9, 2019

We'll check the update. What I can say now is that the updates has be done as a separate module, because ir_config_parameter_multi_multi_company doesn't depend on portal and cannot depend on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.