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

[17.0][MIG] purchase_order_approved #2262

Open
wants to merge 45 commits into
base: 17.0
Choose a base branch
from

Conversation

MarioLM-23
Copy link

Standar migration with changes in old references

LoisRForgeFlow and others added 30 commits May 2, 2024 09:26
Currently translated at 100.0% (9 of 9 strings)

Translation: purchase-workflow-11.0/purchase-workflow-11.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-11-0/purchase-workflow-11-0-purchase_order_approved/nl_NL/
Currently translated at 100.0% (29 of 29 strings)

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_order_approved/zh_CN/
Currently translated at 10.3% (3 of 29 strings)

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_order_approved/pt_BR/
Currently translated at 9.0% (3 of 33 strings)

Translation: purchase-workflow-14.0/purchase-workflow-14.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-14-0/purchase-workflow-14-0-purchase_order_approved/pt_BR/
…chase' state

In core module, service products can be received by editing the received quantity on 'purchase'
order state. This restablishes that behaviour
Currently translated at 6.6% (2 of 30 strings)

Translation: purchase-workflow-15.0/purchase-workflow-15.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-15-0/purchase-workflow-15-0-purchase_order_approved/it/
weblate and others added 13 commits May 2, 2024 09:26
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/
Currently translated at 90.0% (27 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/fr/
Currently translated at 63.3% (19 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/pt_BR/
Currently translated at 93.3% (28 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/pt_BR/
Currently translated at 100.0% (30 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/
Currently translated at 100.0% (30 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/it/
Currently translated at 100.0% (30 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/
Currently translated at 100.0% (30 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/it/
Currently translated at 100.0% (30 of 30 strings)

Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_order_approved
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_order_approved/es/
@MarioLM-23 MarioLM-23 mentioned this pull request May 2, 2024
42 tasks
@MarioLM-23 MarioLM-23 force-pushed the 17.0-mig-purchase_order_approved branch from b80f4c2 to e410804 Compare May 6, 2024 08:25
Copy link
Contributor

@ChristianSantamaria ChristianSantamaria left a comment

Choose a reason for hiding this comment

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

Tested, code review:
purchase_exception addon was added by mistake

Minor changes in views:

Comment on lines 11 to 12
name="readonly"
>state in ['cancel', 'to approve', 'approved']</attribute>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
name="readonly"
>state in ['cancel', 'to approve', 'approved']</attribute>
name="invisible"
>state not in ['cancel', 'to approve', 'approved']</attribute>

Comment on lines 16 to 17
name="readonly"
>state in ['draft', 'to approve', 'approved', 'sent', 'purchase']</attribute>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
name="readonly"
>state in ['draft', 'to approve', 'approved', 'sent', 'purchase']</attribute>
name="invisible"
>state not in ['draft', 'to approve', 'approved', 'sent', 'purchase']</attribute>

<button
name="button_release"
type="object"
readonly="state in ['approved']"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
readonly="state in ['approved']"
invisible="state != 'approved'"

@MarioLM-23 MarioLM-23 force-pushed the 17.0-mig-purchase_order_approved branch from e410804 to 9dc2d40 Compare May 8, 2024 16:02
@MarioLM-23
Copy link
Author

Tested, code review: purchase_exception addon was added by mistake

Minor changes in views:

Done, What should I do to fix the addon error? Then I will change the commit description
Thank you for the review

@ChristianSantamaria
Copy link
Contributor

@mariobinhex, I understand that you are working on two addons at the same time and these changes were inadvertently sneaked in.
image
, you could revert the three files by hand or position yourself in the previous commit and not add the settings in the three files so that later only the approve addon uploads

@MarioLM-23 MarioLM-23 force-pushed the 17.0-mig-purchase_order_approved branch from 9dc2d40 to f908e8a Compare May 10, 2024 11:19
@MarioLM-23 MarioLM-23 force-pushed the 17.0-mig-purchase_order_approved branch from f908e8a to 1751d75 Compare May 10, 2024 12:58
@MarioLM-23
Copy link
Author

@ChristianSantamaria Hello and thanks for the explanation I have already made the changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet