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

[17.0][MIG] stock_production_lot_expired_date #1580

Merged

Conversation

Deriman-Alonso
Copy link
Member

Standart Migration

@Deriman-Alonso Deriman-Alonso mentioned this pull request Apr 16, 2024
63 tasks
Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on runboat, LGTM!

@Christian-RB
Copy link

@Deriman-Alonso Check your commits names and separate the pre-commit from the migration itself please.
Looks good
Thanks

@Deriman-Alonso Deriman-Alonso force-pushed the 17.0-mig-stock_production_lot_expired_date branch from 4cf8561 to 5badf9d Compare May 6, 2024 07:27
@Deriman-Alonso
Copy link
Member Author

@Christian-RB Hi, That's done.
Thanks for the review.

@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). 🤖

@peluko00
Copy link

Can you rebase @Deriman-Alonso please

@dreispt
Copy link
Member

dreispt commented Nov 30, 2024

/ocabot migration stock_production_lot_expired_date
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Nov 30, 2024
@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-1580-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b7b8ec3 into OCA:17.0 Nov 30, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at dc0e272. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants