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]signup_error caused by missing params #135

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

digitalsatori
Copy link

@digitalsatori digitalsatori commented Jul 26, 2022

When signing up new user from frontend, the below error is prompted, the same issue mentioned in #126 :

Traceback (most recent call last):
File "/odoo14/custom/saas/auth_signup_verify_email/controllers/main.py", line 59, in passwordless_signup
sudo_users.signup(values, qcontext.get("token"))
File "/odoo14/custom/saas/saas_apps_signup/models/res_users.py", line 31, in signup
return self.signup_to_try(values, *args, **kwargs)
File "/odoo14/custom/saas/saas_apps_signup/models/res_users.py", line 43, in signup_to_try
operator_id = int(values.pop("operator_id"))
KeyError: 'operator_id'```

@ilmir-k ilmir-k requested a review from em230418 July 26, 2022 11:24
@em230418
Copy link
Member

em230418 commented Aug 2, 2022

Hi, @digitalsatori!
Could you please show steps to reproduce this error and it's traceback?

@digitalsatori
Copy link
Author

Hi, @digitalsatori!
Could you please show steps to reproduce this error and it's traceback?

I updated the description.Actually I just find out that this issue has been correctly fixed in your branch 15.0, maybe we can just back ported here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants