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

Add option to scan/clean the library #101

Merged
merged 1 commit into from Jul 11, 2014
Merged

Add option to scan/clean the library #101

merged 1 commit into from Jul 11, 2014

Conversation

ghost
Copy link

@ghost ghost commented Jul 11, 2014

Issue #100

@ghost
Copy link
Author

ghost commented Jul 11, 2014

This is a very basic implementation, until I get websockets working and receive a notification when the scan/clean has completed and update the web ui appropriately.

@un1versal
Copy link
Contributor

fantastic, though yes seems that the update stuffs websockts isnt working correctly, hence then maybe the movie sets updates arent updating xwmm then, but thats on another issue. ##88 I think.

un1versal pushed a commit that referenced this pull request Jul 11, 2014
Add rudimentary option to scan/clean the library.
see #101 for whats missing.
@un1versal un1versal merged commit 61404de into slash2009:master Jul 11, 2014
@ghost
Copy link
Author

ghost commented Jul 11, 2014

We don't use websockets at the moment just basic HTTP POST requests which only provide one way communication. We assume that an action has completed, websockest allow us to receive a response from XBMC saying whether or not an action succeeded.

I've got a prototype of the websocket code but it's not usable yet.

@ghost ghost deleted the feature/library-maintenance branch July 11, 2014 15:00
@un1versal
Copy link
Contributor

Patiently sitting at the edge of the seat ;) to see the fyfe magic in action

@ghost ghost mentioned this pull request Jul 12, 2014
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

Successfully merging this pull request may close these issues.

None yet

1 participant