Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

PDF export fails randomly #913

Closed
3 of 5 tasks
lukas-schardt opened this issue Oct 25, 2018 · 3 comments · Fixed by #1195
Closed
3 of 5 tasks

PDF export fails randomly #913

lukas-schardt opened this issue Oct 25, 2018 · 3 comments · Fixed by #1195
Assignees
@kesselb
Labels
api All Backend related Issues bug This Issue describes a unwanted behavior

Comments

@lukas-schardt
Copy link
Collaborator

lukas-schardt commented Oct 25, 2018 •
edited by kesselb
Loading

User Story

As a:

user of the GELI platform

I want:

to export the course as PDF

so that:

Acceptance criteria:

  • Display download button again
  • Rename temp file to be unique
  • Fix pdf generation and download
  • Use populate (if possible) to remove localLecture and localUnit
  • Invalidate cache when course / lecture / unit is edited

Additional info:
@kesselb
Copy link
Contributor

kesselb commented Oct 25, 2018 •
edited
Loading

https://sentry.io/share/issue/230478b5826443daba938c0d0c7b0506/ (i guess we run into problems when mutliple users create pdf export at the same time)

@dboschm dboschm mentioned this issue Oct 25, 2018
Merged
dboschm pushed a commit that referenced this issue Oct 25, 2018
Merge branch 'develop' into bugfix/#913-Pdf-Export
1f3a0bf
kesselb added a commit that referenced this issue Oct 25, 2018
@kesselb
Merge branch 'develop' into bugfix/#913-Pdf-Export
20cd5c7
kesselb added a commit that referenced this issue Oct 25, 2018
@kesselb
Merge remote-tracking branch 'origin/bugfix/#913-Pdf-Export' into bug…
bd03bcc 
…fix/#913-Pdf-Export
@kesselb kesselb added bug This Issue describes a unwanted behavior api All Backend related Issues labels Oct 25, 2018
@kesselb kesselb changed the title BUG: Fix PDF Export PDF export fails randomly Oct 25, 2018
dboschm pushed a commit that referenced this issue Oct 25, 2018
Merge branch 'develop' into bugfix/#913-Pdf-Export
43aba38
dboschm pushed a commit that referenced this issue Oct 26, 2018
Merge branch 'develop' into bugfix/#913-Pdf-Export
e5eaefd
@dboschm dboschm mentioned this issue Oct 26, 2018
Open
2 tasks
@kesselb kesselb mentioned this issue Oct 29, 2018
Merged
@dboschm
Copy link
Collaborator

dboschm commented Nov 1, 2018

Can this be closed? PR #919 is already merged.

@kesselb
Copy link
Contributor

kesselb commented Nov 1, 2018 •
edited
Loading

I added two other points to acceptance criteria today. I guess we should keep this ticket until we solved most issues with pdf export. But you can unassign yourself if your part is done.

@dboschm dboschm removed their assignment Nov 1, 2018
@torss torss mentioned this issue Nov 6, 2018
Open
2 tasks
@kesselb kesselb self-assigned this Mar 22, 2019
@kesselb kesselb mentioned this issue Mar 22, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kesselb kesselb

Labels
api All Backend related Issues bug This Issue describes a unwanted behavior
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove file cache from DownloadController geli-lms/geli
3 participants
@kesselb @lukas-schardt @dboschm