fix: mobile push notification count inc

Author: d-gubert

apps/meteor/app/push-notifications/server/lib/PushNotification.ts

@@ -103,9 +103,10 @@ class PushNotification {
 			idOnly,
 		});
 
+		await Push.send(config);
+
 		metrics.notificationsSent.inc({ notification_type: 'mobile' });
 		metrics.notificationsSentTotal.inc({ notification_type: 'mobile' });
-		await Push.send(config);
 	}
 
 	async getNotificationForMessageId({

apps/meteor/server/services/video-conference/service.ts

@@ -674,8 +674,6 @@ export class VideoConfService extends ServiceClassInternal implements IVideoConf
 			return;
 		}
 
-		metrics.notificationsSent.inc({ notification_type: 'mobile' });
-		metrics.notificationsSentTotal.inc({ notification_type: 'mobile' });
 		await Push.send({
 			from: 'push',
 			badge: 0,
@@ -702,6 +700,9 @@ export class VideoConfService extends ServiceClassInternal implements IVideoConf
 				category: 'VIDEOCONF',
 			},
 		});
+
+		metrics.notificationsSent.inc({ notification_type: 'mobile' });
+		metrics.notificationsSentTotal.inc({ notification_type: 'mobile' });
 	}
 
 	private async sendAllPushNotifications(callId: VideoConference['_id']): Promise<void> {