-
-
Notifications
You must be signed in to change notification settings - Fork 115
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
[15.0][ADD] survey_partner_representative: New module #105
[15.0][ADD] survey_partner_representative: New module #105
Conversation
It seems a temporary deb file has slipped into the PR. |
e0b540a
to
27748ce
Compare
Oh, weird, maybe it has to do with the new fragments conversion |
e363e02
to
643b579
Compare
if self.allow_partner_representing: | ||
return super( | ||
SurveySurvey, self.with_context(survey_partner_representative=True) | ||
)._create_answer(*args, **kwargs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This way is more compact and readable:
if self.allow_partner_representing: | |
return super( | |
SurveySurvey, self.with_context(survey_partner_representative=True) | |
)._create_answer(*args, **kwargs) | |
if self.allow_partner_representing: | |
self = self.with_context(survey_partner_representative=True) |
643b579
to
7f5d7ae
Compare
Do surveys on behalf of others. TT46965
7f5d7ae
to
b2bb903
Compare
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 91966a6. Thanks a lot for contributing to OCA. ❤️ |
When we have the modules
survey_contact_generation
,survey_sale_generation
orsurvey_crm_generation
a salesman or a portal users could want to act as a representative of another using the surveys that generate those documents as an entry point.Pending:
cc @Tecnativa TT46965
ready for review @pedrobaeza @victoralmau