[15.0][IMP+FIX] resource_booking: Misc changes

OCA / calendar

GNU Affero General Public License v3.0

24 stars 94 forks source link

[15.0][IMP+FIX] resource_booking: Misc changes #95

Closed victoralmau closed 1 year ago

victoralmau commented 1 year ago

Changes done:

[x] [IMP] Use _prepare_home_portal_values() function (An error was displayed when sharing a record, project for example)
[x] [FIX] Change _default_company() to self.env.company from booking types
[x] [IMP] Change t-esc directives to t-out according to v15 guidelines
[x] [IMP] Partial change from tour test to js (necessarry according to _prepare_home_portal_values() changes)

Please @pedrobaeza can you review it?

@Tecnativa

OCA-git-bot commented 1 year ago

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

victoralmau commented 1 year ago

Ping @sergio-teruel

pedrobaeza commented 1 year ago

/ocabot merge minor

OCA-git-bot commented 1 year ago

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

OCA-git-bot commented 1 year ago

@pedrobaeza The merge process could not be finalized, because command git push origin 15.0-ocabot-merge-pr-95-by-pedrobaeza-bump-minor:15.0 failed with output:

To https://github.com/OCA/calendar
 ! [rejected]        15.0-ocabot-merge-pr-95-by-pedrobaeza-bump-minor -> 15.0 (fetch first)
error: failed to push some refs to 'https://***@github.com/OCA/calendar'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

OCA-git-bot commented 1 year ago

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