Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Commit bb15223

Browse files
committed
Always show the notification bar
per feedback in #15309 (comment) Auditors: @diracdeltas
1 parent f91b856 commit bb15223

File tree

2 files changed

+11
-27
lines changed

2 files changed

+11
-27
lines changed

app/updater.js

+11-25
Original file line numberDiff line numberDiff line change
@@ -87,32 +87,18 @@ var scheduleUpdates = () => {
8787
}
8888

8989
// Linux doesn't have an auto-update mechanism.
90-
// Instead, deliver a periodic nag linking to instructions
91-
if (isLinux() && appConfig.updates.deprecatedNagFrequency) {
92-
electron.ipcMain.on(messages.NOTIFICATION_RESPONSE, (e, message, buttonIndex, persist) => {
93-
appActions.hideNotification(message)
90+
// Instead, show a persistent nag linking to instructions
91+
if (isLinux()) {
92+
appActions.showNotification({
93+
buttons: [],
94+
options: {
95+
persist: false,
96+
advancedText: 'See instructions to upgrade to the latest version',
97+
advancedLink: 'https://brave-browser.readthedocs.io/en/latest/installing-brave.html#linux'
98+
},
99+
position: 'global',
100+
message: 'This version of Brave is no longer supported and will not be updated.'
94101
})
95-
96-
const showDeprecationNotice = () => {
97-
appActions.showNotification({
98-
buttons: [
99-
{text: 'OK'}
100-
],
101-
options: {
102-
persist: false,
103-
advancedText: 'See instructions to upgrade to the latest version',
104-
advancedLink: 'https://brave-browser.readthedocs.io/en/latest/installing-brave.html#linux'
105-
},
106-
position: 'global',
107-
message: 'This version of Brave is no longer supported and will not be updated.'
108-
})
109-
}
110-
111-
setInterval(() => {
112-
showDeprecationNotice()
113-
}, appConfig.updates.deprecatedNagFrequency)
114-
115-
showDeprecationNotice()
116102
}
117103

118104
// Startup check

js/constants/appConfig.js

-2
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,6 @@ module.exports = {
109109
partition: 'persist:tor'
110110
},
111111
updates: {
112-
// (Linux only) display a deprecated warning every 10 minutes
113-
deprecatedNagFrequency: 1000 * 10 * 60,
114112
// Check for front end updates every hour
115113
appUpdateCheckFrequency: 1000 * 60 * 60,
116114
// Check after 2 minutes, near startup

0 commit comments

Comments
 (0)