[MIG] okr_management: Migration to 17.0 #165
Conversation
Task: 34208
mem-adhoc
force-pushed
the
17.0-t-34208-mem-okr
branch
3 times, most recently
from
e93c565 to
33939b5
Compare
mem-adhoc
force-pushed
the
17.0-t-34208-mem-okr
branch
from
33939b5 to
0afab6e
Compare
| from odoo.exceptions import UserError | ||
|
|
||
| class OkrKeyResult(models.Model): | ||
| _name = 'okr.key.result' |
There was a problem hiding this comment.
usar "." cuando es un submodelo, "_" cuando es parte del nombre (por ej "key_result")
|
|
||
| name = fields.Char() | ||
| description = fields.Text() | ||
| objective =fields.Text() |
There was a problem hiding this comment.
El objective, deberia ser un objective_id relacionado al modelo okr.objective
| from odoo.exceptions import UserError | ||
|
|
||
| class OkrManagement(models.Model): | ||
| _name = 'okr.management' |
There was a problem hiding this comment.
Esto tendira que ser el modelo okr.objective
| progress= fields.Integer( | ||
| compute='_compute_okr_progress', | ||
| store = False, | ||
| ) |
There was a problem hiding this comment.
Aca son necesarios 3 campos:
Target que es el puntaje a obtener (esta mas abajo, eso esta bien)
Resultado (es el resultado parcial, este falta)
Progress esta bien y es computado pero creo que lo estas usando como el campo anterior (Resultado)
| importance = fields.Integer()#importancia dentro del kr | ||
| target = fields.Integer()#objetivo esperado | ||
|
|
||
| @api.depends('peso', 'target') |
There was a problem hiding this comment.
Faltan dependencias a resultado (campo que no existe)
La dependdencia a peso esta bien pero el campo no existe
| def _compute_okr_progress(self): | ||
| #mejorar codigo | ||
| prog = 0 | ||
| for rec in self: | ||
| if rec.importance and rec.target: | ||
| prog = rec.importance |
There was a problem hiding this comment.
Revisar formula del excel para implementar aca
| ('inspiracional', 'Inspiracional'), | ||
| ], | ||
| ) | ||
| result = fields.Float() |
There was a problem hiding this comment.
Result deberia ser computado y tener en cuenta el progreso que tienen los key result
| 'license': 'AGPL-3', | ||
| 'images': [ | ||
| ], | ||
| 'depends': [ |
There was a problem hiding this comment.
Agregar dependencia minimo a modulo 'base'
| ], | ||
| 'installable': True, | ||
| 'auto_install': False, | ||
| 'application': False, |
Task: 34208