Skip to content

Commit

Permalink
disable new setting nextUpdateTime introduced in #2999 by default
Browse files Browse the repository at this point in the history
Signed-off-by: Benjamin Brahmer <[email protected]>
  • Loading branch information
Grotax committed Jan 9, 2025
1 parent 6dbfa9a commit 3d049d5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
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
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ The update interval is used to determine when the next update of all feeds shoul
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
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

0 comments on commit 3d049d5

Please sign in to comment.