Looks good overall. Inline are one or two optional suggestions, plus one NF for how NotificationModel::removeAllNotificationsForClient() handles its loops.
« Back to merge proposal
Looks good overall. Inline are one or two optional suggestions, plus one NF for how NotificationMod el::removeAllNo tificationsForC lient() handles its loops.