Skip to content

[ADD] estate: add a new addon estate, which add a basic model for real estate properties #774

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 21 commits into
base: 18.0
Choose a base branch
from

Conversation

legi-odoo
Copy link

This should be good 👍

@robodoo
Copy link

robodoo commented May 19, 2025

Pull request status dashboard

Copy link

@mathcoutant mathcoutant left a comment

Choose a reason for hiding this comment

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

The commits are not named properly.

Otherwise, it's great !



class Property(models.Model):
_name = "estate_property"

Choose a reason for hiding this comment

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

You should use dots instead of underscores for the model names. (Sorry for the bad info previously).

Suggested change
_name = "estate_property"
_name = "estate.property"

@legi-odoo legi-odoo changed the title chapter 1 to 4 [ADD] estate: add a new addon estate, which add a basic model for real estate properties May 20, 2025
@legi-odoo legi-odoo force-pushed the 18.0-tutorial-legi branch from 40bcb60 to ab90ae2 Compare May 20, 2025 12:08
@legi-odoo legi-odoo force-pushed the 18.0-tutorial-legi branch from ab90ae2 to 401c9d7 Compare May 20, 2025 12:12
Copy link

@mathcoutant mathcoutant left a comment

Choose a reason for hiding this comment

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

Small comments

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants