Skip to content

[IMP] dms: Add Category/Directory Code - #525

Open
SirPyTech wants to merge 2 commits into
OCA:18.0from
PyTech-SRL:18.0-imp-dms-add_code_complete_name
Open

SirPyTech wants to merge 2 commits into
OCA:18.0from
PyTech-SRL:18.0-imp-dms-add_code_complete_name

Conversation

@SirPyTech

Copy link
Copy Markdown
Contributor

Add a "Code" field to categories and directories, that is used to compute their complete name as [code] name.
If the new code field is left empty, the behavior remains as it was before this PR.

Comment thread dms/models/directory.py
Comment thread dms/views/dms_category.xml
Comment thread dms/views/dms_directory.xml
Comment thread dms/views/dms_category.xml
@SirPyTech
SirPyTech marked this pull request as ready for review October 14, 2026 08:38
@SirPyTech
SirPyTech force-pushed the 18.0-imp-dms-add_code_complete_name branch from a2c04ca to 5dc05fc Compare October 14, 2026 13:19

@HekkiMelody HekkiMelody left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code and functional review, LGTM

Comment thread dms/models/directory.py
@SirPyTech
SirPyTech force-pushed the 18.0-imp-dms-add_code_complete_name branch from 5dc05fc to 10e777f Compare October 15, 2026 13:52
@SirPyTech
SirPyTech requested a review from HekkiMelody October 15, 2026 13:54

@anusriNPS anusriNPS left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code and Functional Review: LGTM

@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

@PicchiSeba PicchiSeba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review: LGTM

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.

5 participants