Skip to content

feat: upload placeholder interlinear export PDF#209

Open
delgado-jacob wants to merge 7 commits intoglobalbibletools:mainfrom
delgado-jacob:pdf_export/upload_placeholder
Open

feat: upload placeholder interlinear export PDF#209
delgado-jacob wants to merge 7 commits intoglobalbibletools:mainfrom
delgado-jacob:pdf_export/upload_placeholder

Conversation

@delgado-jacob
Copy link
Copy Markdown
Contributor

Split from: #134

  • Adds ExportStorageRepository and S3 client helpers for export storage
  • Uploads placeholder interlinear PDFs to S3 from the export job
  • Adds S3-related env config for exports

@arrocke
Copy link
Copy Markdown
Member

arrocke commented Mar 31, 2026

@delgado-jacob Just as a heads up, I just merged a PR to switch to tanstack. It doesn't affect this PR accept for conflicts in package.json and package-lock.json, but if you build more UI, it will be relevant. Let me know when you are ready for a review here.

@delgado-jacob delgado-jacob force-pushed the pdf_export/upload_placeholder branch from ca0fce9 to a1481f7 Compare April 5, 2026 00:30
@delgado-jacob
Copy link
Copy Markdown
Contributor Author

@delgado-jacob Just as a heads up, I just merged a PR to switch to tanstack. It doesn't affect this PR accept for conflicts in package.json and package-lock.json, but if you build more UI, it will be relevant. Let me know when you are ready for a review here.

@arrocke I rebased this onto the latest main and it should be ready for review. I have one final branch of changes stacked against this one for splitting #134. I'll probably need to review and update that one to make sure it's consistent with the new tanstack changes before submitting.

@delgado-jacob delgado-jacob requested a review from arrocke April 11, 2026 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants