- We can now set a button name to the notification action.
- We can set an icon button as well.
- After the button is clicked, the notification gets closed.
When trying to use the notify function that comes from a normal user with sudo flag enabled, it wasn't letting the user access it. Adding the su flag to the check should allow sudo users using this feature.
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default)
* [IMP] use black color for text for default notification.
* [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
Add self-test buttons in demo environment,
Updated readme to show how to test it.
Add buttons to users form
Do not rely on SUPERUSER_ID and avoid getattr usage
Only the admin user (sudo) is allowed to send notifications to other
users. The normal users can only send notifications to themselves.
This is to prevent attackers to craft malicious notifications and send
them to other users using RPC.
Correction based on the idea of @hbrunn
- Use the 'session' class of the JS Framework (session no lounger bound
to web client)
- Test change: compare emitted & received messages based on content, not
order. Using string comparison raises false positives.
trisdoan
SilvioC2C
David
DavidJForgeFlow
Guewen Baconnier
Benoit Aimont
Aiendry Sarkar
laurent.corron
Shepilov Vladislav
Aitor Bouzas
Damien Bouvy
Jay Vora(SerpentCS)
Laurent Mignon (ACSONE)