search

tkem / mopidy-mobile

Mopidy Web client extension and hybrid app for mobile devices
Apache License 2.0
78 stars 14 forks source link

Limit number of URIs per get_images() request #183

Closed tkem closed 8 years ago

tkem commented 8 years ago

This may be an issue with back ends that don't implement get_images() but rely on the default implementation that uses lookup(). Mopidy-Tidal seems to be especially affected by this, according to @ZenithDK: see also #126.

tkem commented 8 years ago

In addition to requesting images in smaller batch sizes, it would be nice to be able to "cancel" image retrieval, e.g. on $ionicView.leave. Image retrieval may then be moved to $ionicView.enter (confirm that's fired after resolves), and may check against the current controller's local images.

tkem commented 8 years ago

@ZenithDK: When you settled in, it would be nice if you would give MoMo v1.6.11 a try with mopidy-tidal. You'll still see a lot of lookups, but server responsibility and general usability should be much improved (hopefully).

ZenithDK commented 8 years ago

Just tried this out (I'm off sick today, man flu 😑) and it works tons better. Thanks a lot! On 8 Mar 2016 21:07, "Thomas Kemmer" notifications@github.com wrote:

@ZenithDK https://github.com/ZenithDK: When you settled in, it would be nice if you would give MoMo v1.6.11 a try with mopidy-tidal. You'll still see a lot of lookups, but server responsibility and general usability should be much improved (hopefully).

— Reply to this email directly or view it on GitHub https://github.com/tkem/mopidy-mobile/issues/183#issuecomment-193946639.