Added ability to read downloaded status from Radarr #468
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
version.txt is still at 1.20.2 when it should be 1.20.3
Bumped up version number to reflect latest release.
As well restores the ability to create movie requests even when CouchPotato or Radarr aren't available.
# Conflicts: # version.txt
Merging in changes from Master. Need to do further testing.
Turned off logging for SyncedCron (had it turned on for testing). Changed radarr timeout for response back to 15000.
|
Let me look at this a bit, I actually started reworking download status to
be a single method that will spur a check to the enabled clients and return
a unified response. I was about to start working on Radarr so I may
integrate your check rather than reinvent the wheel.
…On Mon, Apr 24, 2017, 3:52 PM Omar Sandoval ***@***.***> wrote:
This is to resolve issue #452
<#452>. I tested it
on my own installation with great results. Not sure if there are certain
guidelines to follow but I tried to implement this as a hybrid of
CouchPotato and Sonarr. Please take a look and let me know what you think.
------------------------------
You can view, comment on, or merge this pull request online at:
#468
Commit Summary
- Bumped up version number to reflect latest release.
- Merge pull request #1 from ohmar/dev-testing
- Bump version number
- Fix adding movies to CouchPotato
- Fix indentation
- Merge remote-tracking branch 'refs/remotes/lokenx/master'
- Fix update.txt
- Added the ability to read downloaded status from Radarr.
- Reverting to avoid future issues.
- Merge pull request #3 from ohmar/master
- Bumped version number.
File Changes
- *M* lib/exports/radarrExports.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-0>
(8)
- *M* server/methods/admin/version.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-1>
(2)
- *M* server/methods/cronjobs/updateRadarr.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-2>
(5)
- *M* server/methods/helpers/movies/radarrMethods.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-3>
(318)
- *M* server/methods/search/requestMovie.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-4>
(138)
- *M* server/startup/cronjobs.js
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-5>
(1)
- *M* version.txt
<https://github.com/lokenx/plexrequests-meteor/pull/468/files#diff-6>
(2)
Patch Links:
- https://github.com/lokenx/plexrequests-meteor/pull/468.patch
- https://github.com/lokenx/plexrequests-meteor/pull/468.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#468>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AQbPDT8PQGkWxWc_1AYBxTQ7hpr-uWmdks5rzQubgaJpZM4NGsoC>
.
|
|
For some reason my formatting makes it seem like I changed much more than I really did. Here are the main changes; added radarrMovieStatus, updated updadeRadarr and updated cronjobs. |
|
Awesome, I'll probably be working on it tonight or tomorrow.
…On Mon, Apr 24, 2017, 4:49 PM Omar Sandoval ***@***.***> wrote:
For some reason my formatting makes it seem like I changed much more than
I really did. Here are the main changes; added radarrMovieStatus, updated
updadeRadarr and updated cronjobs.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#468 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AQbPDcf78owEy8VHGX3nG3xsqNWKGLabks5rzRjagaJpZM4NGsoC>
.
|
|
Any updates on this? |
|
Sorry for the delay, I haven't been able to dive into this like I wanted to so I will be merging this into dev-testing as is since it looks good to go as is. |
|
I understand that "available" status from radarr is not currently working, right? Just making sure because mine is not. Want to be sure I don't have something set wrong on my end. Running radarr and plexrequests via the unraid docker containers. |
|
Available status from Radarr is working in dev-testing but not master. |
RickyGrassmuck
pushed a commit
that referenced
this pull request
Sep 27, 2017
* Bumped up version number to reflect latest release. version.txt is still at 1.20.2 when it should be 1.20.3 * Bump version number * Fix adding movies to CouchPotato As well restores the ability to create movie requests even when CouchPotato or Radarr aren't available. * Fix indentation * Fix update.txt * Added the ability to read downloaded status from Radarr. * Reverting to avoid future issues. * Bumped version number. Turned off logging for SyncedCron (had it turned on for testing). Changed radarr timeout for response back to 15000.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is to resolve issue #452. I tested it on my own installation with great results. Not sure if there are certain guidelines to follow but I tried to implement this as a hybrid of CouchPotato and Sonarr. Please take a look and let me know what you think.