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

Any chance to get maraschino working with Sickrage api? #363

Open
slugshell opened this issue May 25, 2014 · 29 comments
Open

Any chance to get maraschino working with Sickrage api? #363

slugshell opened this issue May 25, 2014 · 29 comments

Comments

@slugshell
Copy link

Hello there, first of all, i love maraschino, lovely interface, you did a great job!

Here my problem,
lately I was forced to skip sickbeard and change to a fork called sickrage, only because I need a working databse like tvrage, since thetvdb sadly seems to have difficulties accomplishing this, for example (season order - american dad).

Problem i have now, sickrage seems to use a different api, please have a look here https://github.com/echel0n/SickRage/blob/master/sickbeard/webapi.py

If i try to use the original sickbeard webapi.py with sickrage i get:
2014-05-25 14:41:53 CP Server Thread-4 :: DB error: no such column: tv_shows.tvdb_id
2014-05-25 14:41:53 CP Server Thread-4 :: API :: no such column: tv_shows.tvdb_id

Is there any chance for a quick fix?

Thank you very much,

greetings,

slugshell

@mboeru
Copy link

mboeru commented Aug 26, 2014

I'm facing the same problem. Unfortunately I do not see any errors in the marachino log files.
If I try to access for example http://192.168.1.xxx:7000/xhr/sickbeard/get_show_info/79349/ I get "Internal Server Error" and nothing to show for it in the log file.

Would there be any way to enable more debugging so I can see the actual error in the log file?

Many Thanks,
mboeru

@RoostayFish
Copy link

Here's my fork with v0.1 sickrage integration. https://github.com/RoostayFish/maraschino/tree/sickrage

Please test and submit bugs and ill get it running smoothly :)

@mboeru
Copy link

mboeru commented Aug 27, 2014

Cool! will test tonight. Many Thanks!

@mboeru
Copy link

mboeru commented Aug 28, 2014

Hi,

I have tested with your branch but It still seems not to be working, and no errors in the maraschino log.
Should I see a different module when adding module to the page? I am not seeing any Sickrage module so I tried with the SickBeard module
Image

This is what I have:

[root@cubox maraschino]# git remote -v
origin  https://github.com/RoostayFish/maraschino.git (fetch)
origin  https://github.com/RoostayFish/maraschino.git (push)
[root@cubox maraschino]# git branch
  master
* sickrage

And I used a clean database.

@RoostayFish
Copy link

Hmm, could i get your other OS setup info?

Ill spin up a VM and test.

On mine it appears as a new module under Sickbeard.

The logging is almost non existent currently, thats one of the things im working on adding.

@mboeru
Copy link

mboeru commented Aug 28, 2014

I'm a a CuBox-i4Pro hardware, arm processor quad core, 2G ram, with fedora 20.

If you need anything else let me know. I will try this also on my main computer, with osx

@RoostayFish
Copy link

Im developing on osx, so i know it works on that :)

Im not sure ill get a chance to do it today but please keep me uptodate with how your getting on.

@mboeru
Copy link

mboeru commented Aug 28, 2014

I think it has something to do with my use of git and getting that branch. I have downloaded the branch zipped and now I can see the module.

Connected to my sickrage and it worked, after I refreshed the maraschino page. Probably a glitch that does not load it after I press save or something?
Also another thing I am seeing, if I set the webroot to / it does not load, if it's empty it load great.

Do you think there is any chance this will get added to the main repo?

Cheers!

@RoostayFish
Copy link

Im working from the https://github.com/gugahoi/maraschino repo for the plex support, but it shouldnt be hard to merger here with some small modifications.

Once its finished that is 👍

@mboeru
Copy link

mboeru commented Aug 28, 2014

Cool, I'll let you know if I find any other issues with it

@N3MIS15
Copy link
Collaborator

N3MIS15 commented Aug 28, 2014

Please, when done send a PR and I will review it. I didn't even know sickrage existed until now..

@mboeru
Copy link

mboeru commented Aug 28, 2014

Yeah, There was a sickbeard fork by MrOrange I think that had better torrent support. That closed down and SickRage stepped up in it's place with both tvrage support, but also the torrent stuff.

