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

Update PDF.js #78

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

anusha-palliyil
Copy link

No description provided.

@rgisiger
Copy link

Could we consider merging this, and also consider even updating to a more recent version, or is this repo not maintained anymore ?

@senny
Copy link
Owner

senny commented Nov 28, 2024

yes, unfortunately, the repo is more or less no longer maintained. I have moved on from the projects that have used this gem and currently do not plan to make further changes.

If someone is willing to continue to maintain and test patches, I am open to sharing access.

@rgisiger
Copy link

I can understand your position, and unfortunately we have to accept this.

I have not the pretention to be able to maintain the Gem and to test patches, but it could be great to find a way to only update the PDF.js version used. Because it's really easy and convenient to just add a Gem for that. Otherwise, I will have to make PDF.js work in a standalone way on my business applications.

@senny
Copy link
Owner

senny commented Nov 29, 2024

@rgisiger as a starting point, you could pin the gem in your project to this PR. However, I can not guarantee that things function as expected and are compatible with your application.

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.

4 participants