Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PI-1558 Update PSR extension to PDF when complete #2384

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

marcus-bcl
Copy link
Contributor

Previously, an empty PDF would be uploaded to the Alfresco when the initial draft was started. Following the recent changes, Delius now uploads a .doc file instead as the draft. However, the filename remains the same after the report is completed.

So we end up with a PDF document saved as e.g.

Nat Short format pre-sentence report_13102023_112036_Larson_D_X729742.doc

This PR updates the extension to .pdf before saving

Previously, an empty PDF would be uploaded to the Alfresco when the initial draft was started.  Following the recent changes, Delius now uploads a .doc file instead as the draft.  However, the filename remains the same after the report is completed.

So currently we end up with a PDF document saved as e.g.
```
Nat Short format pre-sentence report_13102023_112036_Larson_D_X729742.doc
```

This PR updates the extension to .pdf before saving
@marcus-bcl marcus-bcl requested a review from a team as a code owner October 13, 2023 11:03
@github-actions github-actions bot added the bug Something isn't working label Oct 13, 2023
@marcus-bcl marcus-bcl merged commit ced3a3e into main Oct 13, 2023
35 checks passed
@marcus-bcl marcus-bcl deleted the PI-1558-psr-extension branch October 13, 2023 14:38
marcus-bcl added a commit that referenced this pull request Oct 16, 2023
marcus-bcl added a commit that referenced this pull request Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants