...

[dspangenberg]

Summary by CodeRabbit

• New Features

  • Neue Einstellungen für die Rechnungsstellung wurden ergänzt und stehen nun als eigener Konfigurationsbereich zur Verfügung.

• Bug Fixes

  • Die Standardwerte für ZUGFeRD-bezogene Kontaktdaten wurden vereinfacht, sodass fehlende Angaben nun korrekt mit einem festen Standard vorbelegt werden.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 83b430ce-9358-4031-8cff-c8b7be4ab327

📥 Commits

Reviewing files that changed from the base of the PR and between 8fc0357 and 538697f.

📒 Files selected for processing (2)

• app/Settings/InvoicingSettings.php
• database/migrations/tenant/2026_07_02_183342_zugferd_settings_update.php

---

Walkthrough

Diese Änderung fügt eine neue InvoicingSettings-Klasse hinzu und vereinfacht die bestehende Zugferd-Settings-Migration, indem die Umbenennungslogik entfernt und nur noch der Schlüssel zugferd.seller_contact_person_id mit Default-Wert 0 gesetzt wird.

Changes

Settings-Änderungen

Layer / File(s)	Summary
Neue InvoicingSettings-Klasse app/Settings/InvoicingSettings.php	Neue Klasse InvoicingSettings (erbt von Settings) mit statischer group()-Methode, die 'invoicing' zurückgibt.
Vereinfachung der Zugferd-Migration database/migrations/tenant/2026_07_02_183342_zugferd_settings_update.php	Nicht mehr benötigter GeneralSettings-Import entfernt; Migrationslogik reduziert auf das Hinzufügen von zugferd.seller_contact_person_id mit Default 0, ohne vorheriges Umbenennen von zugferd.seller_contact_id.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

Ein neues Setting hoppelt herein,
invoicing soll's fortan sein.
Die Migration wird schlank und klar,
kein Umbenennen mehr, wunderbar!
🐰 Karottenkonfetti, die Arbeit ist getan!

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch develop

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 PHPStan (2.2.2)

PHPStan was skipped because the config uses disallowed bootstrapFiles, bootstrapFile, or includes directives.

---

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

_{Comment @coderabbitai help to get the list of available commands.}