Skip to content
This repository was archived by the owner on Nov 28, 2024. It is now read-only.

Fix refresh starred #21

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

scastrorico
Copy link
Collaborator

Application doesn't refresh starred post when used with theoldreader.com (it generates a 404 error), althought I think it work well with Freshrss.

The URL used by application to retrieve starred items is this:
<server>/reader/api/0/stream/contents/user/-/state/com.google/starred.

According to this page https://github.com/mihaip/google-reader-api/blob/master/wiki/ApiStreamContents.wiki and theoldreader api reference page https://github.com/theoldreader/api, it seems that the correct URL must be this:
<server>/reader/api/0/stream/contents?s=user/-/state/com.google/starred

I tested it and work well in both systems.

Sort ItemIds by timestamp (Downward). That prevents mark all items as
read if they are disordered.
In webview settings, if blockNetwokImage option is false images are not
loaded (although they are in device and loaded with file:// scheme).
In theoldreader.com, when application try to refresh starred post, it
get a document not found error.
@Alkarex
Copy link
Owner

Alkarex commented Apr 23, 2016

Thanks @scastrorico 👍
Could you please update your pull request to remove the conflicts?

@scastrorico
Copy link
Collaborator Author

I removed the conflicts.

@Alkarex
Copy link
Owner

Alkarex commented Nov 28, 2024

I am archiving this project, as there are better open source alternatives, more actively maintained and developed:
check Readrops or Capy Reader

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants