It can be useful to have a printable version of a claim notice, for testing purposes or for when supplier has no email address for example
Created attachment 64261 [details] [review] Bug 18796: Allow to print notice while claiming serials It can be useful to have a printable version of a claim notice, for testing purposes or for when supplier has no email address for example Test plan: 1. Go to Serials > Claims and select a vendor 2. Select some serials by clicking on checkboxes 3. Click to the little arrow next to 'Send notification' button and click on 'Print notification' 4. Verify that the downloaded PDF contains the content of the selected notice 5. Verify that the 'Send notification' button still works as before 6. Run `prove t/db_dependent/Letters.t`
Created attachment 64263 [details] [review] Bug 18796: Allow to print notice while claiming serials It can be useful to have a printable version of a claim notice, for testing purposes or for when supplier has no email address for example Test plan: 1. Go to Serials > Claims and select a vendor 2. Select some serials by clicking on checkboxes 3. Click to the little arrow next to 'Send notification' button and click on 'Print notification' 4. Verify that the downloaded PDF contains the content of the selected notice 5. Verify that the 'Send notification' button still works as before 6. Run `prove t/db_dependent/Letters.t` Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Rebase looks easy ;)
ah JavaScript should be at bottom ;)
Needs a huge rebase :D
(In reply to Fridolin Somers from comment #3) > Rebase looks easy ;) (In reply to Fridolin Somers from comment #5) > Needs a huge rebase :D What two years of changes can do... :)
Created attachment 124988 [details] [review] Bug 18796: Allow to print notice while claiming serials It can be useful to have a printable version of a claim notice, for testing purposes or for when supplier has no email address for example Test plan: 1. Go to Serials > Claims and select a vendor 2. Select some serials by clicking on checkboxes 3. Click to the little arrow next to 'Send notification' button and click on 'Print notification' 4. Verify that the downloaded PDF contains the content of the selected notice 5. Verify that the 'Send notification' button still works as before 6. Run `prove t/db_dependent/Letters.t` Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Patch rebased on master
Hi Julian, I've tried adding some basic formatting, but in the output it comes out as: Printtest <b>Print</b> <div style="background:red"> test </div> So there are no line breaks for the div element and the bold formatting is also lost. Can you tell if I am doing something wrong there?
Ping?