lemon24 / reader

A Python feed reader library.
https://reader.readthedocs.io
BSD 3-Clause "New" or "Revised" License
455 stars 38 forks source link

Some feeds have .updated None #214

Open lemon24 opened 3 years ago

lemon24 commented 3 years ago

Some feeds have .updated = None:

This is fine, .updated is optional, but it is not useful: I would like to know when did this feed get a new/updated entry?.

We can't use .last_updated, because that includes the last check on not modified feeds (when did reader try to update this feed last?).

Two options come to mind, both involving a new Feed attribute that contains max(max(e.updated, e.published) for e in entries); the difference is what entries are considered when calculating this: