-
Notifications
You must be signed in to change notification settings - Fork 5
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
Updated created datetime to utc #784
Conversation
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.
Can you add the #nullable enable
to the EventLog classes where you're using nullable to get rid of the warnings?
Would also be nice to fix the technical debt of using ISystemClock which is obsolete, as I guess this will be one of the last changes on AuditLog for a while?
thats right. I was waiting to rewrite both in authentication and authorization when upgraded to .net8. I see authz is updated to .net8. I will fix this |
Quality Gate passedIssues Measures |
Description
Related Issue(s)
Verification
Documentation