Generate PDF report. Use custom font.
Created by: zwippie
Merge request reports
Activity
2 def response_email(invitation_sent_at:, requester_email:, show_secret:) 2 def response_email(invitation_sent_at:, requester_email:, response:) 3 3 @invitation_sent_at = invitation_sent_at 4 @link = show_response_url(showSecret: show_secret) 4 @response = response 5 @link = show_response_url(showSecret: response.show_secret) 6 7 attachments.inline['Rapport-CATja-Screening.pdf'] = render_report 5 8 6 9 mail to: requester_email, 7 10 subject: 'Resultaten invulling' 8 11 end 12 13 private 14 15 def render_report Created by: timraasveld
Misschien is het in de nieuwe situatie die deze PR creërt simpeler om als argument aan
response_email
de hele response mee te geven i.p.v. alleen deshow_secret
. Dan hoeft er geen logic om de response op te zoeken in ResponseMailer.Slechts een idee, take it or leave it :)
Created by: timraasveld
Nieuw voorbeeld voor de reviewer Standaard font.pdf (oudere commit) Dejavu Sans font.pdf
Created by: zwippie
Voorbeeld PDF: test.pdf
Created by: zwippie
@timraasveld Je suggesties mbt rubocop, rapport titel en table header zijn uitgevoerd.
Created by: zwippie
Voorbeeld met table header: test.pdf
Please register or sign in to reply