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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] stock_picking_show_linked: Migration to 18.0 #2219

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

[18.0][MIG] stock_picking_show_linked: Migration to 18.0 #2219

wants to merge 15 commits into from

Conversation

fd-oerp
Copy link

@fd-oerp fd-oerp commented Dec 11, 2024

No description provided.

Juany Davila and others added 14 commits December 11, 2024 14:54
@fd-oerp
stock_picking_show_linked: Add to 14.0
edf36c7
@oca-travis @fd-oerp
Update stock_picking_show_linked.pot
8878ff2
@OCA-git-bot @fd-oerp
README.rst
3e4d743
@OCA-git-bot @fd-oerp
icon.png
81a5d78
@OCA-git-bot @fd-oerp
stock_picking_show_linked 14.0.1.0.1
44adda2
@fd-oerp
[MIG] stock_picking_show_linked: migration to 15.0
789024c
@fd-oerp
[UPD] Update stock_picking_show_linked.pot
c4bc2fb
@OCA-git-bot @fd-oerp
[UPD] README.rst
27b6513
@bosd @fd-oerp
[MIG] stock_picking_show_linked: Migration to 16.0
51951f3
@fd-oerp
[UPD] Update stock_picking_show_linked.pot
d57ba9f
@OCA-git-bot @fd-oerp
[BOT] post-merge updates
b7b61ab
@mymage @fd-oerp
Added translation using Weblate (Italian)
702176c
@mymage @fd-oerp
Translated using Weblate (Italian)
a42b59c 
Currently translated at 100.0% (5 of 5 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_picking_show_linked
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_picking_show_linked/it/
@fd-oerp
[IMP] stock_picking_show_linked: pre-commit auto fixes
cf59722
@fd-oerp
Copy link
Author

fd-oerp commented Dec 11, 2024

Hi @rousseldenis

Could you please review this PR and let me know if any changes are needed from my end?

Thank you

@fd-oerp fd-oerp force-pushed the 18.0-mig-stock_picking_show_linked branch from 1ef64ca to c2bc7fe Compare December 11, 2024 20:47
@rousseldenis
Copy link
Contributor

/ocabot migration stock_picking_show_linked

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 12, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 12, 2024
Open
23 tasks
rousseldenis
rousseldenis requested changes Dec 12, 2024
View reviewed changes
stock_picking_show_linked/models/stock_picking.py Outdated Show resolved Hide resolved
stock_picking_show_linked/models/stock_picking.py Outdated Show resolved Hide resolved
stock_picking_show_linked/models/stock_picking.py Outdated Show resolved Hide resolved
@rousseldenis
Copy link
Contributor

@fd-oerp Shouldn't it be good to get the whole chain or is this intended to show the direct destination and origin pickings ?

@fd-oerp fd-oerp force-pushed the 18.0-mig-stock_picking_show_linked branch from a1babfa to 48edd9a Compare December 12, 2024 15:16
@fd-oerp
Copy link
Author

fd-oerp commented Dec 12, 2024

@fd-oerp Shouldn't it be good to get the whole chain or is this intended to show the direct destination and origin pickings ?

Hi @rousseldenis Thank you for your feedback. As far as I understand, this module is designed to show the direct destination and origin pickings. Please let me know if you believe that it could be improved in any other way.

@rousseldenis
Copy link
Contributor

@fd-oerp Shouldn't it be good to get the whole chain or is this intended to show the direct destination and origin pickings ?

Hi @rousseldenis Thank you for your feedback. As far as I understand, this module is designed to show the direct destination and origin pickings. Please let me know if you believe that it could be improved in any other way.

No, it was intended like that : #1444 (comment)

@fd-oerp
Copy link
Author

fd-oerp commented Dec 12, 2024

@fd-oerp Shouldn't it be good to get the whole chain or is this intended to show the direct destination and origin pickings ?

Hi @rousseldenis Thank you for your feedback. As far as I understand, this module is designed to show the direct destination and origin pickings. Please let me know if you believe that it could be improved in any other way.

No, it was intended like that : #1444 (comment)

Appreciate your input, Thank you

@fd-oerp
Copy link
Author

fd-oerp commented Dec 12, 2024

Hi @LoisRForgeFlow

Could you please review this PR and let me know if any changes are needed from my end ?
Thank you

@fd-oerp
[18.0][MIG] stock_picking_show_linked: Migration to 18.0
297798d 
[18.0][MIG] stock_picking_show_linked: Migration to 18.0
[18.0][MIG] stock_picking_show_linked: Migration to 18.0
[18.0][MIG] stock_picking_show_linked: Migration to 18.0

[18.0][MIG] stock_picking_show_linked: Migration to 18.0

[18.0][MIG] stock_picking_show_linked: Migration to 18.0, suggested improvements

[18.0][MIG] stock_picking_show_linked: Migration to 18.0, improve contributors
@fd-oerp fd-oerp force-pushed the 18.0-mig-stock_picking_show_linked branch from 600d57e to 297798d Compare December 12, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rousseldenis rousseldenis rousseldenis approved these changes

Assignees
No one assigned
Labels
migration needs review
Projects
None yet
Milestone
18.0
Development

Successfully merging this pull request may close these issues.
6 participants
@fd-oerp @rousseldenis @OCA-git-bot @oca-travis @bosd @mymage