LG-10126: SendProofingNotificationJob refactor#8843

Merged

tomas-nava merged 16 commits intomainfrom

tomas/lg-10126-refactor

Jul 26, 2023

Commits on Jul 24, 2023

only invoke job when we know we need it

Tomas Apodaca
authored and
Tomas Apodaca
committed
flag was set twice

Tomas Apodaca
authored and
Tomas Apodaca
committed
rename method & update tests

Tomas Apodaca
authored and
Tomas Apodaca
committed
don't need mocks; reload before checking values

Tomas Apodaca
authored and
Tomas Apodaca
committed
log 'skipped' when enrollment is nil

Tomas Apodaca
authored and
Tomas Apodaca
committed
remove begin/ensure

Tomas Apodaca
authored and
Tomas Apodaca
committed
improve handling telephony response

Tomas Apodaca
authored and
Tomas Apodaca
committed
move sms tests into their own context

Tomas Apodaca
authored and
Tomas Apodaca
committed
Merge branch 'main' into tomas/lg-10126-refactor
Tomas Apodaca
authored
send SMS on other types of failures

Tomas Apodaca
authored and
Tomas Apodaca
committed

Commits on Jul 25, 2023

Merge branch 'main' into tomas/lg-10126-refactor
Tomas Apodaca
authored
rename analytics events

Tomas Apodaca
authored and
Tomas Apodaca
committed
use the passed enrollment id in event

Tomas Apodaca
authored and
Tomas Apodaca
committed
capture and log exceptions

Tomas Apodaca
authored and
Tomas Apodaca
committed
fix job name in analytics event

Tomas Apodaca
authored and
Tomas Apodaca
committed

Commits on Jul 26, 2023

alphabetize analytics methods

Tomas Apodaca
authored and
Tomas Apodaca
committed