-
Notifications
You must be signed in to change notification settings - Fork 12
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
Client side playlists. #5
Comments
I guess it allows clients to make playlists too! |
Yes it does. However the changes seem to reflect for everyone. That's pretty weird. |
No no it doesn't allow us to make client side playlists! It allows the clients to make playlist which is stored in the main database. |
Hmm.. okay. So we need to setup a lightweight database client-side right? |
@mukkachaitanya I think that is what @prampey meant by saying that. We can use Preferences instead of using a db, because, given db's location, there is a possibility, someone can fiddle/corrupt it. My two cents. |
@SebastinSanty That's a node module. We can't use that client-side. From what I see, there seem to be 2 ways to go:
|
First one might require a lot of code changes and we should be able to identify users properly. |
@mukkachaitanya Are you still doing this? I might want to give a shot at this! Thanks :) |
@SebastinSanty Try it out. Do tell if you need any help :D |
Current app only allows the admin to make playlists and these playlists sync on the client side too. One would also want to allow the client to make his own client playlists, which can be stored locally on the client side.
The text was updated successfully, but these errors were encountered: