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.

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

[MIG] 18.0 delivery carrier default tracking url #933

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

Conversation

gaelTorrecillas
Copy link
Contributor

@gaelTorrecillas gaelTorrecillas commented Jan 15, 2025

Hello,

I migrated delivery carrier default tracking url, thanks you to your review.

Origin branch : 14.0

Regards,

@gaelTorrecillas gaelTorrecillas force-pushed the 18.0-mig-delivery_carrier_default_tracking_url branch 3 times, most recently from fb6a3fb to 5833b67 Compare January 15, 2025 10:05
@rousseldenis
Copy link
Contributor

@gaelTorrecillas Thanks for this. Could you follow migration guide ?

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0#technical-method-to-migrate-a-module-from-170-to-180-branch

As commit history should be respected.

@rousseldenis
Copy link
Contributor

/ocabot migration delivery_carrier_default_traking_url

@gaelTorrecillas
Copy link
Contributor Author

gaelTorrecillas commented Jan 15, 2025

@rousseldenis hello,

Yes, but I think that in this pr is not good, I'll pass again this process.

edit : I forced update.

Regards,

@gaelTorrecillas gaelTorrecillas force-pushed the 18.0-mig-delivery_carrier_default_tracking_url branch from 5833b67 to 59452a5 Compare January 15, 2025 14:39
@gaelTorrecillas gaelTorrecillas force-pushed the 18.0-mig-delivery_carrier_default_tracking_url branch from 59452a5 to 7cd8a2c Compare February 19, 2025 12:41
@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). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants