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.

Sign up for GitHub

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

Jump to bottom

WireViz container missing ttf-liberation for PNGs #1477

Closed
copyrights opened this issue Mar 18, 2023 · 3 comments · Fixed by #1478
Closed

WireViz container missing ttf-liberation for PNGs #1477

copyrights opened this issue Mar 18, 2023 · 3 comments · Fixed by #1478
Labels
👶 good first issue Good for newcomers 🐞 bug Something isn't working 🐋 docker Related to Docker code 👋 help wanted Help wanted!

Comments

@copyrights
Copy link
Contributor

WireViz container is missing ttf-liberation package
grafik
@copyrights copyrights mentioned this issue Mar 18, 2023
Merged
@copyrights
Copy link
Contributor Author

I'm very sorry that I haven't tested it enough in #1439 😞

@ggrossetie
Copy link
Member

@copyrights Yep, I noticed this issue after releasing 0.20.0. that's why I didn't check png in https://kroki.io/
Feel free to submit a pull request to add the missing font 😄

By default, the font is arial, so liberation font should work fine.

@ggrossetie ggrossetie added 🐞 bug Something isn't working 🐋 docker Related to Docker code 👋 help wanted Help wanted! 👶 good first issue Good for newcomers labels Mar 18, 2023
@copyrights
Copy link
Contributor Author

already summited in #1478

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
👶 good first issue Good for newcomers 🐞 bug Something isn't working 🐋 docker Related to Docker code 👋 help wanted Help wanted!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add missing font package to wireviz container copyrights/kroki
2 participants
@copyrights @ggrossetie