-
-
Notifications
You must be signed in to change notification settings - Fork 456
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
Overseer with lidarr support #3800
base: develop
Are you sure you want to change the base?
Conversation
) Co-authored-by: Loetwiek <[email protected]>
* docs: update README.md * docs: update .all-contributorsrc --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
2e7f050
to
078c8b0
Compare
I just rebased all the commits, it's all in your hands now ! |
…ich made the build not work
Added some fields I missed in the migration file and fixed some image display issue
…nly a <Link> Ported Links to work for the new React version installed
…maps to await them I changed mapMovieResult and other equivalents to return Promises and forgot to fix some API endpoints. So that's fixed now.
brilliant, hope this gets approved |
I'll repost here what I said on discord:
And also:
|
Just fixed this |
Fixed this too |
Since I was previously using an outdated version of Lidarr, my API calls were not functionnal with the latest versions I updated and fixed them. I've had to simply not set `album.artist.addOptions.monitor` at all and it works... fix #9
183e221
to
c105f10
Compare
The Lidarr API should now work as expected (at least to add stuff, the scanning part hasn't changed) |
Pushed the latest commit to docker |
!!! |
Tremendous work here! Is there any specific areas of testing that you'd like to give extra attention? |
I haven't fixed the lidarr scan and the UI requesting elements from lidarr directly so not those. But requesting should work for both artists and releases, artist are not always marked as available though. I also don't really know how I should make the search work, rn I'm not satisfied with how it works. I think there isn't anything specifically but if you find new bugs feel free to report them. |
56af105
to
6c07d22
Compare
Good afternoon. I want to see how the integration with ldap works, I'm trying to run docker. It gives an error when starting. What should I do to launch the docker container?
|
What does your docker compose look like ? |
|
You haven't mapped anything to /app |
yarn run v1.22.19 I've mapped all the folders. The error about missing files is still preserved. |
I had the same problem and switching from bind mount to docker volumes fixed it. It appears that for whatever reason (permissions?) this is one that just won't work correctly with bind mounts unless you want to pre-download everything into your mount folder first. I had the same issue with the emqx docker too and just gave up on fighting that one after many days of fighting it. |
Pushed a new build to docker with some fixes |
First of all thanks for all the job you done @ano0002.
|
|
I am new to the whole docker ecosystem. Currently using Unraid. I did the docker pull and then docker run. How do I get the apps linked so music shows up in Overseerr? Any help would be greatly appreciated! |
While I appreciate you pointed to the solution, alot of us here are clueless to how to mount to docker volumes. I ask chatGPT for help and this solved the issue: version: '3' volumes: |
Hello, where are we with the final integration to overseer? @sct |
Merge this already!!!! |
Description
Implemented Lidarr Support into overseerr
Here is a sample
docker-compose.yml
for it :