search

OCA / vertical-association

Odoo addons for membership related tasks
GNU Affero General Public License v3.0
38 stars 112 forks source link

[FIX] membership_initial_fee: associate members #99

Closed chienandalu closed 3 years ago

chienandalu commented 3 years ago

If the invoice partner is associated to the real member, we should evalute past membership lines from that one

cc @Tecnativa TT27052

pedrobaeza commented 3 years ago

But having such method on base and then expanding it on the other module.

chienandalu commented 3 years ago

I like the idea to avoid further issues, but the problem is that they both depend on the membership core module

pedrobaeza commented 3 years ago

Fast-tracking for fixing the issue:

/ocabot merge patch

To be fw-port it to 13?

OCA-git-bot commented 3 years ago

This PR looks fantastic, let's merge it! Prepared branch 12.0-ocabot-merge-pr-99-by-pedrobaeza-bump-patch, awaiting test results.

chienandalu commented 3 years ago

I pinged @astirpe in his migration PR: https://github.com/OCA/vertical-association/pull/84

OCA-git-bot commented 3 years ago

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