human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

Add optional signature line to distribution printout

Open cielf opened this issue 1 year ago • 4 comments

Summary

Some (but not all) banks use the distribution printout as a receipt, requiring a signature from the partner on pickup or delivery. Add that at the end of the printout, optionally

Details

The flag for inclusion should be on an organization level, default to false

The added lines should be (adjust to fit the 8.5 x 11 page): Received by: (Print Name) _____________________________ (Signature and Date) ________________________________ Delivered by: (Print Name) _____________________________ (Signature and Date) ________________________________

Note: One bank is using the comments as a workaround for this.

Criteria for completion:

  • [ ] Option to add signature lines to distribution printout (receipt) added to organization - editable and showing.
  • [ ] That option controls the inclusion of the signature lines on the distribution printout
  • [ ] Tests to support the above functionality
  • [ ] Provide a screenshot in the PR

cielf avatar Apr 28 '24 14:04 cielf

I'd be happy to take this one

sean-dickinson avatar May 06 '24 18:05 sean-dickinson

You got it!

dorner avatar May 06 '24 18:05 dorner

@dorner I'm curious to hear what you were thinking in terms of testing this functionality. It doesn't look like there's anything currently set up for testing this pdf generation or for any other similar flag functionality on the organization. I'd be happy to test it out, but I'm not sure the best way.

sean-dickinson avatar May 06 '24 20:05 sean-dickinson

Good question. Most of the functionality here is visual, which tests wouldn't be able to show. I think a minimal test that shows that the result generates, without crashing, and without any specific expectations on the output, should be fine. If we don't have that, it'd be a good idea to add it for this.

dorner avatar May 07 '24 00:05 dorner