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

[14.0][FIX] delivery_send_to_shipper_at_operation #720

Merged
merged 1 commit into from
Dec 12, 2023
Merged

[14.0][FIX] delivery_send_to_shipper_at_operation #720

merged 1 commit into from
Dec 12, 2023

Conversation

jbaudoux
Copy link
Contributor

@jbaudoux jbaudoux commented Oct 12, 2023 •
edited
Loading

When duplicating an already send picking, allow to register it at the shipper

Includes: #734

@jbaudoux
Copy link
Contributor Author

cc @sebalix @mt-software-de

@sebalix sebalix changed the title [FIX] delivery_send_to_shipper_at_operation [14.0][FIX] delivery_send_to_shipper_at_operation Oct 12, 2023
@sebalix
Copy link
Contributor

sebalix commented Oct 12, 2023

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

@sebalix The merge process could not start, because command git fetch origin pull/720/head:tmp-pr-720 failed with output:

fatal: couldn't find remote ref pull/720/head

@jbaudoux
Copy link
Contributor Author

/ocabot merge patch

@jbaudoux jbaudoux force-pushed the 14-fix_delivery_send_to_shipper_at_operation branch from 6490c23 to 6441422 Compare December 11, 2023 14:23
@jbaudoux
Copy link
Contributor Author

rebased

@jbaudoux jbaudoux force-pushed the 14-fix_delivery_send_to_shipper_at_operation branch from 6441422 to 285175e Compare December 11, 2023 14:48
@jbaudoux
[FIX] delivery_send_to_shipper_at_operation
6b62ddf 
When duplicating an already send picking, allow to register it at the
shipper
@jbaudoux jbaudoux force-pushed the 14-fix_delivery_send_to_shipper_at_operation branch from 285175e to 6b62ddf Compare December 12, 2023 08:29
@OCA-git-bot
Copy link
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). 🤖

@jbaudoux
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-720-by-jbaudoux-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9c725af into OCA:14.0 Dec 12, 2023
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@jbaudoux jbaudoux mentioned this pull request Mar 15, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mt-software-de mt-software-de mt-software-de approved these changes

@sebalix sebalix sebalix approved these changes

@TDu TDu TDu approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jbaudoux @sebalix @OCA-git-bot @TDu @mt-software-de