search

OCA / calendar

GNU Affero General Public License v3.0
24 stars 95 forks source link

[14.0][IMP] resource_booking: Allow a booking to span more than one calendar day #77

Closed carmenbianca closed 1 year ago

carmenbianca commented 1 year ago

This PR solves two issues:

Internal task: https://gestion.coopiteasy.be/web#id=9834&action=475&active_id=434&model=project.task&view_type=form&menu_id=536

OCA-git-bot commented 1 year ago

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

carmenbianca commented 1 year ago

Ready for review after a fair amount of tinkering.

carmenbianca commented 1 year ago

@huguesdk I tried hacking the same result by forcibly setting hour_to to 24.0 on resource.calendar.attendance records -> didn't work.

carmenbianca commented 1 year ago

This is now rebased on top of #80. Will need to merge that PR before merging this.

carmenbianca commented 1 year ago

Now that #80 is merged, I rebased this on top of 14.0.

norlinhenrik commented 1 year ago

LGTM When I configure the resource calender with work hours every day 00:00-23:59, I can book a resource for a month. :)

pedrobaeza commented 1 year ago

Please squash commits a bit and check pre-commit.

carmenbianca commented 1 year ago

Rebased on top of 14.0. Commits squashed. pre-commit works locally.

pedrobaeza commented 1 year ago

/ocabot merge patch

You can now fw-port it to upper versions.

OCA-git-bot commented 1 year ago

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

OCA-git-bot commented 1 year ago

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