CYMO - Technical training 19.0 by MrRose765 · Pull Request #1172 · odoo/tutorials

MrRose765 · 2026-02-16T13:19:19Z

No description provided.

robodoo · 2026-02-16T13:19:21Z

…er 3 - Server framework 101

ltinel

Good job :) A few "FYIs", but nothing to say really

estate/models/estate_property.py

estate/__manifest__.py

Chapter 4 - Security

ltinel

Great work 😄

FYI, if you scroll to the bottom of your PR, you'll find some Runbot links, among which the ci/style check. In there, you'll find a few styling suggestions (typically incorrect use of whitespace or newlines). Could you implement those to make the check pass?

estate/models/estate_property.py

MrRose765 · 2026-02-17T12:57:37Z

Thank you for your feedback !

FYI, if you scroll to the bottom of your PR, you'll find some Runbot links, among which the ci/style check. In there, you'll find a few styling suggestions (typically incorrect use of whitespace or newlines). Could you implement those to make the check pass?

I don't have access to the runbot at the moment 😔. Sysadmins told me that I have to check with HR... I will try to change asap to pass the checks.

Chapter 5 - Finally, Some UI To Play With

Chapter 6 - Basic views

Chapter 7 - Relations Between Models

ltinel

Good job :)

estate/models/estate_property.py

estate/models/estate_property_offer.py

estate/views/estate_property_views.xml

estate/views/res_users_views.xml

estate_account/models/estate_property.py

Chapter 8 - Computed Fields and Onchanges

…se offer. Chapter 9 - Ready For Some Action ?

Chapter 10 - Constraints

estate/models/estate_property.py

estate/views/estate_property_offer_views.xml

estate/models/inherited_model.py

- Add Python constraint to ensure selling price is acceptable relative to expected price. - Add SQL constraints for unique property names and positive prices. - Add decorations (colors) to the property list view based on status. - Add 'Offers' stat button to the property form view. - Set default ordering for property types and tags. Chapter 11 - Add The Sprinkles

Chapter 12 - Inheritance

Chapter 13 - Interact With Other Modules

Chapter 14 - A brief history of QWeb

ltinel

Great work! 🤩

awesome_owl/static/src/card/card.js

awesome_owl/static/src/todo_list/todo_item.xml

awesome_owl/static/src/todo_list/todo_item.js

Discover the web framework - Chapter 1 - Owl components

Tutorial - Define module data

Tutorials - Restrict access to data

Tutorial - Safeguard your code with unit tests

ltinel · 2026-02-27T10:15:54Z

estate/security/estate_security.xml

+        <field name="model_id" ref="model_estate_property"/>
+        <field name="groups" eval="[(4, ref('estate_group_user'))]"/>
+        <field name="domain_force">[
+            '|', ('salesman_id', '=', user.id), ('salesman_id', '=', False)


Nit:

Suggested change

'|', ('salesman_id', '=', user.id), ('salesman_id', '=', False)

('salesman_id', 'in', [user.id, False])

ltinel · 2026-02-27T10:16:52Z

estate/security/estate_security.xml

+            '|', ('company_id', '=', False),
+                ('company_id', 'in', company_ids)


Nit:

Suggested change

'|', ('company_id', '=', False),

('company_id', 'in', company_ids)

('company_id', 'in', company_ids + [False])

ltinel · 2026-02-27T10:18:08Z

estate/security/estate_security.xml

+        <field name="name">Estate Property: see all </field> 
+        <field name="model_id" ref="model_estate_property"/> 
+        <field name="groups" eval="[(4, ref('estate_group_manager'))]"/> 
+        <field name="domain_force">[(1,'=',1)]</field> 


Not needed :)

Suggested change

<field name="domain_force">[(1,'=',1)]</field>

ltinel · 2026-02-27T10:18:53Z

estate/security/estate_security.xml

+    <record id="estate_group_manager" model="res.groups">
+        <field name="name">Manager</field>
+        <field name="privilege_id" ref="groups_privilege_real_estate"/>
+        <field name="implied_ids" eval="[(4, ref('estate_group_user'))]"/>


Nit: (same below)

Suggested change

<field name="implied_ids" eval="[(4, ref('estate_group_user'))]"/>

<field name="implied_ids" eval="[Command.link(ref('estate_group_user'))]"/>

ltinel · 2026-02-27T10:48:13Z

estate/tests/test_estate_property.py

+
+        with self.assertRaises(UserError, msg="You cannot sell a property without an accepted offer."):
+            property_no_offer.action_set_sold()
+            property_refused_offer_only.action_set_sold()


