search

OCA / survey

Survey addons
GNU Affero General Public License v3.0
27 stars 111 forks source link

[15.0][FIX] survey_resource_booking: adapt to partner refactor #104

Closed chienandalu closed 11 months ago

chienandalu commented 11 months ago

We need to adapt tests to https://github.com/OCA/calendar/pull/110

cc @Tecnativa TT45338

please review @carolinafernandez-tecnativa @pedrobaeza

OCA-git-bot commented 11 months ago

Hi @Yajo, some modules you are maintaining are being modified, check this out!

OCA-git-bot commented 11 months ago

On my way to merge this fine PR! Prepared branch 15.0-ocabot-merge-pr-104-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 11 months ago

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

chienandalu commented 11 months ago

Also failing affecting to the tests in sale_resource_booking and website_sale_resource_booking (investigating...)

pedrobaeza commented 11 months ago

One way to avoid this compatibility problem is to add an inverse function to partner_id field, so that it put on partner_ids the proper value. Do you know what I mean?

chienandalu commented 11 months ago

Yes, I was thinking about that, although in this case the problem is that using the Form class, the field should be in the view

pedrobaeza commented 11 months ago

Not sure if putting it as invisible will serve.