fasheng / elfeed-protocol

Provide extra protocols to make like Fever, NewsBlur, Nextcloud/ownCloud News and Tiny Tiny RSS work with elfeed
GNU General Public License v3.0
100 stars 18 forks source link

Batch sync of read states #23

Closed ieure closed 4 years ago

ieure commented 4 years ago

One thing I found somewhat surprising about elfeed-protocol is that when an item's read state is changed, it performs a synchronous request to Nextcloud to set it. This is in contrast to other RSS syncing setups I've used, where changes of this nature are buffered locally, then sent in a batch when synchronizing.

The synchronous mechanism has some undesirable (at least to me) side-effects:

On the one hand, "doctor, it hurts when I do this," but on the other hand, it's always struck me as unusual that marking as read/unread is a sync request.

What do you think about buffering read/unread/starred/unstarred and updating them all at once when elfeed-search-fetch is called?

fasheng commented 4 years ago

Thanks your advice, I will consider to implement it in following releases.

fasheng commented 4 years ago

Hi, you could enable the feature in latest elfeed-protocol 0.7.2, please have a try and reopen the issue if any problem occurs ;)

(setq elfeed-protocol-lazy-sync t)
ieure commented 4 years ago

Thank you! I updated yesterday and it's great, exactly what I wanted.