[romanetar]

ref: https://app.clickup.com/t/9014802374/86bc72z5c

Summary

Part of a stack of 5 PRs that make every configured audit formatter handle many-to-many collection changes (ADR-004). Without a case for EVENT_COLLECTION_MANYTOMANY_UPDATE / DELETE, a configured formatter returns null and AuditLogOtlpStrategy drops the entry, so attaching or detaching items left no audit trail.

Covers: PresentationType and SummitEventType (allowed_ticket_types), SummitMediaUploadType (presentation_types) and SummitDocument (event_types).

One commit per formatter, each with its tests under tests/OpenTelemetry/Formatters/ (update with added and removed ids, delete with preloaded deleted_ids, and null when there is no diff).

Stack

Review in order; each PR is green on its own. The coverage guard test and the ADR-004 update land in the last PR (#671).

1. fix(audit): log many-to-many collection changes on CFP selection entities #667 CFP: selection plans, track chairs, invitations, categories, speakers
2. fix(audit): log many-to-many collection changes on CFP type entities #668 CFP: presentation types, event types, media upload types, documents
3. fix(audit): log many-to-many collection changes on registration entities #669 Registration: badge types, attendee badges, extra questions, tax types
4. fix(audit): log many-to-many collection changes on promo and discount codes #670 Promo and discount codes
5. fix(audit): log many-to-many collection changes on summit, sponsor and venue rooms and add coverage guard #671 Summit, sponsor, venue rooms + guard test + ADR-004

🤖 Generated with Claude Code

[coderabbitai]

Important

• Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 9a381429-baed-4897-8f6b-528884dcb9a2

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[github-actions]

📘 OpenAPI / Swagger preview

➡️ https://OpenStackweb.github.io/summit-api/openapi/pr-668/

This page is automatically updated on each push to this PR.

[github-actions]

📘 OpenAPI / Swagger preview

➡️ https://OpenStackweb.github.io/summit-api/openapi/pr-668/

This page is automatically updated on each push to this PR.