-
-
Notifications
You must be signed in to change notification settings - Fork 869
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] Migration partner_contact_birthdate partner_contact_gender part… #315
[MIG] Migration partner_contact_birthdate partner_contact_gender part… #315
Conversation
…ner_contact_personal_information_page to v10
As said in #307, you should add the post init hook to call Also, I think you should split the migration commit, one per module. Otherwise, 👍 (code review + functional tests) |
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.
Hi @invitu thanks for migration. But please make a single PR per module. It helps a lot 😄
@@ -4,7 +4,7 @@ | |||
|
|||
{ | |||
"name": "Contact's birthdate", | |||
"version": "9.0.1.0.0", | |||
"version": "10.0.1.0.0", | |||
"author": "Jairo Llopis," |
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.
Please add Tecnativa, we join all together in a new team.
@@ -5,7 +5,7 @@ | |||
{ | |||
"name": "Contact gender", | |||
"summary": "Add gender field to contacts", | |||
"version": "9.0.1.1.0", | |||
"version": "10.0.1.1.0", | |||
"category": "Customer Relationship Management", | |||
"website": "https://odoo-community.org/", |
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.
Please change to https://www.tecnativa.com as we are in the new team
@@ -5,7 +5,7 @@ | |||
{ | |||
"name": "Contact gender", | |||
"summary": "Add gender field to contacts", | |||
"version": "9.0.1.1.0", | |||
"version": "10.0.1.1.0", | |||
"category": "Customer Relationship Management", | |||
"website": "https://odoo-community.org/", | |||
"author": "Grupo ESOC, Odoo Community Association (OCA)", |
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.
Please add Tecnativa, we join all together in a new team
@rafaelbn, in this case, we can accept all the modules together are they are simple and bound. |
Sorry I put |
OK @pedrobaeza just minor comments and #307 (comment) |
I did a PR for those 3 modules because they are small and depend on partner_contact_personal_information_page requested fixes are commited |
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.
LGTM code & test
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.
@pedrobaeza ready to merge, tested in runbot
"author": "Jairo Llopis," | ||
"Tecnativa," | ||
"Odoo Community Association (OCA)", |
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.
Shouldn't it be "Jairo Llopis, Tecnativa, Odoo Community Association (OCA)" ?
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10
OCA#315) [MIG] Migration partner_contact_birthdate partner_contact_gender partner_contact_personal_information_page to v10 Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
…ner_contact_personal_information_page to v10