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

[13.0][MIG] event_sale_reservation: Migration to 13.0 #254

Merged
merged 4 commits into from
Feb 3, 2022

Conversation

victoralmau
Copy link
Member

Migration to 13.0

Please @chienandalu and @pedrobaeza can you review it?

@Tecnativa TT31321

@victoralmau victoralmau force-pushed the 13.0-mig-event_sale_reservation branch 2 times, most recently from ba25313 to 535655e Compare January 25, 2022 13:08
@pedrobaeza
Copy link
Member

/ocabot migration event_sale_reservation

.travis.yml Outdated Show resolved Hide resolved
@victoralmau victoralmau force-pushed the 13.0-mig-event_sale_reservation branch 2 times, most recently from 9059189 to 7fb5c00 Compare January 26, 2022 08:31
@pedrobaeza
Copy link
Member

/ocabot migration event_sale_reservation

@OCA-git-bot OCA-git-bot added this to the 13.0 milestone Jan 27, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 27, 2022
29 tasks
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.

Tested functionally. As for the event_registration_multi_qty:

Jairo Llopis and others added 4 commits January 28, 2022 10:37
This module extends the functionality of event_sale to support selling
reservations of events that still don't exist and to allow you to schedule the
creation of events based on how many reservations already exist.

Includes integration workaround to avoid test conflicts with
`event_sale_registration_multi_qty`.

@Tecnativa TT27664

[UPD] Update event_sale_reservation.pot

[UPD] README.rst
@victoralmau victoralmau force-pushed the 13.0-mig-event_sale_reservation branch from 7fb5c00 to 41a1f11 Compare January 28, 2022 09:42
@victoralmau
Copy link
Member Author

Changes done. Any other comments to take into account to apply it?

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.

Thanks :)

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-254-by-pedrobaeza-bump-nobump, awaiting test results.

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

@OCA-git-bot OCA-git-bot merged commit a5a5a1d into OCA:13.0 Feb 3, 2022
@OCA-git-bot
Copy link
Contributor

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

None yet

4 participants