-
-
Notifications
You must be signed in to change notification settings - Fork 795
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
[ADD] [16.0] report_display_name_in_footer #897
[ADD] [16.0] report_display_name_in_footer #897
Conversation
a36a15d
to
718b320
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼 Functional review
Hello @HviorForgeFlow @lmignon @alexis-via , Please, could you review this PR 🙏🏼 ! is easy, intelligent and simple! 😄 Thank you! ❤️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful module! Very good solution to the problem.
It would be nice to have an option per report, in stock.picking you have several reports. Maybe in a future module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one. LGTM (code review)
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 0335d02. Thanks a lot for contributing to OCA. ❤️ |
Adds document name in report PDF footers.
Can configured via parameteres (see CONFIGURE.md)
MT-5999 @moduon @yajo @rafaelbn @EmilioPascual @fcvalgar @Gelojr please review if you want :)