Skip to content

Conversation

@pilarvargas-tecnativa
Copy link
Contributor

@Tecnativa TT58247

@pedrobaeza @victoralmau please review

Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

Can you check the CI errors and add tests?

In the Create/Update registrations process, I cannot see (or modify) the data for new registrations that are going to be created/modified. Can you check this?

ejemplo2

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 17.0-add-event_sale_reser branch 4 times, most recently from 47ccd39 to dfc69c9 Compare October 21, 2025 20:45
Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

Functional review OK

Related to odoo/odoo#230500

@pedrobaeza pedrobaeza added this to the 17.0 milestone Nov 6, 2025
@pedrobaeza
Copy link
Member

@pilarvargas-tecnativa is this not working until odoo/odoo#230500 is merged, right? Please try to speed up its merge.

@pilarvargas-tecnativa
Copy link
Contributor Author

@eduezerouali-tecnativa please review

@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 17.0-ocabot-merge-pr-484-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8e00d57 into OCA:17.0 Jan 27, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-add-event_sale_reser branch January 27, 2026 09:12
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.

5 participants