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

Read articles haunting me #2717

Closed
3 tasks done
erebion opened this issue Jul 22, 2024 · 19 comments
Closed
3 tasks done

Read articles haunting me #2717

erebion opened this issue Jul 22, 2024 · 19 comments
Labels

Comments

@erebion
Copy link

erebion commented Jul 22, 2024

IMPORTANT

As suggested by @Grotax in #1908, I'm opening a new issue.

While typing the issue, I realised I am on an alpha version. But maybe you will still find the report helpful. If not, please just close. But maybe it helps somehow. :)

  • I have read the CONTRIBUTING.md and followed the provided tips
  • I accept that the issue will be closed without comment if I do not check here <--- would read, but 404... but I have to check it to not have the issue closed...?
  • I accept that the issue will be closed without comment if I do not fill out all items in the issue template.

Explain the Problem

What problem did you encounter?

Steps to Reproduce

Explain what you did to encounter the issue

  1. Mark article as read
  2. After a while (no specific amount of time) it will be back as unread

System Information

  • News app version: 25.0.0-alpha3
  • Nextcloud version: Nextcloud Hub 8 (29.0.3)
  • Cron type: systemd timer
  • PHP version: 8.2.20
  • Database and version: Postgres, either 14 or 15 (cannot access right now, can add later if important)
  • Browser and version: Firefox ESR, latest 115
  • OS and version: Debian
Contents of nextcloud/data/nextcloud.log
Paste output here
Contents of Browser Error Console Read http://ggnome.com/wiki/Using_The_Browser_Error_Console if you are unsure what to put here
Paste output here

don't have access right now, can add this later

@erebion erebion added the bug label Jul 22, 2024
@SMillerDev
Copy link
Contributor

What do you have your cleanup threshold set to in the admin settings?

@erebion
Copy link
Author

erebion commented Jul 23, 2024

Set to 200.

Also:

Deleting unread articles automatically: off
Max Redirects: 10
Timeout: 60s
Interval for updating: 3600s

@SMillerDev
Copy link
Contributor

And roughly how many articles do you have?

@erebion
Copy link
Author

erebion commented Jul 23, 2024

And roughly how many articles do you have?

Differs per feed. None are extremely large, the largest have a number in the lower hundreds. Most have less than a hundred posts. I use aroudn twenty feeds.

Articles come back in many feeds. For example the Debian project's blog. Suddenly an article from 14 years ago is shown as "new".

@SMillerDev
Copy link
Contributor

Most have less than a hundred posts. I use aroudn twenty feeds.

So the total amount of articles is more than 200, which means that after every cleanup it'll pull in the old articles again, since only the newest 200 remain.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

So the total amount of articles is more than 200, which means that after every cleanup it'll pull in the old articles again, since only the newest 200 remain.

If that's what the setting does, then the wording is not clear on what the implications are.

It reads like "We will delete every articles more than 200".

It does not read like "We will haunt you with every arcticle more than 200 and show old ones as new".

If that is what this setting is meant to do, then I'd like to suggest changing the wording to something that makes this clear.

Either way, I find it weird that it would get articles older then the most current 200 articles, from years ago, and then show them as new. Doesn't it just save "Everything older than this is too old"?

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Also, I just realised that the German version states the opposite:

Setting:

Defines the maximum amount of articles that can be read per feed which will not be deleted by the cleanup job; if old articles reappear after being read, increase this value; negative values such as -1 will turn this feature off.

It therefore states:

if old articles reappear after being read

German version states:

wenn ältere Artikel nach dem Lesen erhalten bleiben sollen

Translated back into English:

if older articles are to be kept after reading

Which changes the meaning completely and explains my utter confusion.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Where can I work on the translation? Transifex seems to only get used for Nextcloud itself, but not Nextcloud News, right?

@Grotax
Copy link
Member

Grotax commented Jul 24, 2024

The algorithm for cleanup works actually a bit differently than just keeping 200 read items or whatever the value is.

News fetches the feed and then checks

If the feed has less than 200 items in the feed file then News will keep 200 read items and whatever amount of unread items.

If a feed has more than 200 items then we take whatever that number is as limit, if I'm not wrong that is also stored in the DB.

But there are cases where this might still not work. Also some feeds behave a bit strange because of the way they are generated.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Sure, though I'd still like to help out with the translation. Having a translation that pretty much states the opposite does not help.

@Grotax
Copy link
Member

Grotax commented Jul 24, 2024

In our documentation this is also explained https://nextcloud.github.io/news/admin/

Transifex is also used for the news app.

It might seem logical to also filter items based on date like when we see an item that is much older than the oldest we have in the db then we don't add it to the db.
But the issue with that is that it's quite common for feeds to have the wildest date combinations. Or feeds that work based on most viewed or other metrics where an old item can suddenly come back to the feed.

In your case I would just adjust the value to a higher number and see if that works for you.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Transifex is also used for the news app.

Would you be so kind to let me know where I can find the project?

In your case I would just adjust the value to a higher number and see if that works for you.

I've just disabled cleanup completely. It's just a bit of text anyway.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Setting it to -1 does not resolve the issue, articles still come back.

@erebion
Copy link
Author

erebion commented Jul 24, 2024

Will I have to wait for all articles to come back, mark them as read and then be done, not just by setting this to -1?

@erebion
Copy link
Author

erebion commented Aug 1, 2024

Setting it to -1 does not resolve the issue, articles still come back.

It seems that Nextcloud News does not store changes in the settings.

I've now changed it with

# sudo -u www-data php -f occ config:app:set --value "\-1" --update-only  news autoPurgeCount
Config value 'autoPurgeCount' for app 'news' is now set to '\-1', stored as mixed in fast cache

Now the UI just shows it set to \-1, but due to nextcloud/server#26109, I cannot set it to -1 without a backslash. But maybe it interprets something that looks invalid as having that disabled..?

@Grotax
Copy link
Member

Grotax commented Aug 17, 2024

Yea the issue is that the Admin UI is not able to update the fields anymore: #2533

-1 will probably lead to an error but I'm not sure.

I'm sorry that you are facing these issues, the problem is that news has not nearly enough (developer) support for what it tries to be...

@erebion
Copy link
Author

erebion commented Aug 17, 2024

Can I manually set that somewhere? Maybe in the database?

@Grotax
Copy link
Member

Grotax commented Aug 18, 2024

Yes you could update the setting in the database. Nextcloud has an app config table.

But I don't know the details at this time so you have to check.

@Grotax
Copy link
Member

Grotax commented Nov 8, 2024

Update on this the admin settings work again in the latest alpha version.

And I will close this :)

@Grotax Grotax closed this as completed Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants