search

OCA / delivery-carrier

Odoo Carriers And Deliveries Management
https://odoo-community.org/psc-teams/logistics-18
GNU Affero General Public License v3.0
112 stars 354 forks source link

[14.0][FIX] commercialName is mandatory for dropoff site so we can add it every time #886

Closed bguillot closed 1 month ago

bguillot commented 1 month ago

commercialName is not an mandatory field in basic case but mandatoiry for dropoff site delivery.

I propose to send it every time to avoid other delivery types issues instead of adding it in only in laposte_dropoff_site module

OCA-git-bot commented 1 month ago

Hi @florian-dacosta, some modules you are maintaining are being modified, check this out!

hparfr commented 1 month ago

/ocabot merge minor

OCA-git-bot commented 1 month ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-886-by-hparfr-bump-minor, awaiting test results.

rousseldenis commented 1 month ago

@bguillot For next time, don't forget to mention module name in your commit message. Thanks

OCA-git-bot commented 1 month ago

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