Skip to content

[18.0] [MIG] sale_order_line_menu: Migration to 18.0#3470

Merged
OCA-git-bot merged 17 commits into
OCA:18.0from
Borruso:18.0-mig-sale_order_line_menu
Jan 29, 2025
Merged

[18.0] [MIG] sale_order_line_menu: Migration to 18.0#3470
OCA-git-bot merged 17 commits into
OCA:18.0from
Borruso:18.0-mig-sale_order_line_menu

Conversation

@Borruso

@Borruso Borruso commented Dec 7, 2024

Copy link
Copy Markdown
Contributor

Migration from 17.0 to 18.0

Issue: #3344

Freni-OSI and others added 16 commits December 7, 2024 15:31
… (Belgium))

Translated using Weblate (French (France))

Currently translated at 100.0% (1 of 1 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_order_line_menu
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_order_line_menu/fr_FR/

Added translation using Weblate (Spanish)

Translated using Weblate (Spanish)

Currently translated at 100.0% (1 of 1 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_order_line_menu
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_order_line_menu/es/

Added translation using Weblate (French (Belgium))

Added translation using Weblate (French)

Translated using Weblate (French)

Currently translated at 100.0% (1 of 1 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_order_line_menu
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_order_line_menu/fr/
New views and fields added to the sale_order_line_menu

MT-2028 @moduon @rafaelbn @dreispt @Shide @fcvalgar please review
Currently translated at 100.0% (19 of 19 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_order_line_menu
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_order_line_menu/es/
@Borruso Borruso force-pushed the 18.0-mig-sale_order_line_menu branch 2 times, most recently from 51a7f4c to 414a138 Compare December 7, 2024 15:20
@Naraka

Naraka commented Dec 9, 2024

Copy link
Copy Markdown
Contributor

If you try to create a new order line within the form of an existing order line, it throws an error.

Captura desde 2024-12-09 17-05-02

@rousseldenis

Copy link
Copy Markdown
Contributor

/ocabot migration sale_order_line_menu

@sheragar sheragar left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. Funtional Review 👍

@Borruso Borruso force-pushed the 18.0-mig-sale_order_line_menu branch 2 times, most recently from 77878fb to 96bf699 Compare December 12, 2024 14:32
@Borruso

Borruso commented Dec 12, 2024

Copy link
Copy Markdown
Contributor Author

If you try to create a new order line within the form of an existing order line, it throws an error.

Captura desde 2024-12-09 17-05-02

Can you check again?

@Naraka Naraka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, tested in runboat

@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

@Shide Shide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code LGTM

Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml
Comment thread sale_order_line_menu/views/sale_order_line_views.xml Outdated
Comment thread sale_order_line_menu/views/sale_order_line_views.xml
Comment thread sale_order_line_menu/views/sale_order_line_views.xml
@Borruso Borruso force-pushed the 18.0-mig-sale_order_line_menu branch from 96bf699 to 7f8fa7a Compare January 10, 2025 09:35
@Borruso Borruso force-pushed the 18.0-mig-sale_order_line_menu branch 3 times, most recently from c8a0606 to fdbf560 Compare January 13, 2025 08:54
@Borruso Borruso force-pushed the 18.0-mig-sale_order_line_menu branch from fdbf560 to c7bde69 Compare January 13, 2025 09:01

@PieterPaulussen PieterPaulussen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the changes regarding my feedback. There are indeed other issues that need to be addressed in a different PR. But this will suffice for now. THanks.

@joepsanders

Copy link
Copy Markdown

@PieterPaulussen is this PR ready for merging? What other issues need to be addressed? Could you point that PR? Thanks!

@PieterPaulussen

Copy link
Copy Markdown
Contributor

@joepsanders The PR has 4 approvals, including mine. I thinks it's safe to merge.

@pedrobaeza

Copy link
Copy Markdown
Member

Technically, a PSC should review it, no matter the number of approvals, but I see some respected contributors that have approved already, so merging:

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-3470-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 75da61d into OCA:18.0 Jan 29, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 95a65c1. Thanks a lot for contributing to OCA. ❤️

trisdoan pushed a commit to trisdoan/sale-workflow that referenced this pull request Mar 13, 2025
Signed-off-by pedrobaeza
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.