Skip to content

Add a "Export PDF" button to Arches reports - #8

Merged
samuelscandrett merged 11 commits into
dev/1.0.xfrom
sds/1_report_print_button
Aug 27, 2026
Merged

Add a "Export PDF" button to Arches reports#8
samuelscandrett merged 11 commits into
dev/1.0.xfrom
sds/1_report_print_button

Conversation

@samuelscandrett

Copy link
Copy Markdown
Member

Closes #1

Rather than replacing the existing print functionality, as this would require copying and modifying a chunk of core code, I have opted for adding a report footer containing a button that calls the search export url for a single resource. This does require copying the core viewmodels/report.js file, but the file itself is small and has been extended with our function.

@samuelscandrett samuelscandrett self-assigned this Aug 21, 2026

@CWDamm-Kint CWDamm-Kint left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I can definitely see the value of making the Core Arches UI more modularised, including the search export options and the top nav bar, but adding to the footer seems a good solution for now.

Comment thread arches_pdf_exporting/media/js/viewmodels/report.js Outdated
Comment thread arches_pdf_exporting/media/js/viewmodels/report.js Outdated
Comment thread arches_pdf_exporting/media/js/viewmodels/report.js Outdated
Comment thread arches_pdf_exporting/templates/views/report-templates/default.htm
@CWDamm-Kint

Copy link
Copy Markdown
Collaborator

Optional, but maybe it should be indented?

image

@CWDamm-Kint CWDamm-Kint left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@samuelscandrett
samuelscandrett merged commit 8667715 into dev/1.0.x Aug 27, 2026
6 checks passed
@samuelscandrett
samuelscandrett deleted the sds/1_report_print_button branch August 27, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace existing Arches report print button + functionality with HTML export for the one resource

2 participants