[ADD] Server Action: Create Record Set Default Value#35
[ADD] Server Action: Create Record Set Default Value#35samirGuesmi wants to merge 1 commit intoOCA:18.0from
Conversation
9d2a3c0 to
460e02c
Compare
460e02c to
f570f4d
Compare
etobella
left a comment
There was a problem hiding this comment.
This repository is mainly used for automation_oca module and the related modules of it. Shouldn't this module go to server-tools or server-ux?
It has no direct relation with automation_oca module.
|
@etobella server action are automated actions I asked @samirGuesmi to target this repo as it is closer in mind than generic other repo like To me this extends server actions and |
|
Well, automation_oca generates a way to automate things (not only server actions, emails, activities....). By themselves, server actions do not automate anything, they are just snippets of code that can be launched by an automation or a user. |
|
In our context we created this module for automation_oca, thus it felt the right place to go, server-tools being a mixup of things I prefer to target specific repositories. If you refuse it we will move it to server-tools, please give us your choice so we can move forward with this. |
|
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. |
etobella
left a comment
There was a problem hiding this comment.
Although it is not clear which would be the right place to be, the code is Ok.
In standard Odoo, when creating a new record through Server Actions, often relies on name_create(self.value), which only fills in the name field. This is limiting in scenarios where multiple fields need to be pre-populated or dynamically computed.