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

Mark all as ready time limit issue and improvement #37

Open
Sergio-Muriel opened this issue Feb 28, 2015 · 0 comments
Open

Mark all as ready time limit issue and improvement #37

Sergio-Muriel opened this issue Feb 28, 2015 · 0 comments

Comments

@Sergio-Muriel
Copy link
Contributor

It looks like the "ts" parameter of /reader/api/0/mark-all-as-read has a strict condition.
"ts" must be greater than the "item time" to be marked as read, but i think it should be greater or equal than the passed parameter:

If i get item list with /reader/api/0/stream/items/ids?output=json&s=user/-/state/com.google/reading-list&xt=user/-/state/com.google/read , i get a "timestampUsec" parameter i can use to set all the items i have seen since the last update.
With the current strict condition, i have to increment 1 second the timestampUsec i received to set all the items as read .

Another problem with the "ts" parameter is that it looks it is truncated before comparaison, and the last 4 digits are not used to check if the item is older.

Let me know if i dind't explain well my self. My english is not really good...

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

No branches or pull requests

1 participant