-
Notifications
You must be signed in to change notification settings - Fork 211
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Please consider exposing skipHours (in shared model) #228
Comments
This seems like a scheduling concern. gofeed doesn't do any scheduling, it just fetches a feed and parses it. Maybe something like this would be a good application for an example test. |
As I understand it (second hand, I'm not a direct gofeed user) currently skipHours is parsed but only available in the RSS-specific view rather than the unified view. Making it available in the unified view would make it easier for some clients to code for and use. Rgds Damon |
The While you can access rssParser := &rss.Parser{}
feed, _ := rssParser.Parse(reader)
// Access feed.SkipHours Does it work for you to just use |
I am trying to make it as easy as possible for feed readers to do the right thing given that there is huge waste (>99%) in feed access currently. |
Please consider making (RSS) skipHours as easy to use as possible to help gofeed clients minimise feed energy and bandwidth impacts, especially in the light of more and more grid power coming from solar.
https://www.earth.org.uk/RSS-efficiency.html
Rgds
Damon
The text was updated successfully, but these errors were encountered: