Skip to content
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

Sync read later / starred articles #53

Closed
Alkarex opened this issue Feb 29, 2020 · 6 comments
Closed

Sync read later / starred articles #53

Alkarex opened this issue Feb 29, 2020 · 6 comments
Labels
enhancement New feature or request

Comments

@Alkarex
Copy link
Contributor

Alkarex commented Feb 29, 2020

Hello,
I have tested your nice client today, and it looks very good already, congrats :-)
In the Google Reader API (implemented by FreshRSS), there is the concept of starred articles, which are well mapping with your "read later" concept. It would be nice to sync them, i.e. retrieve the list of starred articles to show in your "read later", and the other way around.
Feel free to ask if I can provide any documentation.
Alkarex, from FreshRSS

@Shinokuni
Copy link
Member

Hello,

Thank you!

Starred articles are planned but I am not sure if I should replace read later items by starred items.

@Shinokuni Shinokuni added the enhancement New feature or request label Mar 4, 2020
@notarobot63
Copy link

Hello,
This is really the feature I'm waiting for. I use FreshRSS and I'd love to sync my starred articles.

@Alkarex
Copy link
Contributor Author

Alkarex commented May 2, 2020

I am preparing a table overview of clients compatible with Google Reader API FreshRSS/FreshRSS#2942 (help welcome)
I would love to give a ✔️ to Readrops on that precise feature ;-)

P.S. That is the main feature missing for Readrops to become better than EasyRSS

@Shinokuni
Copy link
Member

Finally, starred/favorites items are here with cfa764e and will be available in 1.3! Sorry for the delay, it was about time.

@Alkarex Working with starred articles made me work again on item read state synchronization which is still not supported for FreshRSS in Readrops. I discovered some SQL constraints (like the limitation of arguments in a request with an IN operator) which means an evolution of the db schema would be potentially required. And to keep synchronization as fast as possible, I think several limitations on the items/ids request are required with an optimization of some SQL requests along with the db schema evolution. Anyway, this feature will require a lot of time (a lot of benchmarks too...).

@Alkarex
Copy link
Contributor Author

Alkarex commented Nov 2, 2020

Thanks for the update @Shinokuni, great news 👍
Let me know if I can help, for instance with some testing. Do you happen to have some development builds somewhere?
I can also provide sparring when it comes to the queries to perform for an efficient sync.
See e.g. FreshRSS/FreshRSS#2566 (comment) (to get server changes; this part does not include pushing changes to the server, but I can provide that as well)

@Shinokuni
Copy link
Member

Thanks, I will probably make a post on FreshRSS/FreshRSS#2566 to sum up the issues and what I would like to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants