Skip to content

fix(reports): honor report-specific PDF branding presets - #996

Merged
Zacgoose merged 19 commits into
CyberDrain:devfrom
MartinRain:fix/bec-branding-preset-minimal
Oct 21, 2026
Merged

Zacgoose merged 19 commits into
CyberDrain:devfrom
MartinRain:fix/bec-branding-preset-minimal

Conversation

@MartinRain

Copy link
Copy Markdown
Contributor

Summary

Fix report-specific PDF branding presets being ignored by the BEC (full and summary), Sharing, Permissions, Mail Flow, and Shadow AI PDF reports.

  • Forward the configured brandingPresetId from the UI through the backend to tenant naming and PDF rendering.
  • Apply distinct full and summary branding presets to the BEC evidence ZIP PDFs.
  • Preserve fallback to global branding when no report-specific preset is configured.
  • Keep changes scoped to affected report components and endpoints, without modifying the shared PDF renderer or adding Graph requests.

Validation

  • Pester: 18/18 targeted tests passed.
  • Vitest: 10/10 targeted tests passed.
  • Functional: All affected reports successfully tested in a running CIPP instance.

Fixes #967

@Zacgoose
Zacgoose merged commit 3da756b into CyberDrain:dev Oct 21, 2026
16 of 18 checks passed
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