Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: Custom logic on Last Update date for complaints #876

Open
wants to merge 48 commits into
base: release/1.0.1
Choose a base branch
from
Open

feat: Custom logic on Last Update date for complaints #876

wants to merge 48 commits into from

Conversation

afwilcox
Copy link
Collaborator

@afwilcox afwilcox commented Jan 16, 2025 •
edited by github-actions bot
Loading

Description

Please provide a summary of the change and the issue fixed. Please include relevant context. List dependency changes.

Fixes # CE-1335

Introduces a new field on the complaint table to track the date the complaint was last updated from a business perspective as the database audit field is insufficient in that the the business doesn't consider a fresh complaint to be updated, and also wants the date to be updated when surrounding data (including data from other systems such as attachments and case management) are touched.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • New Complaint from webEOC - date is not updated
  • Updated Complaint from webEOC - date is not updated
  • Add action taken to complaint in webEOC - date is not updated (as per business rules)
  • Edit Complaint details - date is updated
  • Assign Complaint (both from list view and details - date is updated
  • Update Status - date is updated
  • Add outcome attachments - date is updated
  • Edit HWCR Outcome sections (any) - date is updated
  • Edit CEEB sections (any) - date is updated
  • Sort list view on date - nulls are at the bottom of the sort regardless of order

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

dk-bcps and others added 30 commits December 24, 2024 11:39
@dk-bcps
fix: CE-1330-Attachment-Sort-Order-on-Export (#838)
c1fc16e
@afwilcox @jon-funk
@DerekRoberts @nayr974 @Scarlett-Truong @dk-bcps
feat: CEEB Export Scripts and Export Merge Tool (#832)
89e016b 
Co-authored-by: jon-funk <[email protected]>
Co-authored-by: Derek Roberts <[email protected]>
Co-authored-by: Ryan Rondeau <[email protected]>
Co-authored-by: Scarlett <[email protected]>
Co-authored-by: dmitri-korin-bcps <[email protected]>
@mishraomp
chore: remove limits (cpu and memory) (#842)
b05ba71
@dk-bcps @afwilcox
fix: CE-1306-Update-validation-for-CEEB-Authorization-number-field-to…
d866c43 
…-accept-Pesticide-related-authorizations (#840)

Co-authored-by: afwilcox <[email protected]>
@afwilcox
feat: Link complaints to any status and label changes (#846)
a627209
@nayr974 @mishraomp @afwilcox
fix: CE-1327 Fix Render Loop Bug (#848)
a0b4d74 
Co-authored-by: OMPRAKASH MISHRA <[email protected]>
Co-authored-by: afwilcox <[email protected]>
@dk-bcps @afwilcox
feat: CE-1306-update-validation-for-ceeb-authorization-number-field-t…
8f8db05 
…o-accept-pesticide-related-authorizations (#850)

Co-authored-by: afwilcox <[email protected]>
@mikevespi @afwilcox
feat: Ce 1222 load tests (#849)
4445d65 
Co-authored-by: afwilcox <[email protected]>
@nayr974
fix: Clear Event Listeners On Unmount (#851)
6ef5734
@nayr974
chore: CM-1300 Test DB Connection In Health Check (#854)
c8a2ca0
@afwilcox @jon-funk
@DerekRoberts @nayr974 @Scarlett-Truong @dk-bcps
chore: NatCom bulk data loader (#839)
a75d34c 
Co-authored-by: jon-funk <[email protected]>
Co-authored-by: Derek Roberts <[email protected]>
Co-authored-by: Ryan Rondeau <[email protected]>
Co-authored-by: Scarlett <[email protected]>
Co-authored-by: dmitri-korin-bcps <[email protected]>
@afwilcox
feat: New Equipment Codes (#847)
aa7590c
@Scarlett-Truong @afwilcox
@jon-funk @DerekRoberts @nayr974 @dk-bcps
feat: CE-501 Admin page (#852)
351fb30 
Co-authored-by: afwilcox <[email protected]>
Co-authored-by: jon-funk <[email protected]>
Co-authored-by: Derek Roberts <[email protected]>
Co-authored-by: Ryan Rondeau <[email protected]>
Co-authored-by: dmitri-korin-bcps <[email protected]>
@dk-bcps @afwilcox
chore: CE-1323-filtering-by-unassigned-causes-a-timeout-in-the-list-v…
589b6ba 
…iew (#855)

Co-authored-by: afwilcox <[email protected]>
@mikevespi @afwilcox
feat: CE-1296 add Inspector and Province-wide roles (#857)
801ece5 
Co-authored-by: afwilcox <[email protected]>
@Scarlett-Truong
fix: CE-1339 Linking to child complaint (#858)
4c0e47e
@mikevespi
chore: CE-1354 Add new roles to list in edit user (#859)
9899cd3
@nayr974 @afwilcox
feat: CE-1336 (#856)
2ce000a 
Co-authored-by: afwilcox <[email protected]>
@nayr974 @mishraomp @afwilcox
chore: CE-1336 rename script (#864)
c1db80f 
Co-authored-by: OMPRAKASH MISHRA <[email protected]>
Co-authored-by: afwilcox <[email protected]>
@nayr974 @mishraomp @afwilcox
chore: CE-1336 (#865)
625a11e 
Co-authored-by: OMPRAKASH MISHRA <[email protected]>
Co-authored-by: afwilcox <[email protected]>
@jon-funk @afwilcox
@DerekRoberts @nayr974 @Scarlett-Truong @dk-bcps
feat: crunchy (duplicate to merge) (#861)
bed952c 
Co-authored-by: afwilcox <[email protected]>
Co-authored-by: Derek Roberts <[email protected]>
Co-authored-by: Ryan Rondeau <[email protected]>
Co-authored-by: Scarlett <[email protected]>
Co-authored-by: dmitri-korin-bcps <[email protected]>
@jon-funk
fix: passin s3 config to pgbackrest (#867)
71b5da1
@jon-funk
fix: env mappings for dbdeployer (#868)
8eb0cb4
@jon-funk
fix: crunchy s3 missing (#869)
fe996d8
@jon-funk
fix: backend template with crunchy (#870)
057d519
@jon-funk
chore: enable crunchy in test (#871)
62370a2
@jon-funk
fix: crunchy value keys (#872)
5dffb40
@afwilcox
feat: Complaint Last Update Column
d1d9211
@afwilcox
feat: last updated date for webEOC scenarios
f3ea104
@afwilcox
feat: Set last update complaint date on edit
4896986
afwilcox and others added 14 commits January 15, 2025 10:15
@afwilcox
feat: upd cplt last update field on status change
3698f16
@afwilcox
feat: update complaint last updated on assign
d342b8f
@afwilcox
chore: remove unused import
3f0d164
@afwilcox
feat: Update Complaint Last Update for HWCR Pt. 1
c4dd056
@afwilcox
feat: HWCR last update changes part 1.5
196b4b5
@afwilcox
Merge branch 'release/1.0.1' into CE-1335-last-update-date
595623a
@afwilcox
Merge branch 'CE-1335-last-update-date' of https://github.com/bcgov/n…
666ac14 
…r-compliance-enforcement into CE-1335-last-update-date
@afwilcox
feat: finish complaint last updated date for HWCR
1604940
@afwilcox
feat: CEEB Outcomes update complaint last upd date
97d99d5
@afwilcox
feat: mandatory parameter
a611b69
@afwilcox
feat: Tie new updated date into FE
51ad096
@nayr974
Update person_complaint_xref.service.ts
ae73ccf
@afwilcox
feat: Attachment upload actions update date
71a4fee
@afwilcox
Merge branch 'CE-1335-last-update-date' of https://github.com/bcgov/n…
6134020 
…r-compliance-enforcement into CE-1335-last-update-date
@afwilcox afwilcox marked this pull request as ready for review January 17, 2025 18:41
@afwilcox afwilcox changed the title Ce 1335 last update date feat: Custom logic on Last Update date for complaints Jan 17, 2025
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
2.5% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@afwilcox @dk-bcps @mishraomp @nayr974 @mikevespi @Scarlett-Truong @jon-funk