FYI: Tried using my maraschino.db with the sickrage fork and I get Internal Server Error with nothing logged in the logfile.

@gugahoi
Copy link
Collaborator

gugahoi commented Aug 28, 2014

@RoostayFish: I have found a couple of bugs but I am not sure if it is stuff you just havent implemented yet or if they are really bugs. Might be worth enabling the 'Issues' tab on you repository so we can help track everything. Basic stuff worked fine for me tho.

@RoostayFish
Copy link

Issues tab is now enabled.

@mboeru Ill be working on the logging next to make it easier for debugging

@mboeru
Copy link

mboeru commented Aug 29, 2014

That's great, let me know when to test again.

@neoTUGA
Copy link

neoTUGA commented Oct 30, 2014

No XBMC support on the SickRage fork :(

@RoostayFish
Copy link

Hey neoTuga,

Sorry I didn't get the time to get this merged with the main fork. Had a
new work project thats taking 150% of my time.

I was going to get this polished and merged in the next few months but I
cant promise any dates :(

On 30 October 2014 04:22, neoTUGA [email protected] wrote:

No XBMC support on the SickRage fork :(


Reply to this email directly or view it on GitHub
#363 (comment)
.

@mboeru
Copy link

mboeru commented Dec 23, 2014

Hi @RoostayFish,

I have just tested and it still works great! Do you think you have some time to clean it up and do a PR so this will be in main repo?

Happy Holidays!

@dgwingert
Copy link

Hi all,

I'm new to development, so I hope I'm not stepping on anybody's toes. I took the work done by @RoostayFish and worked it into the version of maraschino with xbmc/kodi support. Can somebody (maybe @mboeru or @neoTUGA ?) let me know if it works for them? It is currently working for me, but I'm new to github and coding as well.

Here is my repo. Make sure to try the sickrage fork.
https://github.com/dgwingert/maraschino/tree/sickrage

Thanks,
dgwingert

@Hellowlol
Copy link

I dont think your stepping on any toes maraschino development has been halted for quite some time. You should change the seachtvdb api command, as it only search tvdb. I think there is one called search indexers or something.

@mboeru
Copy link

mboeru commented Mar 5, 2015

Hi,

I can test tomorrow, I will report back here if I find anything. Many thanks for picking this up.

@Hellowlol
Copy link

@dgwingert See https://github.com/Hellowlol/HTPC-Manager/blob/master2/modules/sickrage.py#L210 for a example to search on tvdb and tvrage for a show at the same time.

@neoTUGA
Copy link

neoTUGA commented Mar 6, 2015

@dgwingert not working for me :(
KODI is fine, but not connecting to sickrage or utorrent... :(

@mboeru
Copy link

mboeru commented Mar 6, 2015

Just tested and the sickrage widget does not seem to show anything. And there is nothing about it in the log either.

@dgwingert
Copy link

Ok, I checked my work more carefully this time. I pushed the new changes to the repo. (https://github.com/dgwingert/maraschino). I downloaded it to another computer and managed to run and connect to sickrage. Let me know if your experience is different of course. @Hellowlol , I'll look into the searchtvdbapi when I get the chance. Thanks for the help.

Oh and @neoTUGA , I have no idea how I might have broken uTorrent, but I'll try to check it at some point. Any info in the log?

@ebright
Copy link

ebright commented Mar 7, 2015

Hey guys. I am also working on adding to maraschino, but I am working on the trakt 2.0 api changes. in the meantime tho, check out this comment ebright@7819479 to add plot details to the sickrage module

@mboeru
Copy link

mboeru commented Mar 7, 2015

@dgwingert Works well on for me :)
I still have the trakt issue that @ebright fixed in #383

@dgwingert
Copy link

I suppose the best thing would be for me to merge my branch with somebody who is actively developing, since my contribution was kind-of a one-off. How should I go about making that happen?

@hadees
Copy link

hadees commented Mar 13, 2015

@dgwingert you need to make a pull requst

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

No branches or pull requests

10 participants