expertiza / expertiza / 12714

37%

main: 10%

push

travis-ci-com

Improve e-mail notifications (#2274)

* Resolving Issue 111

* Notify all reviewers that have performed a review when the submission has been updated.

* basic refactoring done without testing

* more refactoring,email function from assignmment_team to team

* tested and sending emails now

* some more refactoring, no more static function

* add relevant links to reminder emails + tests

* Email testing for submitted_content_controller.
Minor aesthetic corrections.

* Removed incomplete mailer_spec test case.

* Enhance emailing SQL performance with 'includes' and other minor code cleanup.

* fix mail worker spec

* trying tetsting workl

* test case to test emails being sent to reviewrs

* trigger Travis build

* Some rubocop fixes

* Two more rubocop fixes

* Most requested changes

* Move mail creation to mailer_helper

* Add missing end

* Fix method call error

* make minor tweak to allow no password to be provided

* fix view

* revert what i thought was the bug

* fix some syntax

* fix rubocop

* fix ws

Co-authored-by: Dheeraj R Makam <drmakam@ncsu.edu>
Co-authored-by: Reuben John <reubenvjohn@gmail.com>
Co-authored-by: Arnav Julka <arnavjulka@gmail.com>
Co-authored-by: Griffin Brookshire <gbrookshire5@gmail.com>
Co-authored-by: Griffin Brookshire <51344670+griffinbrookshire@users.noreply.github.com>
Co-authored-by: Reuben John <reubenjohn@users.noreply.github.com>
Co-authored-by: John Bumgardner <john.bumgardner@icloud.com>
Co-authored-by: John Bumgardner <38589044+johnbumgardner@users.noreply.github.com>

24 of 24 new or added lines in 5 files covered. (100.0%)

6545 of 13549 relevant lines covered (48.31%)

14.46 hits per line