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

update notifiers #832

Closed
wants to merge 2 commits into from
Closed

update notifiers #832

wants to merge 2 commits into from

Conversation

thezoggy
Copy link
Contributor

  • Safeguard notifiers from affecting the post-processing routine, catch and log exceptions.
  • Instead of calling each notifier update_library routine individually, we now just call all of them similar to the notifications
  • Cleaned up notifier ui page wording so they follow similar pattern.
  • Converted updateXBMC to use show id rather than show name (works the same way as refreshShow/updateShow/etc)
  • Better exception handling for pyTivo and NMJ
  • Cleaned up logging entries, trying to standardize on what level we report and verbiage (not using contractions, prune out duplicate/excessive entries)
  • Added notification support for Synology (notifies Synology DSM) with migration of old setting
  • Dropped getURL helper reference since importing helper is more trouble than its worth, will just rewrite each notifier using Requests when we move to python 2.6+

thezoggy added 2 commits May 12, 2014 17:13
@thezoggy
Safeguard notifiers from affecting the post-processing routine, catch…
437a8a9 
… and log exceptions.
@thezoggy
* Instead of calling each notifier update_library routine individuall…
3da6178 
…y, we now just call all of them similar to the notifications

* Cleaned up notifier ui page wording so they follow similar pattern.
* Converted updateXBMC to use show id rather than show name (works the same way as refreshShow/updateShow/etc)
* Better exception handling for pyTivo and NMJ
* Cleaned up logging entries, trying to standardize on what level we report and verbiage (not using contractions, prune out duplicate/excessive entries)
* Added notification support for Synology (notifies Synology DSM) with migration of old setting
* Dropped getURL helper reference since importing helper is more trouble than its worth, will just rewrite each notifier using Requests when we move to python 2.6+
@ghost ghost closed this in 0d7110b May 20, 2014
@thezoggy thezoggy deleted the dev--update_notifiers3 branch June 21, 2014 12:36
gborri pushed a commit to gborri/Sick-Beard that referenced this pull request Sep 30, 2014
@adam111316
Merge pull request midgetspy#832 from seiki/master
c94da47 
fix transmission seed time
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@thezoggy