-
-
Notifications
You must be signed in to change notification settings - Fork 198
[ADD] maintenance_partner #510
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
Conversation
b95843a to
0da4fbd
Compare
dalonsod
left a comment
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.
Interesting functionality.
Only code review, see comments. And initial hook for requests partner_id initialization could be needed IMO
0da4fbd to
be2b37c
Compare
|
@dalonsod all comments attended |
be2b37c to
082af78
Compare
dalonsod
left a comment
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.
👍
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 63ce930. Thanks a lot for contributing to OCA. ❤️ |
This module shows a related partner on requests & Partners.