Skip to content

Remove superflous SQL category request#1316

Merged
Alkarex merged 2 commits intoFreshRSS:devfrom
Alkarex:initial-requests
Oct 15, 2016
Merged

Remove superflous SQL category request#1316
Alkarex merged 2 commits intoFreshRSS:devfrom
Alkarex:initial-requests

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Oct 15, 2016

A category request was systematically done, although it is not always
needed.

A category request was systematically done, although it is not always
needed.
@Alkarex Alkarex added this to the 1.6.0 milestone Oct 15, 2016
@Alkarex Alkarex changed the title Remove superflous category request Remove superflous SQL category request Oct 15, 2016
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 15, 2016

Especially needed now that the number of "mark as read" requests will increase with "scroll as read" #1309

@Alkarex Alkarex merged commit 9ed59ff into FreshRSS:dev Oct 15, 2016
@Alkarex Alkarex deleted the initial-requests branch October 15, 2016 13:13
@Alwaysin
Copy link
Copy Markdown
Contributor

What did motivate the choice of a default "mark read as scroll"? I think this is a bad idea.

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 15, 2016

It is an option, so you can turn it off. The main reason is to address the UI problem of marking previous articles as read, for which there is currently no other option in FreshRSS #329

@Alwaysin
Copy link
Copy Markdown
Contributor

Yes I've turned it off, I just find it very disturbing when you first install an rss reader that it marks the articles as read directly, when you usually go around and scroll to see how it looks. You may think this is a bug.

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

Successfully merging this pull request may close these issues.

2 participants