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

add Kavita #684

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

add Kavita #684

wants to merge 4 commits into from

Conversation

eniad
Copy link
Contributor

@eniad eniad commented Nov 19, 2023

What this PR does / why we need it:

Adds Kavita, a digital library which supports a vast array of file formats

Which issue (if any) this PR fixes:

Fixes #683

Any other useful info:

It's a good media server with some quirks. It handles large libraries.

why can't I ever catch these things before I submit a PR?
Copy link
Owner

@davestephens davestephens left a comment

Choose a reason for hiding this comment

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

Looks good, minor issue in the Traefik config then we're good to merge!

roles/kavita/tasks/main.yml Outdated Show resolved Hide resolved
@eniad
Copy link
Contributor Author

eniad commented Jan 21, 2024

The developer included a notification to use their dockerhub image (https://hub.docker.com/r/jvmilazz0/kavita). Is there a strong preference for ansible-nas to use the linuxserver image (https://hub.docker.com/r/linuxserver/kavita)?

The notification pop up said something along the lines of being "massively out of date". I suspect the intent was to move users off an old image that hasn't been updated in 3 years. However, the linuxserver image is up-to-date.

I dismissed the notification before reading thoroughly and a cursory search through the kavite github repo didn't find what I was looking for.

@davestephens
Copy link
Owner

davestephens commented Jan 21, 2024 via email

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.

Add kavita
2 participants