Additional product info on invoice and packing slip.

I would like to add some additional information to the invoice and the packing slip. I was thinking of adding additional notifiers to do this. Is it OK to just submit a pr to zencart git with the additional notifiers coded?
I understand that this does not necessarily guarantee they will be accepted.

You are welcome to submit a PR. The core committers are busy right now so PR merges are slower than normal, but your submission will be reviewed.

Just noticed the admin\invoice.php and admin\packingslip.php have moved on since v1.5.7c, do you want me to apply my changes to the latest version? 1.5.8

For info it appears that $show_product_images is not set in invoice.php any more version 1.5.8!

Pull request #4447 created against v.1.5.8

brittainmark:

Just noticed the admin\invoice.php and admin\packingslip.php have moved on since v1.5.7c, do you want me to apply my changes to the latest version? 1.5.8

For info it appears that $show_product_images is not set in invoice.php any more version 1.5.8!

you are obviously free to do what you want.

i have renamed both of those scripts so that i can customize to my clients delight, and not have to worry about all kinds of things. you can then use the encapsulated plugins without having those name clashes that you previously posted about.

i believe there are notifiers that one can use to override the invoice and packing slip buttons used when viewing an order, so that those buttons then link to your new customized scripts.

just an idea.

best.

brittainmark:

For info it appears that $show_product_images is not set in invoice.php any more version 1.5.8!

That might be a bug. Not sure.

*** Update: I am pretty sure it was a merge error. I have submitted a PR to restore the initialization of the two set flags.

carlwhat:

you are obviously free to do what you want.

i have renamed both of those scripts so that i can customize to my clients delight, and not have to worry about all kinds of things. you can then use the encapsulated plugins without having those name clashes that you previously posted about.

i believe there are notifiers that one can use to override the invoice and packing slip buttons used when viewing an order, so that those buttons then link to your new customized scripts.

just an idea.

best.

found the notifer. NOTIFY_ADMIN_ORDERS_MENU_BUTTONS

will look to see what happens with pr