diff --git a/AppImage/scripts/notification_channels.py b/AppImage/scripts/notification_channels.py index 92c35aab..b2906d7e 100644 --- a/AppImage/scripts/notification_channels.py +++ b/AppImage/scripts/notification_channels.py @@ -769,8 +769,15 @@ class EmailChannel(NotificationChannel): _add('Proxmox Updates', data.get('pve_count')) _add('Kernel Updates', data.get('kernel_count')) imp = data.get('important_list', '') - if imp: - _add('Important Packages', imp, 'code') + if imp and imp != 'none': + # Render each package on its own line inside a single cell + pkg_lines = [l.strip() for l in imp.split('\n') if l.strip()] + if pkg_lines: + pkg_html = '
'.join( + f'{esc(p)}' + for p in pkg_lines + ) + rows.append((esc('Important Packages'), pkg_html)) _add('Current Version', data.get('current_version'), 'code') _add('New Version', data.get('new_version'), 'code') diff --git a/AppImage/scripts/notification_events.py b/AppImage/scripts/notification_events.py index 616a790d..c4455cd4 100644 --- a/AppImage/scripts/notification_events.py +++ b/AppImage/scripts/notification_events.py @@ -1780,7 +1780,7 @@ class PollingCollector: 'security_count': str(len(security_pkgs)), 'pve_count': str(len(pve_pkgs)), 'kernel_count': str(len(kernel_pkgs)), - 'important_list': ', '.join(important_lines) if important_lines else 'none', + 'important_list': '\n'.join(important_lines) if important_lines else 'none', 'package_list': ', '.join(important_lines[:6]) if important_lines else '', } diff --git a/AppImage/scripts/notification_templates.py b/AppImage/scripts/notification_templates.py index 7f4e676b..507dc75e 100644 --- a/AppImage/scripts/notification_templates.py +++ b/AppImage/scripts/notification_templates.py @@ -703,7 +703,7 @@ TEMPLATES = { }, 'update_available': { 'title': '{hostname}: Updates available', - 'body': 'Total updates: {total_count}\nSecurity: {security_count}\nProxmox: {pve_count}\nKernel: {kernel_count}\nImportant: {important_list}', + 'body': 'Total updates: {total_count}\nSecurity: {security_count}\nProxmox: {pve_count}\nKernel: {kernel_count}\nImportant packages:\n{important_list}', 'label': 'Updates available (legacy)', 'group': 'updates', 'default_enabled': False, @@ -750,7 +750,7 @@ TEMPLATES = { 'Security updates: {security_count}\n' 'Proxmox-related updates: {pve_count}\n' 'Kernel updates: {kernel_count}\n' - 'Important packages: {important_list}' + 'Important packages:\n{important_list}' ), 'label': 'Updates available', 'group': 'updates',