Skip to content

feat(core): Add audit logging - #3059

Merged
elias-summermatter merged 6 commits into
masterfrom
feat/add-audit-logging
Sep 28, 2026
Merged

elias-summermatter merged 6 commits into
masterfrom
feat/add-audit-logging

Conversation

@elias-summermatter

Copy link
Copy Markdown
Member

What changes

Changelog

Every branch that changes something an editor user can notice adds one entry
under libs/api/changelogs/. It is picked up on deploy and shown to editors in
the notification centre.

  • I added an entry — npm run changelog:generate drafts it from the branch
    diff in all three languages, npm run changelog:create -- "Short title"
    scaffolds one by hand. Use --action-required when every instance has to
    do something manually.
  • Nothing user-facing changed — I labelled this pull request
    no-changelog.

Run npm run changelog:check locally to see what CI will say.

Anything reviewers should know

@codecov

codecov Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 39.96350% with 329 lines in your changes missing coverage. Please review.
✅ Project coverage is 19.64%. Comparing base (8a719eb) to head (6087739).
⚠️ Report is 22 commits behind head on master.

Files with missing lines Patch % Lines
libs/audit-log/api/src/lib/audit-log.model.ts 0.00% 76 Missing ⚠️
...ps/editor/src/app/routes/auditLog/auditLogList.tsx 0.00% 69 Missing ⚠️
libs/one/api/src/lib/medium-audit-log.model.ts 0.00% 61 Missing ⚠️
libs/audit-log/api/src/lib/audit-log.service.ts 75.80% 6 Missing and 9 partials ⚠️
libs/one/api/src/lib/medium-audit-log.resolver.ts 0.00% 15 Missing ⚠️
libs/stats/api/src/lib/medium-stats.model.ts 61.76% 13 Missing ⚠️
libs/audit-log/api/src/lib/audit-log.resolver.ts 0.00% 11 Missing ⚠️
libs/editor/api/src/lib/graphql.ts 0.00% 10 Missing ⚠️
libs/peering/api/src/lib/remote/graphql.ts 0.00% 10 Missing ⚠️
libs/website/api/src/lib/graphql.ts 0.00% 10 Missing ⚠️
... and 8 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3059      +/-   ##
==========================================
+ Coverage   19.03%   19.64%   +0.60%     
==========================================
  Files        1698     1783      +85     
  Lines       48744    52720    +3976     
  Branches    10945    12210    +1265     
==========================================
+ Hits         9278    10356    +1078     
- Misses      37933    40718    +2785     
- Partials     1533     1646     +113     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown

PR 3059 with branch feat/add-audit-logging%0A%0A

No review build running

%0A%0AThere is currently no review build deployed for this PR.%0A%0A

Want to deploy a review build?

%0A%0A1. Check the Slack sheet to see which review instance is currently unused%0A2. Add or select a label to this PR with the format: review_<number>_<project>%0A%0AExample: review_01_bajour%0A%0AWhere:%0A - <number> is the review runner number (e.g. 01, 02, ...)%0A - <project> is the project name (e.g. bajour, tsri, ...)

Comment thread libs/api/prisma/migrations/20260924100000_add_audit_log/migration.sql Outdated

@Itrulia Itrulia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Small things, else looks good

Comment thread libs/audit-log/api/src/lib/audit-log-retention.service.ts Outdated
@elias-summermatter
elias-summermatter merged commit cd280d5 into master Sep 28, 2026
48 of 49 checks passed
@elias-summermatter
elias-summermatter deleted the feat/add-audit-logging branch September 28, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants