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

[15.0][MIG] membership_extension: Migration to 15.0 #128

Merged
merged 48 commits into from
Sep 30, 2022

Conversation

farzeni
Copy link
Contributor

@farzeni farzeni commented Aug 24, 2022

Standard migration from 14.0 to 15.0.

antespi and others added 30 commits August 24, 2022 14:50
* Avoid ensure_one errors.
* Don't do anything when there's no origin (refund made from scratch
  or field erased).
* Use more accurate origin field (if refund link module installed)
- Partners can't be deleted if they have member lines.
…red member (OCA#51)

- A member who is associated to another have the option to set start
date independently from the associate member one
This module uses in previous versions an indirect dependency to
account_invoice_refund_link for detecting the origin invoice where
the refund comes from, but now there's a field in standard for
pointing to that origin invoice, so no need of such code.
Currently translated at 37.8% (14 of 37 strings)

Translation: vertical-association-11.0/vertical-association-11.0-membership_extension
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-11-0/vertical-association-11-0-membership_extension/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: vertical-association-12.0/vertical-association-12.0-membership_extension
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-12-0/vertical-association-12-0-membership_extension/
- Demo membership products weren't membership products
Currently translated at 62.5% (40 of 64 strings)

Translation: vertical-association-12.0/vertical-association-12.0-membership_extension
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-12-0/vertical-association-12-0-membership_extension/de/
Currently translated at 81.2% (52 of 64 strings)

Translation: vertical-association-12.0/vertical-association-12.0-membership_extension
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-12-0/vertical-association-12-0-membership_extension/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: vertical-association-13.0/vertical-association-13.0-membership_extension
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-13-0/vertical-association-13-0-membership_extension/
@farzeni farzeni changed the title [15.0][MIG] $MODULE: Migration to 15.0 [15.0][MIG] membership_extension: Migration to 15.0 Aug 24, 2022
@flachica
Copy link

The commit message must be [MIG] membership_extension 15.0 instead module_extension

@flachica
Copy link

In order to be merge this PR you must write in #114

@rafaelbn rafaelbn added this to the 15.0 milestone Sep 19, 2022
@rafaelbn
Copy link
Member

/ocabot migration membership_extension

@OCA-git-bot OCA-git-bot mentioned this pull request Sep 21, 2022
8 tasks
@rafaelbn
Copy link
Member

Please @fcvalgar @LuisOtto could you review functionality? Just contribution for OCA 😄

@rafaelbn
Copy link
Member

After merging this PR we should review #130

For the School association of our children! Let's digitalize with OpenSource 😄

Copy link

@flachica flachica left a comment

Choose a reason for hiding this comment

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

Functional Ok

You must squash some commits

@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). 🤖

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-128-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4593b19 into OCA:15.0 Sep 30, 2022
@OCA-git-bot
Copy link
Contributor

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

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.