OCA / rma

Odoo for Return Merchandise Authorization (RMA)
GNU Affero General Public License v3.0
79 stars 205 forks source link

[14.0][ADD] rma_employee #393

Closed FrankC013 closed 1 month ago

FrankC013 commented 6 months ago

Creation of a new module to add the employee field within the RMA model

FrankC013 commented 6 months ago

@eantones Please review!

pedrobaeza commented 6 months ago

Which is the goal of using employees instead of users? Isn't this module a bit stripped without no extra features except the field?

FrankC013 commented 6 months ago

The objective of this module is to allow the user to assign an RMA to an employee. It's a purely informational field. It's created in a separate module to avoid the installation of HR when someone installs the RMA.

pedrobaeza commented 6 months ago

But I insist: why not assigning a user instead?

github-actions[bot] commented 2 months ago

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.