Skip to content

[ADD] estate module#874

Draft
mgo-adhoc wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-t-1234-mgo
Draft

[ADD] estate module#874
mgo-adhoc wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-t-1234-mgo

Conversation

@mgo-adhoc
Copy link

PR de tarea de Induccion

@roboadhoc
Copy link

Pull request status dashboard

@@ -0,0 +1,6 @@
from . import (

Choose a reason for hiding this comment

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

aca siempre hacemos un import por linea

quedaria como:

from . import estate_property
from . import estate_proerty_offer
from . import estate_property_tag
from , import estate_property_type

else:
record.best_price = 0.0

@api.onchange("garden")

Choose a reason for hiding this comment

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

en V19 ya no usamos el onchange, preferimos siempre el depends, son basicante lo mismo pero uno se ejecuta en el momento(onchage) y el tro al guardar, depends

def _compute_date_deadline(self):
for record in self:
record.date_deadline = (
fields.Date.add(record.create_date, days=record.validity) if record.create_date else False

Choose a reason for hiding this comment

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

aca paa las fechas hay ya unos util que usamos, buscte en el codigo de odoo Relativedelta. es lo que usamos ocmo convencion para las sumas de fechas y cosas

<?xml version='1.0' encoding='utf-8'?>
<odoo>

// Vista Basica por defecto

Choose a reason for hiding this comment

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

cuidado con esto que no es un comentario

Choose a reason for hiding this comment

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

hay varios en este archivo

<?xml version='1.0' encoding='utf-8'?>
<odoo>

// Vista Basica por defecto

Choose a reason for hiding this comment

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

aca tbien esto no es un comentario!

"views/estate_menus.xml",
],
"author": "ADHOC SA",
"license": "LGPL-3",

Choose a reason for hiding this comment

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

licencias usamos siempre AGPL-3

@@ -0,0 +1,15 @@
{

Choose a reason for hiding this comment

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

Lo que te faltaria aca defini por las dudas y que siempre hacemos es el auto-install: en false, application: True en este caso.
installabel: True, y muy importante la version : "19.0.1.0.0" en este caso

@api.depends("offer_ids.price")
def _compute_best_price(self):
for record in self:
if record.offer_ids:

Choose a reason for hiding this comment

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

Este if talvez puede ser un oneliner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants