caspergasper / Gready-Android

Android app for http://www.goodreads.com/
11 stars 4 forks source link

Long list of bookshelves appears to be cut off #8

Closed Cait closed 14 years ago

Cait commented 14 years ago

I have a lot of bookshelves. The popup listing of bookshelves appears to cut off about halfway down, with the last shelf showing the name "null" (it is actually named "series-forgotten-realms"). If I click on the "(18) null" shelf, the "series-forgotten-realms" shelf opens up with the correct name and all 18 books showing.

My shelves: http://www.goodreads.com/review/list/1005037-cait

(2010-06-22 build)

caspergasper commented 14 years ago

Bloody hell! This is not a bug but a feature, reminding you to stop reading and turn on the tv instead :-)

Part of the problem is I'm only downloading the first 100 shelves --

http://www.goodreads.com/shelf/list?format=xml&key=UvPjrkah6sJXg88qs75xRA&user_id=1005037

For the next release I can test to make sure they all appear.

Casper.

Cait commented 14 years ago

;)

Actually, it may be both a bug and a feature, since the series-forgotten-realms shelf isn't the hundredth -- that would be series-shannara. (Must they all end in embarrassing teenaged reading? My book shame, let me show you it....)

caspergasper commented 14 years ago

Sadly another bug I can't reproduce -- when I load your booklist series-shannara is the last entry. Are you using 2.1?

Cait commented 14 years ago

I rebooted my phone and I can't reproduce this now -- perhaps the download got cut off previously and it wouldn't refresh?

Yes, I think so -- it just happened again at a different point in my list of shelves, and when I hit "home" and reopened the application, the full list of shelves displayed.

caspergasper commented 14 years ago

Cool, I've just spent the whole day working out why authentication doesn't work with Android 2.2 so I could do with some good news! I'll leave this open until > 100 bookshelves can be opened.

caspergasper commented 14 years ago

100 bookshelves should be fixed in today's release (20100703), can you please test? Thanks.

Cait commented 14 years ago

All of my bookshelves are showing up in the list using the new build.

caspergasper commented 14 years ago

Great, I'll close this.