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

Update npgsqldbtype for created date time #90

Merged
merged 9 commits into from
Mar 18, 2024
Merged

Update npgsqldbtype for created date time #90

merged 9 commits into from
Mar 18, 2024

Conversation

acn-dgopa
Copy link
Member

Description

Related Issue(s)

  • #{issue number}

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@acn-dgopa acn-dgopa requested a review from TheTechArch March 12, 2024 09:29
@acn-dgopa acn-dgopa linked an issue Mar 12, 2024 that may be closed by this pull request
Created date and time which shows in logs is different than the actual time in audit log #81
Closed
Alxandr
Alxandr reviewed Mar 14, 2024
View reviewed changes
src/Altinn.Auth.AuditLog.Core/Models/AuthenticationEvent.cs Outdated Show resolved Hide resolved
src/Altinn.Auth.AuditLog.Core/Models/AuthenticationEvent.cs Outdated Show resolved Hide resolved
src/Altinn.Auth.AuditLog.Core/Models/AuthorizationEvent.cs Outdated Show resolved Hide resolved
src/Altinn.Auth.AuditLog.Core/Models/AuthorizationEvent.cs Outdated Show resolved Hide resolved
src/Functions/Altinn.Auth.AuditLog.Functions/Models/AuthenticationEvent.cs Outdated Show resolved Hide resolved
src/Functions/Altinn.Auth.AuditLog.Functions/Models/AuthenticationEvent.cs Outdated Show resolved Hide resolved
src/Functions/Altinn.Auth.AuditLog.Functions/Models/AuthorizationEvent.cs Outdated Show resolved Hide resolved
src/Functions/Altinn.Auth.AuditLog.Functions/Models/AuthorizationEvent.cs Outdated Show resolved Hide resolved
@Alxandr
Copy link
Contributor

Alxandr commented Mar 14, 2024

Also, there seems to be something wrong with the test setup here:

image

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@acn-dgopa acn-dgopa merged commit cee42f6 into main Mar 18, 2024
3 checks passed
@acn-dgopa acn-dgopa deleted the bug/timezone-for-datetime branch March 18, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alxandr Alxandr Alxandr approved these changes

@TheTechArch TheTechArch Awaiting requested review from TheTechArch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Created date and time which shows in logs is different than the actual time in audit log
2 participants
@acn-dgopa @Alxandr