Skip to content

BXC-5550 on-demand PDF generation service APIs - #2159

Open
krwong wants to merge 37 commits into
mainfrom
bxc-5550
Open

BXC-5550 on-demand PDF generation service APIs#2159
krwong wants to merge 37 commits into
mainfrom
bxc-5550

Conversation

@krwong

@krwong krwong commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

https://unclibrary.atlassian.net/browse/BXC-5550

  • update javadoc and names to "aggregate pdf"
  • add extractTextType method to MachineGeneratedContentService to retrieve the textType
  • update getTextTypes method to use extractTextType
  • add AggregatePdfController
  • add Aggregate PDF action to ResultObjectActionMenu
  • add/update tests and test resources

bbpennel
bbpennel requested changes Jun 9, 2026
View reviewed changes
Comment thread operations/src/main/java/edu/unc/lib/boxc/operations/impl/pdf/AggregatePdfService.java
Comment thread ...solr/src/main/java/edu/unc/lib/boxc/search/solr/services/MachineGeneratedContentService.java Outdated
Comment thread static/js/admin/src/action/AggregatePdfAction.js Outdated
Comment thread static/js/admin/src/ResultObjectActionMenu.js Outdated
Comment thread static/js/admin/src/action/AggregatePdfAction.js Outdated
Comment thread ...ices-app/src/main/java/edu/unc/lib/boxc/web/services/rest/modify/AggregatePdfController.java Outdated
Comment thread ...ices-app/src/main/java/edu/unc/lib/boxc/web/services/rest/modify/AggregatePdfController.java
bbpennel
bbpennel requested changes Jun 15, 2026
View reviewed changes
Comment thread operations/src/test/java/edu/unc/lib/boxc/operations/impl/pdf/AggregatePdfServiceTest.java Outdated
Comment thread static/js/admin/src/action/AggregatePdfAction.js Outdated
bbpennel
bbpennel requested changes Jun 15, 2026
View reviewed changes
Comment thread operations/src/main/java/edu/unc/lib/boxc/operations/impl/pdf/AggregatePdfService.java Outdated
krwong added 14 commits June 15, 2026 15:22
@krwong
have createTextTypeList return list of strings and serialize where ne…
39ae903 
…eded
@krwong
update pdf4u submodule
07f7ce9
@krwong
add PdfRequestSender bean to webapp service-context.xml
cc025c8
@krwong
fix aggregatePdf action
70439a7
@krwong
move aggregatePdf conditional and update permission to editResourceType
01f0de6
@krwong
add =, remove []
ce86189
@krwong
try removing target and adding url and application
2c15df1
@krwong
delete aggregatePdfForm
f235d37
@krwong
fix AggregatePdfAction, add AggregatePdfBatchAction, update controlle…
631aef1 
…r and tests
@krwong
add pdfRequestJmsTemplate bean
115e039
@krwong
move workObject and get storage location with workPid
37cbd18
@krwong
fix tests, add PdfRequestSender and jmsTemplate to service-context.xml
7a30d59
@krwong
add DerivativeService and MachineGeneratedContentService to service-c…
0a8d865 
…ontext.xml
@krwong
extractTextType returns no text if text_type doesn't exist, fix pdf4u…
c33ea40 
… command, write null to transcriptList if transcriptValue doesn't exist, update tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbpennel bbpennel bbpennel requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@krwong @bbpennel