OCA / hr-attendance

HR Attendance OCA modules for Odoo
GNU Affero General Public License v3.0
48 stars 117 forks source link

[FIX] hr_attendance_reason: search_read params #150

Closed danielduqma closed 11 months ago

danielduqma commented 1 year ago

search_read doesn't match definition from core, read_kwargs was missing.

This can be tested in a database this module and pos_hr installed, and trying to login in a POS.

OCA-git-bot commented 11 months ago

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

hbrunn commented 11 months ago

/ocabot merge patch

OCA-git-bot commented 11 months ago

Sorry @hbrunn you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

pedrobaeza commented 11 months ago

/ocabot merge patch

OCA-git-bot commented 11 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-150-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 11 months ago

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