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

[12.0][FIX] membership_delegated_partner: changing delegated member #79

Merged

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Nov 21, 2019

Fixes #78

This PR is also a proposal to use a different approach for calculating the membership lines.
The proposal is to use a compute method for the partner in the membership lines.

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, although tests are failing...

@astirpe astirpe force-pushed the 12_fix_membership_delegated_partner branch 2 times, most recently from c137d81 to c0e6672 Compare November 22, 2019 07:19
@astirpe astirpe changed the title [12.0][FIX/IMP] membership_delegated_partner: changing delegated member [12.0][FIX] membership_delegated_partner: changing delegated member Nov 22, 2019
@astirpe astirpe force-pushed the 12_fix_membership_delegated_partner branch from c0e6672 to 806fd72 Compare November 22, 2019 08:27
@astirpe
Copy link
Member Author

astirpe commented Nov 22, 2019

@chienandalu Fixed!

@astirpe
Copy link
Member Author

astirpe commented Nov 29, 2019

@chienandalu would you update your review?

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @astirpe :)

@chienandalu
Copy link
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-79-by-chienandalu-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 29, 2019
Signed-off-by chienandalu
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot OCA-git-bot merged commit 806fd72 into OCA:12.0 Nov 29, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 845ea56. Thanks a lot for contributing to OCA. ❤️

@astirpe astirpe deleted the 12_fix_membership_delegated_partner branch November 29, 2019 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] membership_delegated_partner: changing delegated member
4 participants