Skip to content

Feat/admin audit logging - #8561

Merged
GhassenKefi merged 6 commits into
mainfrom
feat/admin-audit-logging
Oct 26, 2026
Merged

GhassenKefi merged 6 commits into
mainfrom
feat/admin-audit-logging

Conversation

@GhassenKefi

Copy link
Copy Markdown
Contributor
  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

@github-actions

Copy link
Copy Markdown
Contributor

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 202760 was 198420 (+2.18%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

Creating, cloning, importing, archiving and unarchiving boards, moving
them to the trash and restoring them, permanently deleting them (after
the trash retention or together with their team) and transferring
their ownership (directly, through occ deck:transfer-ownership or when
the owner leaves the team) are now written to the admin audit log.

The operations are collected in the new OCA\Deck\AuditLog\Operation
enum, with a short description of each for the documentation.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
Sharing a board with a user, group, team or remote user, changing the
permissions of a share, removing a share, leaving a shared board and
receiving a board share from another server are now written to the
admin audit log. Deck shares are not core shares, so admin_audit did
not see any of them before.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
…it log

Creating cards, moving them to the trash and restoring them, moving
them to another board (through an update or drag and drop), archiving
and unarchiving them, assigning and unassigning users, groups and
teams, moving stacks to the trash and restoring them, and deleting
labels are now written to the admin audit log. Card titles and
descriptions are never logged.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
Attaching a file stored by Deck to a card, deleting it and restoring it
are now written to the admin audit log. These files live in the Deck
app data, so the core file events do not record them. Attachments of
the type "file" are Nextcloud files shared with the card and are
already logged as shares, so they are skipped.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
Adding, editing and deleting comments on cards are now written to the
admin audit log. Only the fact is logged with the comment and card ID,
never the comment text.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
Limiting Deck to certain groups and turning board federation on or off
are now written to the admin audit log. Personal preferences such as
the calendar or notification settings are not logged.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
@GhassenKefi
GhassenKefi force-pushed the feat/admin-audit-logging branch from 6a9b824 to 2eb7266 Compare October 19, 2026 12:23
@github-actions

Copy link
Copy Markdown
Contributor

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 202822 was 198420 (+2.21%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@GhassenKefi
GhassenKefi merged commit dfd6ad9 into main Oct 26, 2026
43 checks passed
@GhassenKefi
GhassenKefi deleted the feat/admin-audit-logging branch October 26, 2026 11:15
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.

2 participants