[MIG] website_sale_resource_booking: Migration to 13.0 #583
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
victoralmau
force-pushed
the
13.0-mig-website_sale_resource_booking
branch
2 times, most recently
from
5b9cb2d
to
2e1f0eb
Compare
This module extends the functionality of ``sale_resource_booking`` to support the eCommerce use case and to allow your visitors to buy products that produce a resource booking, and pre-book them before buying. You can also set a timeout for those pre-bookings to expire if unpaid. @Tecnativa TT28202
Before this patch, when a resource.booking was canceled due to being expired, reactivating it would immediately cancel it again when cron ran again. @Tecnativa TT29651 [UPD] Update website_sale_resource_booking.pot [UPD] README.rst
…er is accessible If someone introduces a different email address for the resource booking, a partner with such email is searched and if not found, created, which is correct in theory. But the core method used for that goal (`find_or_create` in `res.partner`) is not taking multi-company into account, searching partners only matching by mail, and creating them in the default company. Due to this, this commits changes the resource booking partner population code to perform the search and creation controlling the involved company. TT33276
victoralmau
force-pushed
the
13.0-mig-website_sale_resource_booking
branch
from
2e1f0eb
to
7b202be
Compare
Depends on OCA/sale-workflow#1763 and OCA/calendar#60. @Tecnativa TT31328
victoralmau
force-pushed
the
13.0-mig-website_sale_resource_booking
branch
from
7b202be
to
bf7fda1
Compare
victoralmau
approved these changes
Feb 2, 2022
There was a problem hiding this comment.
Code review ok.
It's ready to review.
Ping @chienandalu @pedrobaeza
|
This PR has the |
|
/ocabot merge nobump |
|
/ocabot migration website_sale_resource_booking |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 89bb414. 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Depends on:
@Tecnativa TT31328