-
Notifications
You must be signed in to change notification settings - Fork 14
fix: Removes duplicate log events. #6575
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
base: main
Are you sure you want to change the base?
Conversation
🧪 Review environmenthttps://rnsgynh6sfyywkkj4moei5cgmm0wegiz.lambda-url.ca-central-1.on.aws/ |
|
Thinking maybe this wasn't caught in the original PR but when I change classification I'm not seeing the changed to value "cmj8qf3du0001nb0dejo7nx98","tim.arney@cds-snc.ca","""Data classification changed""","2025-12-16T15:34:28.633Z", |
timarney
left a comment
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.
Will wait for QA but outside of my comment on classification this seems to be working well.
22c132c to
3e0f4f2
Compare
3e0f4f2 to
12b04ec
Compare
QA testingLanguage
Form
Settings
API
Team members
Responses
Issues
|







Fixes #6547
Checks what elements of a template are changed before throwing log events that the element was indeed changed.
Remaining Issues:
Download form (says Form JSON updated, don't include)
API key generation/deletion (does not include actual details, only placeholders)
Manage access (missing strings to make it user-readable, sometimes missing attributes {{userEmail}}
To be fixed in a later itteration:
#6627