search

clarete / hackernews.el

Hacker News client for Emacs
GNU General Public License v3.0
248 stars 26 forks source link

"Bad news, bro: stringp" #15

Closed tbronchain closed 9 years ago

tbronchain commented 9 years ago

Hey, It seems like it's not working, I keep getting the error "Bad news, bro: stringp" while displaying the news "[132]". Hope you're still have time to maintain the project, otherwise I'll try to have a look when I get some spare time. Cheers!

ghost commented 9 years ago

It might be because the api url returns lines with empty strings and null urls, like this.

{"title":"","url":null,"id":8938223,"commentCount":55,"points":387,"postedAgo":"14 hours ago","postedBy":"sinak"},

shassard commented 9 years ago

I've submitted a pull request which changes the backend data source to Hackernews' official REST API over at Firebase. It resolves this issue.

clarete commented 9 years ago

Yup, I just applied that. I'll close this bug but please, feel free to open it again if you guys see this error. Also, I'll change this weird message to something better! Thanks a lot!

tbronchain commented 9 years ago

cool, thanks guys!