Skip to content

[ADD] inventory: cluster picking #5732

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
Closed
wants to merge 1 commit into from
Closed

[ADD] inventory: cluster picking #5732

wants to merge 1 commit into from

Conversation

@Felicious
Copy link
Contributor

@Felicious Felicious commented Sep 6, 2023

Summary

Add new documentation about cluster picking. Includes:

  1. An image illustrating how orders are grouped (reviewers please let me know whether the image is helpful! 😄)
  2. How to process cluster picking in barcode

Task

@Felicious Felicious requested a review from a team September 6, 2023 02:13
@Felicious Felicious self-assigned this Sep 6, 2023
@robodoo
Copy link
Collaborator

robodoo commented Sep 6, 2023

Pull request status dashboard.

@Felicious
Copy link
Contributor Author

Felicious commented Sep 6, 2023 •
edited
Loading

Hi @odoo/inventory-doc-review! This new doc about the last picking method is ready for review! You can review it on this runbot instance

Note: I pushed some commits to update an image and the runbot link has been updated accordingly! (:

@Felicious Felicious force-pushed the 16.0-inventory-cluster-picking branch 2 times, most recently from cf32928 to 8f6709d Compare September 7, 2023 01:50
hojo-odoo
hojo-odoo reviewed Oct 16, 2023
View reviewed changes
Copy link
Contributor

@hojo-odoo hojo-odoo left a comment

Choose a reason for hiding this comment

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

Hey @Felicious,

Nice job on this. I left a few comments and suggestions.

Thanks,
hojo

@Felicious Felicious force-pushed the 16.0-inventory-cluster-picking branch from d34b0f0 to 3deac6c Compare October 25, 2023 17:54
@C3POdoo C3POdoo requested a review from a team October 25, 2023 17:55
@Felicious Felicious requested review from hojo-odoo and removed request for a team October 25, 2023 17:56
hojo-odoo
hojo-odoo approved these changes Nov 9, 2023
View reviewed changes
Copy link
Contributor

@hojo-odoo hojo-odoo left a comment

Choose a reason for hiding this comment

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

Hey @Felicious,

Great job on this. It should be ready to go once you address my comments.

Thanks,
hojo

@Felicious Felicious force-pushed the 16.0-inventory-cluster-picking branch from c1faa62 to 7301627 Compare November 10, 2023 01:45
@Felicious Felicious requested a review from a team November 10, 2023 01:45
@Felicious Felicious force-pushed the 16.0-inventory-cluster-picking branch from 7301627 to 5b0232d Compare November 10, 2023 01:53
@Felicious
Copy link
Contributor Author

Felicious commented Nov 10, 2023

Hi @odoo/crm-doc-review.

Thank you in advance for the review 😄

ksc-odoo
ksc-odoo approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@ksc-odoo ksc-odoo left a comment

Choose a reason for hiding this comment

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

@Felicious Just finished my review of this doc -- GREAT WORK! Only had a collection of small adjustments that need your attention. But, once you give those a look, and make the necessary changes, I think this one will be ready for the next stage. Nice job!

meng-odoo
meng-odoo approved these changes Nov 15, 2023
View reviewed changes
Copy link
Contributor

@meng-odoo meng-odoo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Hi @Felicious! I just gave this a final review and it looks great---very clear and well-written. I just had minor suggestions/typo fixes. Then please go ahead and move this to technical review. Thanks!

@Felicious @hojo-odoo
@ksc-odoo @meng-odoo
[ADD] inventory: cluster picking
f93ac8b 
Co-authored-by: hojo-odoo <[email protected]>
Co-authored-by: ksc-odoo <[email protected]>
Co-authored-by: meng-odoo <[email protected]>
@Felicious Felicious force-pushed the 16.0-inventory-cluster-picking branch from 362a3c8 to f93ac8b Compare November 16, 2023 01:10
StraubCreative
StraubCreative approved these changes Nov 22, 2023
View reviewed changes
Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

@robodoo delegate=samueljlieber

@StraubCreative
Copy link
Contributor

StraubCreative commented Nov 23, 2023

@robodoo r+

robodoo pushed a commit that referenced this pull request Nov 23, 2023
@Felicious @hojo-odoo
@ksc-odoo @meng-odoo
[ADD] inventory: cluster picking
32900ef 
closes #5732

Signed-off-by: Zachary Straub (zst) <[email protected]>
Co-authored-by: hojo-odoo <[email protected]>
Co-authored-by: ksc-odoo <[email protected]>
Co-authored-by: meng-odoo <[email protected]>
@fw-bot fw-bot mentioned this pull request Nov 23, 2023
Closed
@robodoo robodoo closed this Nov 23, 2023
This was referenced Nov 23, 2023
Closed
Closed
This was referenced Nov 23, 2023
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@samueljlieber samueljlieber samueljlieber left review comments

@StraubCreative StraubCreative StraubCreative approved these changes

+3 more reviewers

@ksc-odoo ksc-odoo ksc-odoo approved these changes

@meng-odoo meng-odoo meng-odoo approved these changes

@hojo-odoo hojo-odoo hojo-odoo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Felicious Felicious

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@Felicious @robodoo @StraubCreative @samueljlieber @ksc-odoo @meng-odoo @hojo-odoo