I would caution against that unless you want to use it on multiple transaction types. You don't want to inadvertently break transfer orders by making a change meant only for sales orders. As it is I'm thinking about moving cashSalePrintCustomButton to its own cashSaleModule.js that's deployed on only cash sales