-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2687 Added email text for sendgrid #3050
2687 Added email text for sendgrid #3050
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3050 +/- ##
===========================================
- Coverage 92.97% 92.96% -0.02%
===========================================
Files 276 276
Lines 7316 7317 +1
Branches 646 646
===========================================
Hits 6802 6802
- Misses 413 414 +1
Partials 101 101
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm @raftmsohani 🚀 cc: @ttran-hub
* - Initial commit for duplicate record manager for TANF section 1 records * - more efficient error generation * - Added a way to track precedence in the error messages to avoid generating less meaningful messages - Updated error generation * - Fix lint - Add docstrings * - Added support to delete deplicate records * - add support for bulk/raw delete * - Update deletes on rollbacks to raw delete since it wont hurt anything * - Updated elastic and django to support raw/bulk deleting * - Updating rollback test since it is very relevant now that we are doing raw deletes * - raise log level * - Update test * - remove import * - Update error generation to use record and schema - rename var * - Update error precedence logic * - fix lint * - use correct method * - remove extra call to bulk create errors * - move duplicate manager into case consistency validator - Starting to remove SortedRecordSchemaPairs from case consistency validator - updating to support in memory record removal if they havent been serialized * - update add_record logic to include rpt_month_year * - Move SortedRecordSchema pairs to util - Update case consistency validator to not use OG SortedRecordSchemaPairs - Update dup logic to not consider records on the same line * - remove unused import * - Update class to store records based on hash pending the section type * - Update parse to leverage new data structure * - clearing after bulk create * - Rename to be more accurate - Add method to generate bulk create dictionary - Stub function for removing cases * - Updated class name - basing off of doc instead of schema * - Adding infrastructure to remove records from memory or the DB pending their state * - fix logic issue * - fix logic error * - added back list of cases * - fixed test_add_record * - Fix all case consistency tests * - Fix most lint errors * - remove print * - update rollback logic - fix most parse tests * - Update duplicate error messages - Added logging - Updated tests * - functionized dup manager a bit - updated validator to query errors from dup manager - added test for s1 duplicate detection * - Add test for S1 partial duplicate detection * - fix lint * - Add section 2 tests * - Fix reference error * - order by id * - parametrizing batch size - initial integration tests * - move creation logic in parse.py - add container to monitor what cases have serialized records - add test for in memory and DB based record removal with dupes detected * - added section 2 duplicate tests * - Test for partial duplicates - test doesnt behave as I would expect * - fix lint * - Fixed logic error where last case in file wouldnt be removed if it has cat4 errors * - Update SortedRecords to leverage an unsorted container to make bulk createing faster. Dont have to iterate/generate the required structure * - naming and doc string updates * - renaming functions - removing unnecessary members * - update doc strings * - Add test for family affiliation negating partial duplicity * - Updated to support duplicate checking on section 3/4 * - fix lint * - Update docstring * - Update to allow all record types to have duplicate detection * - add duplicate detection unit tests for all program types and record types * - fix lint * - Add cases for ssp * - Move parser fixtures to their own conftest.py * - Move conftest.py to the test folder * - removing whitespace * - Remove 'test' from fixture names * - Fix failing tests due to merge * - Update message for failing test * - Remove Vars from compose file - Remove unnecessary checks from cat4 validator - Updated error counts after removing check * - fixed test * - linting * - move partial hash checking logic to schema/fields * - Move fixtures to conftest.py * - Fix failing tests * - fixed tests * Revert "- move partial hash checking logic to schema/fields" This reverts commit ab379ce. * - Better way to do partial hash checking - Still in schema * - Moved hash generation into schema - updated tests and factories to now support agnostic hash generation * - Updated error message - fixed incorrect indentation on error counting * - move fixtures to conftest * add radio button for jurisdiction type * - fix lint * - fixed tests from merge conflict * add tests * - Update error creation logging to track all errors - Updated queries to most correct syntax for aggregates function - Updated tests * - lint * - move fixture * - Updated aggregates query to take into account cat4 errors * - lint * - import as Query * - moved fixture to correct location * - Updated cat4 duplicate errors to include fields that are duplicated * - fix lint * - update tests * - update from merge conflict * - Add additional exclusion criteria for necessary tests - add missing fixture decorators * - renamed parameters and classes to make more sense with their duties. - Added much more detailed descriptions where information surrounding duplicate detection can be confused. * - implement TODOs that came out of OH * - remove todo - add warning labels to shared object * - fix lint * - adding some generic parsing flow docs * update error string * fix tests * - Added useful debug logging to partial hash functions * - Updated duplicate detection to allow "partial duplicates" of s3 and s4 files - Added tests for partial dup s3/s4 * - updated file status to be consistent with expectations * - Updated stt filter to display stt name instead of code to make it easier to filter * - Remove custom filter in favor of dependency - Update STTFilter to leverage dependency filter * - update doc string * - fix lint * - Updated case_consistency validator to track which case has been validated to identify which case might need to be removed - decoupled dup manager and case consistency val error counts - Added more descriptive logging * - Update tests * - Add edge case cat4 test - unblock util function that was causing errors * - Updated manager to correctly delete serialized records - Updated DFS to correctly count total records and total records created - Updated cat4 test case to be more extensive * - Updated test * - fix lint * - Add elastic specific exception handling * - add extra case to file for duplicate detection * - Added creation of LogEntry objects in exception blocks * - Updated fiscal year filter to not list `All` twice - Updated STT filter lookup options * - fix lint * - Add line number to duplicate error ParserError objects * - Remove dup test class - Updated to use correct error generator * - fix lint * - Updated based on review feedback * - Update tests * - fix lint * don't show jurisdiction type for acf users * Create sprint-99-summary.md (#3038) * Create sprint-99-summary.md * Update docs/Sprint-Review/sprint-99-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> --------- Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Create sprint-100-summary.md (#3039) * Create sprint-100-summary.md * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> --------- Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * #3021 Adds FAQ update & release notes (#3040) * Adds FAQ update & release notes * Update product-updates/knowledge-center/index.html Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> * Update index.html * Adds request access updates * Update product-updates/knowledge-center/index.html Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> * Whitespace change to re-trigger full pipeline Whitespace change to re-trigger full pipeline --------- Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * 2693 cat2 messaging cleanup (#2963) * clean option string util * add friendly name and item num to cat 2 * fix linting * fix a couple tests * add parens to friendly names * lint * fix tests * fix more tests * fix merge duplication * fix tests * lint * fix tets * udpate required validator * Merge branch 'develop' into 2693-cat2-messaging-cleanup * fix tests * fix test * fix tests * fix lint --------- Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Eric Lipe <125676261+elipe17@users.noreply.github.com> * 2687 Added email text for sendgrid (#3050) * 2687 Added email text for sendgrid * 2687 linting --------- Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * - Added missing territories * adding placeholder for action to table * Frontpage link * Update TDP-environments-README.md --------- Co-authored-by: Eric Lipe <elipe@teamraft.com> Co-authored-by: Jan Timpe <jtimpe@goraft.tech> Co-authored-by: jtimpe <111305129+jtimpe@users.noreply.github.com> Co-authored-by: Eric Lipe <125676261+elipe17@users.noreply.github.com> Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: robgendron <163159602+robgendron@users.noreply.github.com> Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> Co-authored-by: Miles Reiter <reitermb@gmail.com> Co-authored-by: raftmsohani <97037188+raftmsohani@users.noreply.github.com>
* Adding action during deploy * Bad branch name, quick merge. (#3069) * - Initial commit for duplicate record manager for TANF section 1 records * - more efficient error generation * - Added a way to track precedence in the error messages to avoid generating less meaningful messages - Updated error generation * - Fix lint - Add docstrings * - Added support to delete deplicate records * - add support for bulk/raw delete * - Update deletes on rollbacks to raw delete since it wont hurt anything * - Updated elastic and django to support raw/bulk deleting * - Updating rollback test since it is very relevant now that we are doing raw deletes * - raise log level * - Update test * - remove import * - Update error generation to use record and schema - rename var * - Update error precedence logic * - fix lint * - use correct method * - remove extra call to bulk create errors * - move duplicate manager into case consistency validator - Starting to remove SortedRecordSchemaPairs from case consistency validator - updating to support in memory record removal if they havent been serialized * - update add_record logic to include rpt_month_year * - Move SortedRecordSchema pairs to util - Update case consistency validator to not use OG SortedRecordSchemaPairs - Update dup logic to not consider records on the same line * - remove unused import * - Update class to store records based on hash pending the section type * - Update parse to leverage new data structure * - clearing after bulk create * - Rename to be more accurate - Add method to generate bulk create dictionary - Stub function for removing cases * - Updated class name - basing off of doc instead of schema * - Adding infrastructure to remove records from memory or the DB pending their state * - fix logic issue * - fix logic error * - added back list of cases * - fixed test_add_record * - Fix all case consistency tests * - Fix most lint errors * - remove print * - update rollback logic - fix most parse tests * - Update duplicate error messages - Added logging - Updated tests * - functionized dup manager a bit - updated validator to query errors from dup manager - added test for s1 duplicate detection * - Add test for S1 partial duplicate detection * - fix lint * - Add section 2 tests * - Fix reference error * - order by id * - parametrizing batch size - initial integration tests * - move creation logic in parse.py - add container to monitor what cases have serialized records - add test for in memory and DB based record removal with dupes detected * - added section 2 duplicate tests * - Test for partial duplicates - test doesnt behave as I would expect * - fix lint * - Fixed logic error where last case in file wouldnt be removed if it has cat4 errors * - Update SortedRecords to leverage an unsorted container to make bulk createing faster. Dont have to iterate/generate the required structure * - naming and doc string updates * - renaming functions - removing unnecessary members * - update doc strings * - Add test for family affiliation negating partial duplicity * - Updated to support duplicate checking on section 3/4 * - fix lint * - Update docstring * - Update to allow all record types to have duplicate detection * - add duplicate detection unit tests for all program types and record types * - fix lint * - Add cases for ssp * - Move parser fixtures to their own conftest.py * - Move conftest.py to the test folder * - removing whitespace * - Remove 'test' from fixture names * - Fix failing tests due to merge * - Update message for failing test * - Remove Vars from compose file - Remove unnecessary checks from cat4 validator - Updated error counts after removing check * - fixed test * - linting * - move partial hash checking logic to schema/fields * - Move fixtures to conftest.py * - Fix failing tests * - fixed tests * Revert "- move partial hash checking logic to schema/fields" This reverts commit ab379ce. * - Better way to do partial hash checking - Still in schema * - Moved hash generation into schema - updated tests and factories to now support agnostic hash generation * - Updated error message - fixed incorrect indentation on error counting * - move fixtures to conftest * add radio button for jurisdiction type * - fix lint * - fixed tests from merge conflict * add tests * - Update error creation logging to track all errors - Updated queries to most correct syntax for aggregates function - Updated tests * - lint * - move fixture * - Updated aggregates query to take into account cat4 errors * - lint * - import as Query * - moved fixture to correct location * - Updated cat4 duplicate errors to include fields that are duplicated * - fix lint * - update tests * - update from merge conflict * - Add additional exclusion criteria for necessary tests - add missing fixture decorators * - renamed parameters and classes to make more sense with their duties. - Added much more detailed descriptions where information surrounding duplicate detection can be confused. * - implement TODOs that came out of OH * - remove todo - add warning labels to shared object * - fix lint * - adding some generic parsing flow docs * update error string * fix tests * - Added useful debug logging to partial hash functions * - Updated duplicate detection to allow "partial duplicates" of s3 and s4 files - Added tests for partial dup s3/s4 * - updated file status to be consistent with expectations * - Updated stt filter to display stt name instead of code to make it easier to filter * - Remove custom filter in favor of dependency - Update STTFilter to leverage dependency filter * - update doc string * - fix lint * - Updated case_consistency validator to track which case has been validated to identify which case might need to be removed - decoupled dup manager and case consistency val error counts - Added more descriptive logging * - Update tests * - Add edge case cat4 test - unblock util function that was causing errors * - Updated manager to correctly delete serialized records - Updated DFS to correctly count total records and total records created - Updated cat4 test case to be more extensive * - Updated test * - fix lint * - Add elastic specific exception handling * - add extra case to file for duplicate detection * - Added creation of LogEntry objects in exception blocks * - Updated fiscal year filter to not list `All` twice - Updated STT filter lookup options * - fix lint * - Add line number to duplicate error ParserError objects * - Remove dup test class - Updated to use correct error generator * - fix lint * - Updated based on review feedback * - Update tests * - fix lint * don't show jurisdiction type for acf users * Create sprint-99-summary.md (#3038) * Create sprint-99-summary.md * Update docs/Sprint-Review/sprint-99-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> --------- Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Create sprint-100-summary.md (#3039) * Create sprint-100-summary.md * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> * Update docs/Sprint-Review/sprint-100-summary.md Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> --------- Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * #3021 Adds FAQ update & release notes (#3040) * Adds FAQ update & release notes * Update product-updates/knowledge-center/index.html Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> * Update index.html * Adds request access updates * Update product-updates/knowledge-center/index.html Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> * Whitespace change to re-trigger full pipeline Whitespace change to re-trigger full pipeline --------- Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * 2693 cat2 messaging cleanup (#2963) * clean option string util * add friendly name and item num to cat 2 * fix linting * fix a couple tests * add parens to friendly names * lint * fix tests * fix more tests * fix merge duplication * fix tests * lint * fix tets * udpate required validator * Merge branch 'develop' into 2693-cat2-messaging-cleanup * fix tests * fix test * fix tests * fix lint --------- Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Eric Lipe <125676261+elipe17@users.noreply.github.com> * 2687 Added email text for sendgrid (#3050) * 2687 Added email text for sendgrid * 2687 linting --------- Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: Andrew <84722778+andrew-jameson@users.noreply.github.com> * - Added missing territories * adding placeholder for action to table * Frontpage link * Update TDP-environments-README.md --------- Co-authored-by: Eric Lipe <elipe@teamraft.com> Co-authored-by: Jan Timpe <jtimpe@goraft.tech> Co-authored-by: jtimpe <111305129+jtimpe@users.noreply.github.com> Co-authored-by: Eric Lipe <125676261+elipe17@users.noreply.github.com> Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: robgendron <163159602+robgendron@users.noreply.github.com> Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> Co-authored-by: Miles Reiter <reitermb@gmail.com> Co-authored-by: raftmsohani <97037188+raftmsohani@users.noreply.github.com> * swapping lines, readying for deploy * YAML spacing. * bypassing PR checks for faster turnaround * Pointing to blank revision, fixed to generic/latest * Dynamic filenames for testing differing environments. * moving badges out of table for sizing, hopefully * Newlines * appending git info to frontend base page * Feat/1620 merge pg upgrade (#3072) * - Upgraded local to postrgres 15.7 * - updating version on terraform * - disabling tests for now * - adding timeouts to TF for safety * - Deleting DB from TF state * - re-add DB * - remove deployment until DB is ready * - updated apt.yml to have pg15 client - updated deploy to only run apt commands * - re-enable deply dev * - UPdate client version - re-enable migrations * - really disable tests --------- Co-authored-by: Eric Lipe <elipe@teamraft.com> Co-authored-by: andrew-jameson <ajameson@teamraft.com> * removing comment * undoing mass line comments * Update .circleci/build-and-test/workflows.yml * Apply suggestions from code review Co-authored-by: Alex P. <63075587+ADPennington@users.noreply.github.com> --------- Co-authored-by: andrew-jameson <ajameson@teamraft.com> Co-authored-by: Eric Lipe <elipe@teamraft.com> Co-authored-by: Jan Timpe <jtimpe@goraft.tech> Co-authored-by: jtimpe <111305129+jtimpe@users.noreply.github.com> Co-authored-by: Eric Lipe <125676261+elipe17@users.noreply.github.com> Co-authored-by: Alex P <63075587+ADPennington@users.noreply.github.com> Co-authored-by: robgendron <163159602+robgendron@users.noreply.github.com> Co-authored-by: Lauren Frohlich <61251539+lfrohlich@users.noreply.github.com> Co-authored-by: Miles Reiter <reitermb@gmail.com> Co-authored-by: raftmsohani <97037188+raftmsohani@users.noreply.github.com>
Summary of Changes
Pull request closes #2687 _
Email task needs to send html content to admin users. When sending html, django email also has to submit text content since Sendgrid email service doesn't accept empty text email.
How to Test
In develop branch:
Change your branch to current PR branch and verify you can receive the email.
Deliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
OFA System Admin
andOFA Admin
receives one email per day (ideal run time between 0100-0200 UTC) presenting a count of all users with an "Access Request" status as of that day. Prototype in notes.-
[ ] Django Admin console has additional filter to show timestamp of when a user's status changed to, "Access request"not neededlfrohlich
and/oradpennington
confirmed that ACs are met.Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):