Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@max3903
Copy link
Member

@max3903 max3903 commented Oct 5, 2025

/ocabot migration pms_base

@max3903 max3903 added this to the 18.0 milestone Oct 5, 2025
@max3903 max3903 self-assigned this Oct 5, 2025
@max3903
Copy link
Member Author

max3903 commented Oct 5, 2025

/ocabot migration pms_base

@OCA-git-bot OCA-git-bot mentioned this pull request Oct 5, 2025
1 task
@max3903 max3903 force-pushed the 18.0-mig-pms_base branch 7 times, most recently from 62d12a8 to 9ecf4bc Compare October 5, 2025 23:10
@max3903 max3903 force-pushed the 18.0-mig-pms_base branch from 9ecf4bc to 311bf3e Compare October 6, 2025 02:53
Copy link
Member

@mymage mymage left a comment

Choose a reason for hiding this comment

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

Functional OK

Copy link
Member

@bealdav bealdav left a comment

Choose a reason for hiding this comment

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

Just tested, LGTM

One comment

"Children Count", compute="_compute_childs_property"
)
floors_num = fields.Integer(string="Floor")
unit_floor = fields.Integer()
Copy link
Member

@bealdav bealdav Oct 17, 2025

Choose a reason for hiding this comment

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

Suggestion: could you add an help attribute here ? I don't understand unit_floor.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants