-
Notifications
You must be signed in to change notification settings - Fork 190
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
Displayed articles are not sorted by date #1608
Comments
I think this needs to be handled by the client, for the api this should not matter. So in the case of the old webapp the javascript would need to sort the items according to the users preference. |
Agree that its a client thing, however in this project the web Frontend/"client" seems quite though coupled to the backend. like database queries on it's own. correct? And yes, a user setting is OK and welcome.... whatever allpws to sort properly by date instead of some articleid and feed grouping. |
Yea that's true, so far nobody managed to modernize the webapp, if it happens one day I think this should be part of that. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
The issue is still relevant. |
Yea but I think it can be closed, the frontend urgently needs a complete rewrite as the technology it uses is no longer supported and bugs/security issues will stay open. If that is not done the problem will keep existing for forever. |
Related to #195 |
You could write a comment there, but I fear it won't change much. The frontend based on the original angular js, that is no longer supported. The frontend needs to be rebuild with a new JavaScript framework, Vue js makes sense since Nextcloud provides common components for that and most internally (by the company) maintained apps use Vue now or soon. There were attempts to do that but it's not that easy to do and currently there is again no ongoing progress. |
please, fix this bug. |
IMPORTANT
Explain the Problem
Articles shown in the frontend are not sorted by it's date, but instead of some import date / import ID and grouping.
Please see the video below, you can clearly see that there are articles once "x hours ago", then "days ago", then "hours ago" again, then somewhen "weeks ago"...
I do expect that WHATEVER the source of the news article is, is on top if it was published the most recently.
cut.mp4
This issue seems similar but was closed and stayed closed: #51
What I quickly tried is changing the
OrderBy
atnextcloud/apps/news/lib/Db/ItemMapperV2.php
topub_date
, then the situation minimally improved, but guess thats the point where changes are needed.Just for reference, this is how TT-RSS orders newest:
Steps to Reproduce
Explain what you did to encounter the issue
System Information
Would be really cool if you can sort the news by date, instead of some ID made up on your own at database. It would be very fine if the user needs to check "sort by date" / toggle between ID and date, whatever is easiest and works
Thanks!
The text was updated successfully, but these errors were encountered: