Skip to content

[18.0][ADD] delivery_pre_shipping: New module to send a delivery to the carrier before the picking is validated#1070

Open
carlos-lopez-tecnativa wants to merge 1 commit intoOCA:18.0from
Tecnativa:18.0-add-delivery_carrier_manual_send
Open

[18.0][ADD] delivery_pre_shipping: New module to send a delivery to the carrier before the picking is validated#1070
carlos-lopez-tecnativa wants to merge 1 commit intoOCA:18.0from
Tecnativa:18.0-add-delivery_carrier_manual_send

Conversation

@carlos-lopez-tecnativa
Copy link
Contributor

TT58440

@Tecnativa @pedrobaeza @sergio-teruel could you please review this?

@pedrobaeza
Copy link
Member

I would remove the carrier word from the module name.

@pedrobaeza
Copy link
Member

And reading the context README, I would rename it to delivery_pre_shipping, or at least, delivery_manual_shipping, to use the same terminology as Odoo.

@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 18.0-add-delivery_carrier_manual_send branch from 861ba6d to 3ecbb14 Compare October 29, 2025 16:31
@carlos-lopez-tecnativa carlos-lopez-tecnativa changed the title [18.0][ADD] delivery_carrier_manual_send: New module to send a delivery to the carrier before the picking is validated [18.0][ADD] delivery_manual_shipping: New module to send a delivery to the carrier before the picking is validated Oct 29, 2025
@pedrobaeza pedrobaeza added this to the 18.0 milestone Oct 29, 2025
@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 18.0-add-delivery_carrier_manual_send branch from 3ecbb14 to f5eda2e Compare October 29, 2025 17:15
@carlos-lopez-tecnativa carlos-lopez-tecnativa changed the title [18.0][ADD] delivery_manual_shipping: New module to send a delivery to the carrier before the picking is validated [18.0][ADD] delivery_pre_shipping: New module to send a delivery to the carrier before the picking is validated Oct 29, 2025
@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 18.0-add-delivery_carrier_manual_send branch from f5eda2e to 7fcd350 Compare October 29, 2025 19:01
@pedrobaeza
Copy link
Member

Thinking twice about this, in fact the way Odoo intends to do this is through flexible routes + checking the mark "Generate Shipping Labels", and having 2 pickings when this happens. Please reflect this in the README, but it's not bad to have this alternative anyway.

@sergio-teruel
Copy link
Contributor

@carlos-lopez-tecnativa ¿Se puede revisar ya?

@carlos-lopez-tecnativa carlos-lopez-tecnativa marked this pull request as ready for review February 12, 2026 15:47
@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 18.0-add-delivery_carrier_manual_send branch from 7fcd350 to 3b46870 Compare February 17, 2026 14:23
@carlos-lopez-tecnativa
Copy link
Contributor Author

@carlos-lopez-tecnativa ¿Se puede revisar ya?

Yes, please. This PR is ready for review and merge.

@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 18.0-add-delivery_carrier_manual_send branch from 3b46870 to 9fd2786 Compare February 17, 2026 14:33
@carlos-lopez-tecnativa
Copy link
Contributor Author

Thinking twice about this, in fact the way Odoo intends to do this is through flexible routes + checking the mark "Generate Shipping Labels", and having 2 pickings when this happens. Please reflect this in the README, but it's not bad to have this alternative anyway.

Added to the README, but this feature only works when you have stock available. In this case, this module is intended to work when you do not have stock yet and you have not validated the picking.

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.

3 participants