OCA / operating-unit

Manage Operating Units in Odoo
GNU Affero General Public License v3.0
72 stars 238 forks source link

[ADD] operating_unit_custom_header #646

Closed cjallais closed 2 months ago

cjallais commented 8 months ago

This PR adds a new module to add report header and footer by operating unit instead of company:

AaronHForgeFlow commented 8 months ago

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

cjallais commented 8 months ago

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 https://github.com/OCA/operating-unit/pull/646#issuecomment-1898889074, 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.

AaronHForgeFlow commented 8 months ago

@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.

cjallais commented 8 months ago

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 https://github.com/OCA/operating-unit/pull/646#issuecomment-1905701650, 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.

AaronHForgeFlow commented 7 months ago

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.

github-actions[bot] commented 3 months ago

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. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.