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.

Sign up for GitHub

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

disable new setting nextUpdateTime by default #3039

Merged
merged 1 commit into from
Jan 10, 2025
Merged

disable new setting nextUpdateTime by default #3039

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ You can also check [on GitHub](https://github.com/nextcloud/news/releases), the
### Changed
- add explanations for the individual values in the feed information table
- show error message from `opml` import in web-ui
- disable new setting "nextUpdateTime" introduced in #2999 by default

### Fixed
- fix proxy port removed if standard port for the protocol (#3027)
Expand Down
6 changes: 4 additions & 2 deletions docs/admin.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@
By default, the value is set to 3600 seconds (1 hour) You can configure this interval as an administrator.
The new value is only applied after the next run of the updater.

Since News 25.2.0 News no longer will update all feeds instead it will make a individual decision based on when an update for the feed will make sense. Therefore this interval is now only to be understood as the check if an update should be done.
Starting with News 25.2.0, the app can dynamically adjust update schedules based on feed activity. This feature, disabled by default, can be enabled by the Nextcloud administrator.

Check failure on line 66 in docs/admin.md

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Nextcloud'?

Raw Output:
{"message": "[Vale.Spelling] Did you really mean 'Nextcloud'?", "location": {"path": "docs/admin.md", "range": {"start": {"line": 66, "column": 157}}}, "severity": "ERROR"}

This behavior can be disabled in the Settings although we generally do not recommend that.
By analyzing feed data, the app can optimize update frequencies, potentially reducing server load and network traffic. However, this feature may not work correctly with all feeds.

Users can check the calculated next update time in the app's settings. This information will only be displayed when the dynamic update scheduling feature is enabled.
2 changes: 1 addition & 1 deletion lib/AppInfo/Application.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class Application extends App implements IBootstrap
'useCronUpdates' => true,
'exploreUrl' => '',
'updateInterval' => 3600,
'useNextUpdateTime' => true,
'useNextUpdateTime' => false,
];

public function __construct(array $urlParams = [])
Expand Down
Loading