This line should be in a separate with self.assertRaises(, otherwise it'll never be called.

ltinel · 2026-02-27T12:14:27Z

estate/tests/test_estate_property.py

+            property_no_offer.action_set_sold()
+            property_refused_offer_only.action_set_sold()
+
+        def test_garden_onchange(self):


The indentation seems a bit off here

[ADD] estate: Create Real Estate module manifest

18ca504

MrRose765 changed the title ~~CYMO - Technical training 2026~~ CYMO - Technical training 19.0 Feb 16, 2026

[IMP] estate: create estate_property model add basic fields. \n Chapt…

427542d

…er 3 - Server framework 101

ltinel self-requested a review February 16, 2026 15:19

ltinel reviewed Feb 16, 2026

View reviewed changes

estate/models/estate_property.py Outdated Show resolved Hide resolved

estate/__manifest__.py Show resolved Hide resolved

estate/__manifest__.py Outdated Show resolved Hide resolved

estate/__manifest__.py Outdated Show resolved Hide resolved

[IMP] estate: Create access rights for estate property.

a1468e3

Chapter 4 - Security

MrRose765 force-pushed the 19.0-training-cymo branch from 73a4aef to 2aa5be2 Compare February 17, 2026 09:58

ltinel reviewed Feb 17, 2026

View reviewed changes

estate/models/estate_property.py Show resolved Hide resolved

MrRose765 added 2 commits February 17, 2026 16:31

[IMP] estate: Default menus and form view for estate properties.

cd2072a

Chapter 5 - Finally, Some UI To Play With

[IMP] estate: Add list,form and search views for estate properties

808607a

Chapter 6 - Basic views

MrRose765 force-pushed the 19.0-training-cymo branch from 407eb5d to d732a5b Compare February 17, 2026 15:32

[IMP] estate: Add property types, tags and offers.

36b521e

Chapter 7 - Relations Between Models

MrRose765 force-pushed the 19.0-training-cymo branch 3 times, most recently from 6b43021 to 8cd5978 Compare February 19, 2026 08:24

ltinel reviewed Feb 19, 2026

View reviewed changes

MrRose765 added 3 commits February 19, 2026 14:53

[IMP] estate: Add total_area, best_price and validity date.

cce5435

Chapter 8 - Computed Fields and Onchanges

[IMP] estate: Add possibility to sold/cancel property and accept/refu…

ad97a22

…se offer. Chapter 9 - Ready For Some Action ?

[IMP] estate: Add constraints on prices and tag/property names.

140606d

Chapter 10 - Constraints

MrRose765 force-pushed the 19.0-training-cymo branch from 8cd5978 to 041421a Compare February 19, 2026 19:33

ltinel reviewed Feb 20, 2026

View reviewed changes

estate/models/estate_property.py Outdated Show resolved Hide resolved

estate/views/estate_property_offer_views.xml Outdated Show resolved Hide resolved

ltinel reviewed Feb 20, 2026

View reviewed changes

estate/models/inherited_model.py Show resolved Hide resolved

MrRose765 force-pushed the 19.0-training-cymo branch from 041421a to 5856ab1 Compare February 20, 2026 10:51

MrRose765 added 3 commits February 20, 2026 11:54

[IMP] estate: link users to properties and protect deletion

3b94c34

Chapter 12 - Inheritance

[ADD] estate_account: Add creation of invoice when a property is sold.

88da1d4

Chapter 13 - Interact With Other Modules

[IMP] estate: Add kanban view for estate properties.

9231b81

Chapter 14 - A brief history of QWeb

MrRose765 force-pushed the 19.0-training-cymo branch from 5856ab1 to 9231b81 Compare February 20, 2026 10:54

ltinel reviewed Feb 23, 2026

View reviewed changes

awesome_owl/static/src/card/card.js Outdated Show resolved Hide resolved

awesome_owl/static/src/todo_list/todo_item.xml Outdated Show resolved Hide resolved

awesome_owl/static/src/todo_list/todo_item.js Outdated Show resolved Hide resolved

[IMP] awesome_owl: Add counter and todolists components

f65151f

Discover the web framework - Chapter 1 - Owl components

MrRose765 force-pushed the 19.0-training-cymo branch from 47e5510 to f65151f Compare February 23, 2026 13:28

[IMP] estate: introduce standard and demo data

81d376b

Tutorial - Define module data

MrRose765 force-pushed the 19.0-training-cymo branch from 83305bd to a850825 Compare February 25, 2026 10:17

MrRose765 added 2 commits February 25, 2026 13:28

[IMP] estate: add managers and user groups with their rules.

a6b1799

Tutorials - Restrict access to data

[IMP] estate: Add constraints and tests on estate properties.

46c3f0a

Tutorial - Safeguard your code with unit tests

MrRose765 force-pushed the 19.0-training-cymo branch from 2e88cf5 to 46c3f0a Compare February 25, 2026 12:28

ltinel reviewed Feb 27, 2026

View reviewed changes

	'\|', ('salesman_id', '=', user.id), ('salesman_id', '=', False)
	('salesman_id', 'in', [user.id, False])

		'\|', ('company_id', '=', False),
		('company_id', 'in', company_ids)

	'\|', ('company_id', '=', False),
	('company_id', 'in', company_ids)
	('company_id', 'in', company_ids + [False])

	<field name="implied_ids" eval="[(4, ref('estate_group_user'))]"/>
	<field name="implied_ids" eval="[Command.link(ref('estate_group_user'))]"/>

Conversation

MrRose765 commented Feb 16, 2026

Uh oh!

robodoo commented Feb 16, 2026

Uh oh!

ltinel left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ltinel left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

MrRose765 commented Feb 17, 2026

Uh oh!

ltinel left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ltinel left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

ltinel Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants