Audit logging and code style #12
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cableman
changed the title
cableman
force-pushed
the
feature/os2form-audit
branch
2 times, most recently
from
4ccf6e4 to
1b72571
Compare
cableman
changed the title
cableman
force-pushed
the
feature/os2form-audit
branch
17 times, most recently
from
aae9895 to
5d6927c
Compare
cableman
changed the title
cableman
force-pushed
the
feature/os2form-audit
branch
5 times, most recently
from
26a9d98 to
63db687
Compare
rimi-itk
approved these changes
May 14, 2024
Co-authored-by: Mikkel Ricky <[email protected]>
cableman
force-pushed
the
feature/os2form-audit
branch
from
66afdf1 to
711de21
Compare
…2web_datalookup into feature/os2form-audit
ITKDev: Minor fixes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add audit logging to personal information lookup requests.
NOTE: We have to make sure that we release this in a major release as renaming the interface is a breaking change (it's used in https://github.com/OS2Forms/os2forms/blob/develop/modules/os2forms_digital_post/src/Helper/DigitalPostHelper.php#L93-L95 and other places in the OS2Forms universe).