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

[14][MIG][WIP] delivery_carrier_deposit #332

Merged
merged 45 commits into from
Mar 14, 2024

Conversation

florian-dacosta
Copy link
Contributor

Need to add tests
Maybe refactore to make it more generic (like accept packages or pickings)

@alexis-via alexis-via force-pushed the 14.0-mig-delivery-carrier-deposit branch 2 times, most recently from acbed57 to fcaf5ad Compare April 21, 2021 13:23
@florian-dacosta florian-dacosta force-pushed the 14.0-mig-delivery-carrier-deposit branch from fcaf5ad to 44e8fef Compare June 2, 2021 16:58
@florian-dacosta florian-dacosta force-pushed the 14.0-mig-delivery-carrier-deposit branch from 44e8fef to 45a115d Compare June 21, 2021 16:31
@florian-dacosta florian-dacosta force-pushed the 14.0-mig-delivery-carrier-deposit branch from 45a115d to 8b21bda Compare July 12, 2021 16:23
@florian-dacosta florian-dacosta force-pushed the 14.0-mig-delivery-carrier-deposit branch from 1c474a8 to 2657f18 Compare February 4, 2022 09:56
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 10, 2022
@github-actions github-actions bot closed this Aug 14, 2022
@alexis-via
Copy link
Contributor

Can someone re-open this PR?

@rvalyi rvalyi reopened this Apr 21, 2023
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 23, 2023
@rousseldenis
Copy link
Contributor

@florian-dacosta Do you plan to fix this ?

@rousseldenis
Copy link
Contributor

/ocabot migration delivery_carrier_deposit

@florian-dacosta
Copy link
Contributor Author

florian-dacosta commented Jul 25, 2023

Well... it would say yes. I'd like to rework, essentially on the v16 but then it will be quick to backport here the tests and eventual improvements.
I am not sure when though.. I think soon enough (unless someone do it first ofc) !

@alexis-via alexis-via force-pushed the 14.0-mig-delivery-carrier-deposit branch from c845335 to 75aaae5 Compare August 14, 2023 15:26
@alexis-via
Copy link
Contributor

pre-commit is now green.
The failure on the odoo test is unrelated (fails in delivery_roulier)

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 17, 2023
@github-actions github-actions bot closed this Jan 21, 2024
@bguillot bguillot reopened this Mar 14, 2024
@alexis-via alexis-via force-pushed the 14.0-mig-delivery-carrier-deposit branch from 75aaae5 to eedd2af Compare March 14, 2024 10:18
@alexis-via
Copy link
Contributor

All green !

@bguillot
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-332-by-bguillot-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8a4784d into OCA:14.0 Mar 14, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged 🎉 migration needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.