-
-
Notifications
You must be signed in to change notification settings - Fork 777
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
Fever fixes #1931
Fever fixes #1931
Conversation
Was hardcoded for MySQL. Bug in "before" parameter. Bug in mark all as read.
The lack of "before" parameter from the clients is quite problematic, as newer articles than the last synchronisation with the client will be marked as read erroneously. There is also a few TODOs in the code to address |
I hear from a user that Fiery crashed when marking a category as read |
$dao = $this->getDaoForCategories(); | ||
if (isset($_REQUEST["group_ids"])) { | ||
$group_ids = explode(",", $_REQUEST["group_ids"]); | ||
if (isset($_REQUEST['group_ids'])) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot find any mention of a group_ids
parameter in the documentation https://feedafever.com/api .
There is however a group_id
one.
Merging in /dev to make it easier for testers |
* Fever fixes Was hardcoded for MySQL. Bug in "before" parameter. Bug in mark all as read. * Fix construct * Changelog 1930 FreshRSS#193 FreshRSS#1931
* Fever fixes Was hardcoded for MySQL. Bug in "before" parameter. Bug in mark all as read. * Fix construct * Changelog 1930 FreshRSS#193 FreshRSS#1931
Was hardcoded for MySQL. Bug in "before" parameter. Bug in mark all as read.
#1930