Skip to content

Use new access_list_title when showing audit events#54350

Merged
EdwardDowling merged 7 commits intomasterfrom
edwarddowling/access-list-audit-ui
May 1, 2025
Merged

Use new access_list_title when showing audit events#54350
EdwardDowling merged 7 commits intomasterfrom
edwarddowling/access-list-audit-ui

Conversation

@EdwardDowling
Copy link
Copy Markdown
Contributor

Use access list title when displaying audit logs instead of the uuid name
And bump /e ref

changelog: Show human readable title for access list audit logs

Comment thread web/packages/teleport/src/services/audit/makeEvent.ts Outdated
Copy link
Copy Markdown
Contributor

@flyinghermit flyinghermit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also update the web/packages/teleport/src/Audit/fixtures/index.ts fixture file with access_list_title field. That is used in audit event story for reference.

Comment thread web/packages/teleport/src/services/audit/makeEvent.ts Outdated
Comment thread e
@EdwardDowling EdwardDowling force-pushed the edwarddowling/access-list-audit-ui branch from 9a4879d to 1c8ee45 Compare April 30, 2025 13:05
@EdwardDowling EdwardDowling enabled auto-merge May 1, 2025 15:01
@EdwardDowling EdwardDowling added this pull request to the merge queue May 1, 2025
Merged via the queue into master with commit a1869d1 May 1, 2025
40 checks passed
@EdwardDowling EdwardDowling deleted the edwarddowling/access-list-audit-ui branch May 1, 2025 15:20
@backport-bot-workflows
Copy link
Copy Markdown
Contributor

@EdwardDowling See the table below for backport results.

Branch Result
branch/v16 Create PR
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants