[14.0][ADD] hr_maintenance_workspace

[valentincastravete]

HR Maintenance Workspace

Allows to associate employees and equipments to workspaces.

Table of contents

Implemented Uses

Employees

• Employees can be assigned to workspaces.

Workspaces

• Each workspace has employees and equipment asigned to it. It represents a real workspace.

[mtelahun]

Please see in-line comments about code improvements and automatically generating readme. Also, please add a few tests.
Thank you!

[nimarosa]

Hello @valentincastravete are you going to made the changes suggested in the review?

[valentincastravete]

Hello @valentincastravete are you going to made the changes suggested in the review?

Hi @nimarosa Yes I will do, sorry about my absence.

[JorgeQuinteros]

LGTM Technical Review

[valentincastravete]

@etobella Could you review please?

[etobella]

Just one comment, I would propose to change the name of the module to hr_maintenance_workspace

[valentincastravete]

Just one comment, I would propose to change the name of the module to hr_maintenance_workspace

Module name and its references modified

[valentincastravete]

@nimarosa @mtelahun Could you review please? Thanks!

[valentincastravete]

Hi @angelmoya , can you review and remove the tag "needs fixing" please?

[etobella]

@valentincastravete Can you squash all your commits in a single commit and push force? this will allow us to use runboat too 😄

[valentincastravete]

@valentincastravete Can you squash all your commits in a single commit and push force? this will allow us to use runboat too 😄

Hi, there you go!
@etobella

[etobella]

Do it again please, I just fixed the pre-commit 😢

[valentincastravete]

Do it again please, I just fixed the pre-commit 😢

Done! @etobella

[etobella]

Now you need to fix the precommit from your side, as the error comes from your code 😄

[valentincastravete]

Now you need to fix the precommit from your side, as the error comes from your code 😄

Code fixed 😄

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[valentincastravete]

@etobella
Hi there! Could you please mention someone with merge permission on this PR so it can be finally merged or modified if really needed?
Thanks!

[etobella]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-1115-by-etobella-bump-nobump, awaiting test results.

[OCA-git-bot]

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

[valentincastravete]

@etobella
Thanks a lot!!
Hope this module helps a lot of people.