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

[16.0][MIG] website_sale_stock_provisioning_date: Migration to version 16.0 #846

Merged

Conversation

ernesto-garcia-tecnativa

@Tecnativa TT44233

Ernesto Tejeda and others added 9 commits September 17, 2023 17:50
Show the future provisioning date closest to the current date
for a product in the eCommerce.
The provisioning date was always showing in the same format without
respecting the date format defined in the language settings.

It is added in function `_get_combination_info` instead of in function
`_get_next_provisioning_date` so that the date is still formatted even
if function `_get_next_provisioning_date` is inherited.
@ernesto-garcia-tecnativa
Copy link
Author

@pedrobaeza @chienandalu please review

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

@pedrobaeza
Copy link
Member

/ocabot migration website_sale_stock_provisioning_date

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 18, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 18, 2023
33 tasks
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-846-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit baa4c92 into OCA:16.0 Sep 18, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 08d4dfe. 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.

6 participants