Skip to content

Feat/hrp 296 audit case fields#262

Merged
paymonsattar merged 1 commit intomainfrom
feat/hrp-296-audit-case-fields
Apr 14, 2026
Merged

Feat/hrp 296 audit case fields#262
paymonsattar merged 1 commit intomainfrom
feat/hrp-296-audit-case-fields

Conversation

@paymonsattar
Copy link
Copy Markdown
Collaborator

@paymonsattar paymonsattar commented Apr 1, 2026

Describe your changes

Wires up field-level auditing for case updates. Depends on THIS PR

  • updateCaseData now fetches the full case with relations before updating so resolvers can compare old vs new values.
  • Entra group members are loaded for user display name resolution.
  • Scalar fields produce FIELD_SET, FIELD_UPDATED, or FIELD_CLEARED entries with old and new values.
  • List-type fields (contacts, inspectors, procedures, outcomes, related/linked cases) are diffed via the new resolvers from PR 1.
  • Case history view model updated to support the new files property and bulk file accordion in the template.

Issue ticket number and link

https://pins-ds.atlassian.net/browse/HRP-296

Copy link
Copy Markdown
Collaborator

@oscargay oscargay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Some comments from me, what do you think? Thanks

Comment thread apps/manage/src/app/views/cases/case-history/view-model.ts Outdated
Comment thread apps/manage/src/app/views/cases/view/update-case.ts Outdated
Comment thread apps/manage/src/app/views/cases/view/update-case.ts
Comment thread apps/manage/src/app/views/cases/view/update-case.ts Outdated
Comment thread apps/manage/src/app/views/cases/view/update-case.ts Outdated
Comment thread apps/manage/src/app/views/cases/view/update-case.ts Outdated
@paymonsattar paymonsattar force-pushed the feat/hrp-296-audit-case-fields branch from 6ae3b69 to c7f08ca Compare April 8, 2026 10:50
@oscargay
Copy link
Copy Markdown
Collaborator

LGTM!

feat: granular field-level auditing for case updates

feat: wip

fix: tests
@paymonsattar paymonsattar force-pushed the feat/hrp-296-audit-case-fields branch from 9be912e to 45a5541 Compare April 14, 2026 10:02
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines:
Successfully started running 1 pipeline(s).
2 pipeline(s) were filtered out due to trigger conditions.

@paymonsattar paymonsattar merged commit b91dd14 into main Apr 14, 2026
5 checks passed
@paymonsattar paymonsattar deleted the feat/hrp-296-audit-case-fields branch April 14, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants