[FIX] mail: adjust condition to use proper model for user preferences

Before this commit:
The isInbox condition was always evaluated as false because it incorrectly
checked the user's notification preference in the wrong model. This caused
users to receive two push notifications for the same message.

After this commit:
The isInbox condition now checks the user model for notification preferences,
ensuring the correct behavior.

closes odoo#233289

X-original-commit: 6e27555
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

Author: shmn-odoo

addons/mail/static/src/core/common/store_service.js

@@ -426,7 +426,9 @@ export class Store extends BaseStore {
                 // Prevent duplicate inbox push notifications since they're already handled by
                 // `mail.message/inbox` bus notifications, and the `modelsHandleByPush` heuristic
                 // in `out_of_focus_service.js` isn't reliable enough to detect these cases.
-                const isInbox = this.store.self.notification_preference === "inbox" && model !== "discuss.channel";
+                const isInbox =
+                    this.store.self.main_user_id?.notification_type === "inbox" &&
+                    model !== "discuss.channel";
                 if ((isTabFocused && thread?.isDisplayed) || isInbox) {
                     navigator.serviceWorker.controller?.postMessage({
                         type: "notification-display-response",