Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[16.0][ADD] report_alternative_layout #45

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997 AungKoKoLin1997 commented Oct 24, 2024

This module introduces an alternative layout for QWeb reports with the following
adjustments to existing external layouts:

  • Displays the partner's address on the left side first, followed by the company's
    address on the right side.
  • Uses the commercial partner's address in place of the partner's address.

@qrtl QT4915

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-report_alternaltive_layout branch from b1e0230 to e6ca413 Compare October 24, 2024 04:50
Comment on lines 6 to 8
- Show Commercial Partner: If selected, the commercial partner of the document partner
will show in the report output (instead of the document partner).
- Set the Paper Format field to 'A4 - no company address in header.'
Copy link
Member

Choose a reason for hiding this comment

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

To be consistent with style, and also to make it in the right order (important ones first).

Suggested change
- Show Commercial Partner: If selected, the commercial partner of the document partner
will show in the report output (instead of the document partner).
- Set the Paper Format field to 'A4 - no company address in header.'
- Paper Format: Select 'A4 - no company address in header' if Apply Alternative Layout is selected.
- Show Commercial Partner: If selected, the commercial partner of the document partner
will show in the report output (instead of the document partner).

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-report_alternaltive_layout branch from e6ca413 to b2263de Compare October 28, 2024 10:29
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.

Code review and functional test.

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.

2 participants