source-academy / cadet-frontend / 2324

30%

push

travis-ci

Add notifications (#656)

* Merge remote-tracking branch 'origin/add-notifications-type' into add-notifications

* Minor fix

* Added notifications to state tre

* Change from interface to type

* Added Notification Badge

* Formatting

* Example of notification badge usage

* Added tags

* Added backend notification actions

* Insert fetching for login

* Shifted prop location

* Formatting

* Improved UI of hover tags

* Refactor backend functions

* Added options

* Added notification badge to more locations. Increased functionality of notification badge

* Added missing comma

* Added another missing comma

* Removed periodic fetching

* Refactor

* Disable hover popover for navbar

* Removed padding from tag

* Fix infinite rerender issue

* Added badge to relevant places

* Filter notifications

* Modified notification struct

* Use strict comparison instead of falsey/truthy

* Use relative paths

* Updated tests

* TSLint

* Use strict comparison

* Use yield instead of store.dispatch

* Add acknowledge notification to mock

* Ensure ag-grid rerenders notification cell on acknowledgement

* Added helper function for notification filtering

* Added sorting of assessments, applied helper function

* Minor refactor

* Rewrite notification messages

* Separate HOC functionality from component

* Updated assessment index snapshot

* Added tests for notification badge

* Updated tests

* Acknowledge now acknowledges multiple notifications

* Users now implicity acknowledge by navigating to specific assessment/submission

* Updated styles

* Reverting a small style change

* Updated snapshots

* Added notification support for chatkit

* Modified style for notification badge popover

* Cleanup after rebase

* AcademyNotification -> Notification

* Disable processing of notification tags when enableHover is false

* preProcessData -> sortSubmissions
... (continued)

368 of 1651 branches covered (22.29%)

Branch coverage included in aggregate %.

1908 of 4729 relevant lines covered (40.35%)

2.69 hits per line