Skip to content

Add your news to readme#7714

Merged
Alkarex merged 11 commits intoFreshRSS:edgefrom
TijnvandenEijnde:add-your-news-to-readme
Oct 1, 2025
Merged

Add your news to readme#7714
Alkarex merged 11 commits intoFreshRSS:edgefrom
TijnvandenEijnde:add-your-news-to-readme

Conversation

@TijnvandenEijnde
Copy link
Contributor

@TijnvandenEijnde TijnvandenEijnde commented Jul 3, 2025

Closes #7694

Changes proposed in this pull request:

  • Added Your News in the Android & iOS tables.
  • Corrected check marks (Some check marks used a different icon).
  • Removed a link that was no longer working.

How to test the feature manually:

  1. Double-check if the links are working.

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

@Alkarex Alkarex added this to the 1.27.0 milestone Jul 3, 2025
@Alkarex Alkarex added Documentation 📚 API 🤝 API for other clients labels Jul 3, 2025
@Alkarex Alkarex modified the milestones: 1.27.0, 1.28.0 Aug 18, 2025
@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
@Alkarex Alkarex modified the milestones: 1.27.1, 1.28.0 Sep 25, 2025
@TijnvandenEijnde
Copy link
Contributor Author

Do you have any updates on this?

@Alkarex
Copy link
Member

Alkarex commented Oct 1, 2025

Sorry for the delay. I have just given it a try (I had some difficulties finding the settings to add the synchronisation to FreshRSS)
I cannot find how to see and/or add user labels to articles. Are you sure this is supported?

@TijnvandenEijnde
Copy link
Contributor Author

No problem at all! Thank you! I agree, it is a bit hidden at the moment in the Settings page.

You are totally right, I had mistaken this feature for categories for some reason. I just pushed a new change removing the checkmarks for the labels.

Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Oct 1, 2025
I spotted a memory issue when testing FreshRSS#7714
Attempt to stream results more, instead of keeping too much in memory.
Could be further improved.
@Alkarex Alkarex merged commit fcc272f into FreshRSS:edge Oct 1, 2025
1 check passed
Alkarex added a commit that referenced this pull request Oct 1, 2025
* API optimisation: more streaming of outputs
I spotted a memory issue when testing #7714
Attempt to stream results more, instead of keeping too much in memory.
Could be further improved.

* Apply suggestions from code review

Co-authored-by: Alexis Degrugillier <aledeg@users.noreply.github.com>

* Minor whitespace JSON formatting

---------

Co-authored-by: Alexis Degrugillier <aledeg@users.noreply.github.com>
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Oct 1, 2025
Due to high bandwidth in current sync FreshRSS#8041 (comment)
Follow-up of FreshRSS#7714
@Alkarex
Copy link
Member

Alkarex commented Oct 1, 2025

Follow-up: reduced to one star for sync due to high bandwidth consumption. This can happily be re-evaluated for future versions :-)

Alkarex added a commit that referenced this pull request Oct 1, 2025
Due to high bandwidth in current sync #8041 (comment)
Follow-up of #7714
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API 🤝 API for other clients Documentation 📚

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Your News, Android App (iOS soon)

2 participants