Skip to content

[19.0][MIG] hr_timesheet_amount_security#938

Open
EmilioPascual wants to merge 2 commits into
OCA:19.0from
moduon:19.0-mig-hr_timesheet_amount_security
Open

[19.0][MIG] hr_timesheet_amount_security#938
EmilioPascual wants to merge 2 commits into
OCA:19.0from
moduon:19.0-mig-hr_timesheet_amount_security

Conversation

@EmilioPascual

Copy link
Copy Markdown
Contributor

@fcvalgar @Gelojr @chienandalu please review. Thank you

MT-15011 @moduon

@EmilioPascual EmilioPascual force-pushed the 19.0-mig-hr_timesheet_amount_security branch from 6939419 to 8dfb2ae Compare June 16, 2026 12:37
@OCA-git-bot OCA-git-bot added series:19.0 mod:hr_timesheet_amount_security Module hr_timesheet_amount_security labels Jun 16, 2026

@chienandalu chienandalu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review 👍

@ArantxaSudon ArantxaSudon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revisión funcional

Comprobado que si se le quita a un usuario el permiso Allow to read analytic costs estos ya no pueden verse.

Image

Gracias @EmilioPascual @moduon MT-15011

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@rafaelbn rafaelbn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Funcional test 👍🏼

/ocabot migration hr_timesheet_amount_security

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Sorry @rafaelbn you are not allowed to mark the addon to be migrated.

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.

@rafaelbn

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Sorry @rafaelbn 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.

@rafaelbn

Copy link
Copy Markdown
Member

Please @OCA/human-resources-maintainers , we are maintainters of this module, could please merge? Thank you! ❤️

@tafaRU

tafaRU commented Jun 25, 2026

Copy link
Copy Markdown
Member

Hi @rafaelbn please try again, you should be able to do it yourself now that #912 has been merged.

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.

7 participants