Code review comment for lp:~allenap/maas/notifications-creation

Revision history for this message
Blake Rouse (blake-rouse) wrote :

Looks good. Other than the one question I have. Which actually might allow you to remove that new dependency you added.

We need to do some work to get the notifications to show on the none angular pages as well, since this branch removes notifications from those pages. Its not to hard to enable.

review: Approve

« Back to merge proposal