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

remove wkhtmltopdf from readme #179

Merged
merged 2 commits into from
Jul 11, 2023
Merged

remove wkhtmltopdf from readme #179

merged 2 commits into from
Jul 11, 2023

Conversation

drizuid
Copy link
Member

@drizuid drizuid commented Jul 6, 2023

alpine dropped support for wkhtmltopdf but we left it in the readme as a method to render pdf. There may be some users working on a PR with weasyprint, but no reply lately from them. this PR simply removes the notes in the readme for pdf rendering.

ref: #134

@drizuid drizuid changed the title Drizuid patch 1 remove wkhtmltopdf from readme Jul 6, 2023
@LinuxServer-CI
Copy link
Collaborator

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/bookstack/v23.06.1-pkg-b8fa2e31-dev-edd48f8708f452ef8ace3ecb25d05eff11c2e4ca-pr-179/index.html
https://ci-tests.linuxserver.io/lspipepr/bookstack/v23.06.1-pkg-b8fa2e31-dev-edd48f8708f452ef8ace3ecb25d05eff11c2e4ca-pr-179/shellcheck-result.xml

Tag Passed
amd64-v23.06.1-pkg-b8fa2e31-dev-edd48f8708f452ef8ace3ecb25d05eff11c2e4ca-pr-179
arm64v8-v23.06.1-pkg-b8fa2e31-dev-edd48f8708f452ef8ace3ecb25d05eff11c2e4ca-pr-179

@thespad thespad merged commit 2e802e7 into master Jul 11, 2023
5 checks passed
@thespad thespad deleted the drizuid-patch-1 branch July 11, 2023 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants