-
Notifications
You must be signed in to change notification settings - Fork 704
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 Jellyseerr Community Container #5911
Conversation
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks a lot for the contribution!
Looks already pretty good, just some small comments :)
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Thank you for the suggestions! I've accepted them all. It looks like, based on the Jellyseerr reverse proxy docs, the Caddfyile would just need to be modified a bit to add Jellyseerr at jellyseerr.ncdomain.com. I can't figure out where the Caddy reverse proxy is configured in nextcloud AIO for Jellyfin, for example. Would the Caddy container just recognize Jellyseerr automatically if installed? If so, I can add a PR to change the readme for Caddy. If not, what would need to be done to get that working? |
Hi @Anvil5465 you could update this community container https://github.com/szaimen/aio-caddy and add the required config for jellyseerr here: https://github.com/szaimen/aio-caddy/blob/main/start.sh |
Signed-off-by: Anvil5465 <[email protected]>
Co-authored-by: Simon L. <[email protected]> Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
Btw @Anvil5465 i just invited you to the repo for easier collaboration :) |
Signed-off-by: Anvil5465 <[email protected]>
Signed-off-by: Anvil5465 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks a lot!
Thank you again @Anvil5465! :) BTW, we have a list of ideas for additional community containers if you should be interested: #5251 😉 |
Thank you @szaimen for all the help!! I will check those out. |
This is now released with v10.5.0 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel |
This json should create a Jellyseerr community container, per #5901. Please let me know if there is anything else I can do to make this happen.