Skip to content

Comments

Functional Training - Estate module#1180

Open
antonio-pimentel wants to merge 9 commits intoodoo:19.0from
odoo-dev:anpim-estate-tutorial
Open

Functional Training - Estate module#1180
antonio-pimentel wants to merge 9 commits intoodoo:19.0from
odoo-dev:anpim-estate-tutorial

Conversation

@antonio-pimentel
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Feb 17, 2026

Pull request status dashboard

@blse-odoo blse-odoo self-requested a review February 17, 2026 08:15
Copy link

@blse-odoo blse-odoo left a comment

Choose a reason for hiding this comment

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

This is a good start!
Regarding the commit's messages, did you have a look at the git guidelines? (It's a bit of a detail, but it's good to get familiar with the process)

Comment on lines 10 to 13
date_availability = fields.Date(
string='Available From',
copy=False,
default=fields.Date.add(fields.Date.today(), months=+3))

Choose a reason for hiding this comment

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

The declaration of the fields is only read once when the server starts. Here the default value will be computed just once the day the server is started, not every time a record is created.
If you starts the server on the 01/01, but create a new record on the 17/02, the default will be the value computed at start up, thus 01/04.
You can pass a lambda to default, and it will be computed on each record creation

@blse-odoo
Copy link

Just a small detail about the message of your last commit: the [ADD] tag is for new modules, I believe the [IMP] tag is more appropriate in this case (link to the guidelines)

@antonio-pimentel antonio-pimentel changed the title [Add] Functional Training - Estate module ch 4 [Add] Functional Training - Estate module Feb 19, 2026
@antonio-pimentel antonio-pimentel changed the title [Add] Functional Training - Estate module Functional Training - Estate module Feb 19, 2026
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