Make notification ID a required property
Background
This came up in the discussion for #126 (closed). At the moment, it is possible for a notification to have no ID. This works but also causes some weird behavior. For example, when a notification with no ID is marked as shown, it also prevents any other notification with no ID from being shown in the future.
Ideally all notifications should be required to have an ID.
Moreover, extension APIs like browser.notifications.clear()
require the ID to be a string.
Moreover, addNotification()
and removeNotification()
work on the object reference, they should work on the ID instead.
What to change
TBD
/cc @ThomasGreiner
Edited by Manish Jethani