Skip to content
Permalink
Browse files

[FIX] web: mark as read trigger notification

Install discuss, change the general setting in dev mode to disable the
"Show Effect" checkbox. Change the preferences of the user to let odoo
handle notifications. Ask another user to ping you on a chatter then go
to discuss and click the "Mark as Read" button. Traceback.

The `notification_manager` agent has been replaced by another agent in
07cd24b, the code has been updated to use the new agent.

opw-1986844

closes #33363

Signed-off-by: Julien Castiaux <Julien00859@users.noreply.github.com>
  • Loading branch information...
Julien00859 committed May 14, 2019
1 parent 83b52e3 commit 43295c2a3741fb98f9c5d1a506652442efed0a82
Showing with 5 additions and 1 deletion.
  1. +5 −1 addons/web/static/src/js/chrome/abstract_web_client.js
@@ -478,7 +478,11 @@ var AbstractWebClient = Widget.extend(ServiceProviderMixin, KeyboardNavigationMi
new RainbowMan(data).appendTo(this.$el);
} else {
// For instance keep title blank, as we don't have title in data
this.notification_manager.notify('', data.message, true);
this.call('notification', 'notify', {
title: "",
message: data.message,
sticky: false
});
}
} else {
throw new Error('Unknown effect type: ' + type);

0 comments on commit 43295c2

Please sign in to comment.
You can’t perform that action at this time.