[wishlist] Google Reader API (Feature Request)

[Grillo]

I already implemented this in a branch, I would love if it could be merged.

This implementation gets the last feed titles on witch you are suscribed in Google Reader and differs the ones that are unread.

I've been using it since the last commit without any problems, please tell me your opinions

[Dominic Evans (oldmanuk)]

I had been watching your branch already, and will test the final implementation and merge it as soon as possible. Just haven't had enough time to do so yet.

[Dominic Evans (oldmanuk)]

I have now tested this branch. It works well displaying the stories in the messages feed and detailing which are read vs unread.

However, the 'read' links on each article seem to currently take you to the /reading-list page on google rather than directly to the article's original location, which I wasn't expecting the behaviour to be. Is it possible to change this to take you directly to the article whilst also marking the item as read in google reader?

[Grillo (novalkar)]

there is an API call to manage that :), i'll work on it

[Dominic Evans (oldmanuk)]

One other thing. There is currently a private bug that will need to be fixed before this merge can take place. Ryan has suggested that he expects to have it complete by this weekend, I will update the bug with this info when I have it.

[Dominic Evans (oldmanuk)]

"there is an API call to manage that :), i'll work on it"

Cool, many thanks for your work on this btw.

[Grillo (novalkar)]

I just updated the branch, please tell me if you like the way it's implemented

"Cool, many thanks for your work on this btw."

Thanks for the feedback, I learned a lot by reading the code (I learned Python a couple of weeks ago too) and I thought it would be nice to implement this functionality

[Grillo (novalkar)]

I kinda broke my branch, im going to clean it up and push it back up

[Grillo (novalkar)]

Now it's fully working, I updated it so it merges with the Gnome Keyring Update (tested it too)

[Billie Thompson (billiecodes)]

Hi,

This patch should fix a few issues with this feature.

* It removes the (NEW) from the titles
* gets around gwibbers issue of ignoring URL when deciding if to stack message or not
* Adds a profile image
* Adds profile link to website that originally created post

Will

[Billie Thompson (billiecodes)]

Balls, left a print statement in there

[Billie Thompson (billiecodes)]

Possibly slightly better than the previous patch, as it doesn't break the semantics to message.text.

[Billie Thompson (billiecodes)]

Possibly slightly better than the previous patch, as it doesn't break the semantics of message.text.

[Fernando Miguel (fernandomiguel)]

~/temp/1.0-url-stripping-removed-greader-fixes$ sudo python setup.py install
Traceback (most recent call last):
  File "setup.py", line 6, in <module>
    from DistUtilsExtra.command import *
ImportError: No module named DistUtilsExtra.command

[Billie Thompson (billiecodes)]

sudo apt-get install python-dbus python-gtk2 python-gconf python-notify python-webkitgtk python-simplejson python-egenix-mxdatetime python-distutils-extra python-feedparser

It has the same dependencies gwibber does normally

[Omer Akram (om26er)]

Gwibber's source have greader.py so this might be implemented back in 3.0?