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

[13.0] [MIG] fieldservice_account #598

Merged
merged 82 commits into from
Feb 12, 2021

Conversation

brian10048
Copy link
Contributor

Migration of fieldservice_account to 13.0

#354

@OCA-git-bot
Copy link
Contributor

Hi @bodedra, @osimallen,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot mentioned this pull request Jul 20, 2020
43 tasks
@brian10048 brian10048 force-pushed the 13.0-mig-fsm-account branch 4 times, most recently from e44abf7 to 799713c Compare July 20, 2020 19:34
@brian10048 brian10048 linked an issue Jul 20, 2020 that may be closed by this pull request
43 tasks
@brian10048 brian10048 force-pushed the 13.0-mig-fsm-account branch 2 times, most recently from 0ee9d40 to 3f60c3e Compare July 20, 2020 20:25
@brian10048
Copy link
Contributor Author

💚 This one is ready for functional testing

@brian10048
Copy link
Contributor Author

@max3903

  1. Do we need a migration script for the accounting model changes?
  2. Can I delete the 12.0 migration script?

"email": "tp@email.com",
"partner_id": self.test_loc_partner.id,
"owner_id": self.test_loc_partner.id,
"customer_id": self.test_loc_partner.id,
Copy link

Choose a reason for hiding this comment

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

Suggested change
"customer_id": self.test_loc_partner.id,

customer_id was moved to fieldservice_account_analytic

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch @RLeeOSI

I'll take care of that.

Do you have any thoughts about my questions here: #598 (comment)?

Copy link

@RLeeOSI RLeeOSI Oct 7, 2020

Choose a reason for hiding this comment

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

Yes, a migration script is needed. However, the v13 account migration is not done yet (see the latest here) so it might be best to put that on hold for now. According to the maintainer guide all old migration scripts should be removed

@max3903
Copy link
Member

max3903 commented Oct 15, 2020

@brian10048

  1. Do we need a migration script for the accounting model changes?

Yes

  1. Can I delete the 12.0 migration script?

You can, but I am not sure you need to.

Copy link
Member

@bodedra bodedra left a comment

Choose a reason for hiding this comment

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

Change file name as well.

fieldservice_account/models/__init__.py Outdated Show resolved Hide resolved
fieldservice_account/models/__init__.py Outdated Show resolved Hide resolved
@brian10048
Copy link
Contributor Author

Migration script added

@brian10048 brian10048 force-pushed the 13.0-mig-fsm-account branch 2 times, most recently from 0428254 to 5c80498 Compare October 22, 2020 01:05
Nelson Ramírez Sánchez and others added 6 commits November 10, 2020 20:58
Currently translated at 100.0% (9 of 9 strings)

Translation: field-service-12.0/field-service-12.0-fieldservice_account
Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account/es/
Currently translated at 44.4% (4 of 9 strings)

Translation: field-service-12.0/field-service-12.0-fieldservice_account
Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account/it/
Currently translated at 100.0% (9 of 9 strings)

Translation: field-service-12.0/field-service-12.0-fieldservice_account
Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account/pt_BR/
@hparfr
Copy link
Contributor

hparfr commented Nov 11, 2020

Hi Brian, please have a look to brian10048#19

@bodedra
Copy link
Member

bodedra commented Nov 17, 2020

@hparfr I have commented brian10048#19 (comment) in your PR.

CC @max3903

@hparfr
Copy link
Contributor

hparfr commented Jan 15, 2021

@brian10048 can you please review brian10048#19 ?

Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

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

@brian10048
Copy link
Contributor Author

@max3903 @dreispt Are either of you able to review this PR and sign off on the v13 migration for fieldservice_account? We would greatly appreciate it!

@hparfr
Copy link
Contributor

hparfr commented Feb 11, 2021

Can we merge this PR? I plan to work on v14 port soon.

@brian10048
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-598-by-brian10048-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit baec233 into OCA:13.0 Feb 12, 2021
@OCA-git-bot
Copy link
Contributor

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

Migration to version 13.0