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

[MIG]11.0 mig l10n nl partner salutation #210

Merged
merged 17 commits into from
Jul 29, 2020

Conversation

daramousk
Copy link
Member

No description provided.

@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from 7e5b19a to 2b22557 Compare November 28, 2018 13:51
@hbrunn hbrunn added this to the 11.0 milestone Nov 29, 2018
@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch 2 times, most recently from 55b2714 to b538640 Compare November 29, 2018 14:39
@daramousk daramousk changed the title 11.0 mig l10n nl partner salutation [MIG]11.0 mig l10n nl partner salutation Nov 29, 2018
@daramousk daramousk closed this Nov 30, 2018
@daramousk daramousk reopened this Nov 30, 2018
@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from b538640 to 23c9c89 Compare November 30, 2018 15:55
@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from 23c9c89 to de098d4 Compare January 17, 2019 10:46
@@ -0,0 +1,2 @@
from . import res_partner_title
Copy link
Contributor

Choose a reason for hiding this comment

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

license line

Copy link
Contributor

@NL66278 NL66278 left a comment

Choose a reason for hiding this comment

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

👍 Some comments. Not blocking. LGTM.

@NL66278
Copy link
Contributor

NL66278 commented Jan 21, 2019

Failures are due to OCA/partner-contact#660 (review) not yet merged.

@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from de098d4 to ce93230 Compare January 21, 2019 11:29
@daramousk
Copy link
Member Author

@lfreeke And here

@hbrunn
Copy link
Member

hbrunn commented Mar 18, 2019

@daramousk please rebase on current 11.0

@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from ce93230 to 63cef15 Compare March 25, 2019 18:32
@daramousk daramousk closed this Mar 25, 2019
@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from 63cef15 to 7cead91 Compare March 25, 2019 18:33
@daramousk daramousk reopened this Mar 25, 2019
George Daramouskas and others added 12 commits June 11, 2019 15:37
- Do not add gender field in this module, but rely on partner_contact_gender.
- Use standard directories.
- Make sure tests use same name formatting as in model.
- make test class and method name confirm to actual tests done.
- Properly handle is_company partners.
- Make a difference between unknown gender and other gender.
+ Standard copyright headers
+ Migration of 'unknown' gender value to NULL.
@daramousk daramousk force-pushed the 11.0-mig-l10n_nl_partner_salutation branch from b68a175 to 952da76 Compare June 11, 2019 13:38
@daramousk
Copy link
Member Author

Hello @sbidoul ,
Perhaps this is ready to be merged?

@NL66278
Copy link
Contributor

NL66278 commented May 13, 2020

Closing and reopening to force runbot

@NL66278 NL66278 closed this May 13, 2020
@NL66278 NL66278 reopened this May 13, 2020
@andreschenkels
Copy link
Member

He Guys,
Can I help with this migration? I would love to have this module in 12.0 and 13.0.

@NL66278
Copy link
Contributor

NL66278 commented May 25, 2020

Close and open again. To rerun runbot.

@NL66278 NL66278 closed this May 25, 2020
@NL66278 NL66278 reopened this May 25, 2020
@hbrunn
Copy link
Member

hbrunn commented Jul 29, 2020

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 11.0-ocabot-merge-pr-210-by-hbrunn-bump-major, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 620dfe1 into OCA:11.0 Jul 29, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7d6ec52. 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.

8 participants