vext01 / hgd

Hackathon Gunther Daemon :: Opensource Multi-User Jukebox
http://hgd.theunixzoo.co.uk
ISC License
22 stars 10 forks source link

Improve voted? #116

Closed jonentropy closed 12 years ago

jonentropy commented 12 years ago

It would be a lot easier for clients if the ls command also returned if the user had voted for the track instead of calling a seperate command.

vext01 commented 12 years ago

On Fri, Oct 07, 2011 at 08:14:50AM -0700, Tristan Linnell wrote:

It would be a lot easier for clients if the ls command also returned if the user had voted for the track instead of calling a seperate command.

What separate command? There is no way to get this information currently.

Best Regards Edd Barrett

http://www.theunixzoo.co.uk

jonentropy commented 12 years ago

There is. Id does it. On Oct 7, 2011 4:18 PM, "Edd Barrett" < reply@reply.github.com> wrote:

On Fri, Oct 07, 2011 at 08:14:50AM -0700, Tristan Linnell wrote:

It would be a lot easier for clients if the ls command also returned if the user had voted for the track instead of calling a seperate command.

What separate command? There is no way to get this information currently.

Best Regards Edd Barrett

http://www.theunixzoo.co.uk

Reply to this email directly or view it on GitHub: https://github.com/vext01/hgd/issues/116#issuecomment-2322810

vext01 commented 12 years ago

On Fri, Oct 07, 2011 at 08:19:49AM -0700, Tristan Linnell wrote:

There is. Id does it.

No, Id only tells you about your own vote.

Best Regards Edd Barrett

http://www.theunixzoo.co.uk

jonentropy commented 12 years ago

Exactly. Would be great if ls could tell you that. On Oct 7, 2011 4:21 PM, "Edd Barrett" < reply@reply.github.com> wrote:

On Fri, Oct 07, 2011 at 08:19:49AM -0700, Tristan Linnell wrote:

There is. Id does it.

No, Id only tells you about your own vote.

Best Regards Edd Barrett

http://www.theunixzoo.co.uk

Reply to this email directly or view it on GitHub: https://github.com/vext01/hgd/issues/116#issuecomment-2322841

egelmex commented 12 years ago

sound like it would be usefull, will expect a diff.

jonentropy commented 12 years ago

This is fixed now @vext01 has implemented code from my patch