Skip to content
Snippets Groups Projects

Generate PDF report. Use custom font.

Closed automation_daan requested to merge am-generate-pdf-report into master

Created by: zwippie

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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 de show_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: jornvandebeek

    De tekst in de pdf doet mij wat groot aan, hoe ziet het er bij 12 punten uit?

  • 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

  • Created by: timraasveld

    LGTM

  • Please register or sign in to reply
    Loading