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

install pdf.js and fits through builder images #1119

Merged
merged 4 commits into from
Apr 19, 2024
Merged

install pdf.js and fits through builder images #1119

merged 4 commits into from
Apr 19, 2024

Conversation

rococodogs
Copy link
Member

@rococodogs rococodogs commented Apr 4, 2024

basically #1107 without the docker compose watch additions (which were causing increased rebuild times). updates PDF.js to 4.0.379.

closes #1042

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 126902 lines exceeds the maximum allowed for the inline comments feature.

@rococodogs rococodogs mentioned this pull request Apr 4, 2024
Copy link

codeclimate bot commented Apr 5, 2024

Code Climate has analyzed commit c60a480 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (100% is the threshold).

This pull request will bring the total coverage in the repository to 98.5% (0.0% change).

View more on Code Climate.

@rococodogs rococodogs merged commit f39c88a into develop Apr 19, 2024
7 checks passed
@rococodogs rococodogs deleted the pdfjs-2 branch April 19, 2024 18:24
@rococodogs rococodogs mentioned this pull request Apr 19, 2024
rococodogs added a commit that referenced this pull request May 8, 2024
* ignore deprecation warnings in prod environments (#1121)

* bump .ruby-version to match container (#1120)

* install pdf.js and fits through builder images (#1119)

* remove tracked pdfjs files

* install pdfjs + fits through builder containers

* fix specs

* smol refactor

* allow authenticated users to deposit honors theses (#1126)

* allow all registered users to deposit student_works

* rubo

* just use first admin_set as default

* 🙃

* changelog for 2024.6

* log_level should be a symbol
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.

1 participant