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

[18.0][MIG] fieldservice_sale #1231

Open
wants to merge 102 commits into
base: 18.0
Choose a base branch
from

Conversation

EdgarRetes
Copy link
Contributor

@max3903 max3903 added this to the 18.0 milestone Oct 19, 2024
@max3903 max3903 self-assigned this Oct 19, 2024
@max3903
Copy link
Member

max3903 commented Oct 19, 2024

/ocabot migration fieldservice_sale

@OCA-git-bot OCA-git-bot mentioned this pull request Oct 19, 2024
24 tasks
osi-scampbell and others added 24 commits November 6, 2024 11:59
[IMP] Remove Duplicate Field
Added ability to link a product with FSM order template and create FSM
orders upon sale order confirmation
Added option to install fieldservice_sale module
Added support for selling recurring orders
Travis
Typo, wrong model relationship
Improved values prepation method for recurring
Updated errors on SO smart button for recurring
- Completed field service orders will now update the qty_delivered field on linked SO line
-  suggestions from reviewer comments
- This change will allow to create an FSM order for each sales order and link it to the delivery order and invoice
- Combine field service template info from sale order lines and add it to field service order
Consumable and Storable product types were not generating field services
ibuioli and others added 21 commits November 6, 2024 11:59
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/es_AR/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/pt_BR/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/pt_BR/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/es/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/it/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-16.0/field-service-16.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-16-0/field-service-16-0-fieldservice_sale/sk/
[FIX] fieldservice_sale

[FIX] fieldservice_sale
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: field-service-17.0/field-service-17.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-17-0/field-service-17-0-fieldservice_sale/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-17.0/field-service-17.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-17-0/field-service-17-0-fieldservice_sale/it/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-17.0/field-service-17.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-17-0/field-service-17-0-fieldservice_sale/es_AR/
Currently translated at 100.0% (29 of 29 strings)

Translation: field-service-17.0/field-service-17.0-fieldservice_sale
Translate-URL: https://translation.odoo-community.org/projects/field-service-17-0/field-service-17-0-fieldservice_sale/it/
@EdgarRetes EdgarRetes force-pushed the 18.0-mig-fieldservice_sale branch 2 times, most recently from a011982 to 26e2c63 Compare November 6, 2024 18:30
@EdgarRetes
Copy link
Contributor Author

@brian10048 Do you know how to prevent those warnings?

fsm = self.env["fsm.order"].search(
[
"|",
("sale_id", "=", sale.id),
Copy link
Contributor

Choose a reason for hiding this comment

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

@EdgarRetes I think this line is causing the issue.

I would try this and see if it still works and avoids the warning

Suggested change
("sale_id", "=", sale.id),
("sale_id", "in", sale.ids),

from odoo/odoo@6b907f9

@EdgarRetes
Copy link
Contributor Author

@brian10048 that worked, thanks. Can you check it?

@brian10048
Copy link
Contributor

My thoughts for improvement to make this module more cohesive with core Odoo features

We could remove the field field_service_tracking and combine its usage with the core Odoo field service_tracking

move the "Field Service Tracking" radio
Screenshot_20241107_222410

to the "Create on Order" dropdown
Screenshot_20241107_222343

This would require additional refactoring and migration scripts, but I think it is a good time to make the changes.

@max3903 would you guys be willing to work on the changes for this?

@max3903
Copy link
Member

max3903 commented Nov 8, 2024

What if someone wants to create a task and a FSM order? Same list but additional options?

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.