Skip to content

Conversation

@AungKoKoLin1997
Copy link
Contributor

Forward port of #1202

This PR adds an improvement to restrict the applying changes only for some specific records by adding domain in configuration.
With this changes, we can only adjust the content of report pdf depends on some condition, not globally.

@qrtl QT5088

…g some specific records in report pdfs

Co-authored-by: Yoshi Tashiro <[email protected]>
@OCA-git-bot
Copy link
Contributor

Hi @yostashiro,
some modules you are maintaining are being modified, check this out!

Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

Could we add tests, or create a test module?

<field name="name" />
<field name="report_id" />
<field name="template_id" readonly="report_id != False" />
<field name="report_model" invisible="1" />
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<field name="report_model" invisible="1" />
<field name="report_model" column_invisible="1" />

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