From b59a2101b460200e4276a05f9b8cb4ae7a2e9b62 Mon Sep 17 00:00:00 2001 From: rupak Date: Fri, 30 Jan 2026 02:19:35 +0545 Subject: Add option to enable notifications (#8458) Closes https://github.com/FreshRSS/FreshRSS/issues/7330 - Default behavior is same - Added FreshRSS_Context::userConf()->html5_disable_notif so that, it determines weather user wants notification. (will not show any even it has permission) (i want default false so disable, so old configs get proper default values) - Added button such that checking it makes it request permission too image - test notification actually happening (how can i trigger it, do i have to wait it), this code fixes permissions. --- app/i18n/en/conf.php | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'app/i18n/en/conf.php') diff --git a/app/i18n/en/conf.php b/app/i18n/en/conf.php index 2e4c20d32..bfc32ec3a 100644 --- a/app/i18n/en/conf.php +++ b/app/i18n/en/conf.php @@ -105,14 +105,17 @@ return array( 'none' => 'None', 'small' => 'Small', ), + 'notification' => array( + 'html5_enable_notif' => 'Enable notification', + ), 'notification_timeout' => array( 'bad' => array( - 'label' => 'Show warning banner', // TODO - 'seconds' => 'seconds (at least 1)', // TODO + 'label' => 'Show warning banner', + 'seconds' => 'seconds (at least 1)', ), 'good' => array( - 'label' => 'Show acknowledgement banner', // TODO - 'seconds' => 'seconds (0 means not shown)', // TODO + 'label' => 'Show acknowledgement banner', + 'seconds' => 'seconds (0 means not shown)', ), ), 'privacy' => array( -- cgit v1.2.3