Skip to content

Version bump for audit logs release#19

Merged
chris-okuda merged 2 commits intomainfrom
chrisokuda/sc-32886/version_bump_audit_log
Sep 9, 2025
Merged

Version bump for audit logs release#19
chris-okuda merged 2 commits intomainfrom
chrisokuda/sc-32886/version_bump_audit_log

Conversation

@chris-okuda
Copy link
Copy Markdown
Collaborator

@chris-okuda chris-okuda commented Aug 28, 2025

Scope of changes

Bump the version and adjust the endpoint for the AuditLogs resource from complianceauditlogs to auditlogs (the route in Envoy was changed in a previous pull review).

Type of change

  • bug fix
  • new feature
  • documentation
  • other (describe)

Acceptance criteria

Nothing special.

Author checklist

  • I have manually tested the change and/or added automation in the form of unit tests or integration tests
  • I have updated the dependencies list
  • I have added new test fixtures as needed to support added tests
  • I have added or updated the documentation
  • Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)

Reviewer(s) checklist

  • Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
  • To the best of my ability, I believe that this PR represents a good solution to the specified problem and that it should be merged into the main code base.

Copy link
Copy Markdown
Collaborator

@daniellemaxwell daniellemaxwell 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!

@chris-okuda chris-okuda marked this pull request as ready for review September 9, 2025 16:46
@chris-okuda chris-okuda merged commit 3210543 into main Sep 9, 2025
3 of 4 checks passed
@chris-okuda chris-okuda deleted the chrisokuda/sc-32886/version_bump_audit_log branch September 9, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants