Skip to content
This repository has been archived by the owner on Mar 14, 2022. It is now read-only.

Planet only keeps 2 articles per feed #37

Closed
JonasHelming opened this issue Nov 22, 2021 · 2 comments · Fixed by #38
Closed

Planet only keeps 2 articles per feed #37

JonasHelming opened this issue Nov 22, 2021 · 2 comments · Fixed by #38

Comments

@JonasHelming
Copy link
Contributor

Due to the new_feed_option set to "2", the planet only keeps 2 articles from the same feed while it keeps articles from other feeds for over a year. While this was probably introduced for some spam protection, I do not see a good reason to keep this number so low. I therefore suggest to raise this limit, e.g. to 8 and I will create a PR.
JonasHelming added a commit to JonasHelming/planeteclipse.org that referenced this issue Nov 22, 2021
@JonasHelming
Increases new feed item limit to 8
c2d8846 
fixed EclipseFdn#37

Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
@JonasHelming JonasHelming mentioned this issue Nov 22, 2021
Merged
chrisguindon pushed a commit that referenced this issue Nov 22, 2021
@JonasHelming @chrisguindon
Increases new feed item limit to 8
9870560 
fixed #37

Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
@JonasHelming
Copy link
Contributor Author

Based on a discussion with @chrisguindon , we decided on going up to 25 for now

JonasHelming added a commit to JonasHelming/planeteclipse.org that referenced this issue Nov 22, 2021
@JonasHelming
Increased feed item limit to 25
bc6c18a 
fixed EclipseFdn#37

Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
JonasHelming added a commit to JonasHelming/planeteclipse.org that referenced this issue Nov 22, 2021
@JonasHelming
Merge branch 'master' into EclipseFdnGH-37
924701c
chrisguindon pushed a commit that referenced this issue Nov 23, 2021
@JonasHelming
Gh 37 (#39)
0608caf 
* Increases new feed item limit to 8

fixed #37

Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>

* Increased feed item limit to 25

fixed #37

Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
@chrisguindon
Copy link
Member

Based on a discussion with @chrisguindon , we decided on going up to 25 for now

+1 We can re-visit if this becomes a problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Increases new feed item limit to 8 JonasHelming/planeteclipse.org
2 participants
@chrisguindon @JonasHelming