-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
[stable12] Show EOL warning in the update section #8987
Conversation
@@ -107,14 +107,14 @@ public function displayPanel() { | |||
$notifyGroups = json_decode($this->config->getAppValue('updatenotification', 'notify_groups', '["admin"]'), true); | |||
|
|||
$params = [ | |||
'isNewVersionAvailable' => !empty($updateState['updateAvailable']), | |||
'isNewVersionAvailable' => true, //!empty($updateState['updateAvailable']), |
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.
🙈 let me fix this
6737553
to
30626ed
Compare
Codecov Report
@@ Coverage Diff @@
## stable12 #8987 +/- ##
==============================================
- Coverage 53.91% 53.91% -0.01%
- Complexity 22780 22782 +2
==============================================
Files 1387 1387
Lines 87286 87295 +9
Branches 1331 1331
==============================================
+ Hits 47063 47065 +2
- Misses 40223 40230 +7
|
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.
Rebased and fixed the typo. And works 👍
Signed-off-by: Joas Schilling <coding@schilljs.com>
30626ed
to
5f1f633
Compare
Rebased and fixed the last failing test |
Backport #8986