-
-
Notifications
You must be signed in to change notification settings - Fork 247
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] operating_unit_custom_header #646
[ADD] operating_unit_custom_header #646
Conversation
1e315d0
to
355f134
Compare
This is partially duplicating functionality of https://github.com/OCA/operating-unit/tree/14.0/report_qweb_operating_unit perhaps we can discuss if that module needs to be improved or be more generic |
Hello,
The specifications I had, was that the user could change the header and
footer for each operating unit, not following the same template. That what
this module is doing.
Thanks
Cécile Jallais
Le jeu. 18 janv. 2024, à 12 h 12, Aaron ForgeFlow ***@***.***>
a écrit :
… This is partially duplicating functionality of
https://github.com/OCA/operating-unit/tree/14.0/report_qweb_operating_unit
perhaps we can discuss if that module needs to be improved or be more
generic
—
Reply to this email directly, view it on GitHub
<#646 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKE4U3XIWFGZWMCJSV5QHBDYPFJWTAVCNFSM6AAAAABB7F2BK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJYHA4DSMBXGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
<https://arche-ti.ca>
<https://www.archeti.com/>
<https://www.facebook.com/TIArche> <https://twitter.com/tiarche>
<https://twitter.com/tiarche> <http://goog_48537287>
<https://www.linkedin.com/company/arche-ti>
AVIS DE
CONFIDENTIALITÉ Ce message peut contenir de l'information
légalement
privilégiée ou confidentielle. Si vous n'êtes pas le
destinataire ou
croyez avoir reçu par erreur ce message, nous vous
saurions gré d'en
aviser l'émetteur et d'en détruire le contenu sans le
communiquer à
d'autres ou le reproduire. À moins d'avis contraire de sa
part, le
destinataire autorise l'utilisation du courriel non sécurisé
pour la
transmission de renseignements confidentiels.
CONFIDENTIALITY NOTICE This
communication may contain privileged or
confidential information. If you
are not the intended recipient or
received this communication by error,
please notify the sender and
delete the message without copying or
disclosing it. Unless explicitly
indicated, the receiver hereby authorizes
the use of unsecured and
unencrypted e-mail for the future transmission of
confidential
information.
|
@cjallais thats fine. I think this module is more flexible than the report_qweb_operating_unit module, which could be a reason for your module to take over from the other one. However, for current instances that uses report_qweb_operating_unit the transition to the new module seems not easy, as the default header and footer in the new module is different from the one in report_qweb_operating_unit module. |
I guess you could keep the 2 in version 14, as they have different
functionality and keep just one in version 15.
Le mar. 23 janv. 2024, à 05 h 02, Aaron ForgeFlow ***@***.***>
a écrit :
… @cjallais <https://github.com/cjallais> thats fine. I think this module
is more flexible than the report_qweb_operating_unit module, which could be
a reason for your module to take over from the other one. However, for
current instances that uses report_qweb_operating_unit the transition to
the new module seems not easy, as the default header and footer in the new
module is different from the one in report_qweb_operating_unit module.
—
Reply to this email directly, view it on GitHub
<#646 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKE4U3VEIYDUESRYEHSYG23YP6DERAVCNFSM6AAAAABB7F2BK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBVG4YDCNRVGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
<https://arche-ti.ca>
<https://www.archeti.com/>
<https://www.facebook.com/TIArche> <https://twitter.com/tiarche>
<https://twitter.com/tiarche> <http://goog_48537287>
<https://www.linkedin.com/company/arche-ti>
AVIS DE
CONFIDENTIALITÉ Ce message peut contenir de l'information
légalement
privilégiée ou confidentielle. Si vous n'êtes pas le
destinataire ou
croyez avoir reçu par erreur ce message, nous vous
saurions gré d'en
aviser l'émetteur et d'en détruire le contenu sans le
communiquer à
d'autres ou le reproduire. À moins d'avis contraire de sa
part, le
destinataire autorise l'utilisation du courriel non sécurisé
pour la
transmission de renseignements confidentiels.
CONFIDENTIALITY NOTICE This
communication may contain privileged or
confidential information. If you
are not the intended recipient or
received this communication by error,
please notify the sender and
delete the message without copying or
disclosing it. Unless explicitly
indicated, the receiver hereby authorizes
the use of unsecured and
unencrypted e-mail for the future transmission of
confidential
information.
|
report_qweb_operating_unit is also in v15. I think this module could take over in v16. For this version it is difficult, as long as no duplicate features should be allowed in OCA. |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
This PR adds a new module to add report header and footer by operating unit instead of company: