mirror of
https://github.com/librenms/librenms.git
synced 2024-10-07 16:52:45 +00:00
Push Notifications (Mobile and PC) (#13277)
* Update manifest and add service worker cleanup icons a bit * Push notifications WIP * navigate working * cleanup * acknowledge wired up * Set VAPID keys on composer install * Component to control notification permissions. * Allow all user option to validate * Enable on browser load if transport exists. * Check for transport before showing user permissions translations * Documentation * style fixes * access via the attribute model * fix alerting test * update schema * cleanup subscription on disable * non-configurable db and table for webpush subscriptions (respect system connection) * revert AlertTransport change hopefully phpstan can figure it out * phpstan fixes * Support custom details display * Match transport names to brand's preferred display * less duplicate id errors * Tests are done in Laravel code now so remove legacy function usage... could be better, but ok * Style fixes * Style fixes 2 * Fix alert test * Doc updates requires HTTPS and GMP * unregister subscription when permission is set to denied * cleanup after user deletion * delete the right thing * fix whitespace * update install docs to include php-gmp * suggest ext-gmp * update javascript * Update functions.php Co-authored-by: Jellyfrog <Jellyfrog@users.noreply.github.com>
This commit is contained in:
@@ -283,7 +283,7 @@ if ($options['f'] === 'purgeusers') {
|
||||
if ($purge > 0) {
|
||||
$users = \App\Models\AuthLog::where('datetime', '>=', \Carbon\Carbon::now()->subDays($purge))
|
||||
->distinct()->pluck('user')
|
||||
->merge(\App\Models\User::has('apiToken')->pluck('username')) // don't purge users with api tokens
|
||||
->merge(\App\Models\User::has('apiTokens')->pluck('username')) // don't purge users with api tokens
|
||||
->unique();
|
||||
|
||||
if (\App\Models\User::thisAuth()->whereNotIn('username', $users)->delete()) {
|
||||
|
Reference in New Issue
Block a user