[NOTASK] optional kycStep status update

[Yannick1712]

Release Checklist

Pre-Release

• Check migrations
  • No database related infos (sqldb-xxx)
  • Impact on GS (new/removed columns)
• Check for linter errors (in PR)
• Test basic user operations (on DFX services)
  • Login/logout
  • Buy/sell payment request
  • KYC page

Post-Release

• Test basic user operations
• Monitor application insights log

[TaprootFreak]

Dieser PR erfüllt in seiner aktuellen Form weder die Anforderungen aus CONTRIBUTING.md noch die notwendige Nachvollziehbarkeit für eine Änderung am KYC-Update-Verhalten.

1. Problembeschreibung und Motivation fehlen. Bitte dokumentiert den konkret betroffenen Ablauf, das aktuelle Fehlverhalten, das erwartete Verhalten und weshalb status optional werden muss.
2. Falscher Validator für ein Update-DTO. Laut CONTRIBUTING.md müssen optionale Felder in Update-DTOs @IsOptionalButNotNull() aus den Shared Validators verwenden. @IsOptional() akzeptiert auch explizites null, obwohl die Datenbankspalte status nicht nullable ist.
3. TypeScript-Typ und Runtime-Validierung widersprechen sich. Wenn das Feld optional ist, muss es als status?: ReviewStatus deklariert werden.
4. Consumer-Synchronisierung fehlt. Die Änderung betrifft den API-Vertrag. CONTRIBUTING.md verlangt bei DTO-/Interface- und API-Vertragsänderungen die entsprechenden Service- und Frontend-Anpassungen. Im DFXswiss/services-Consumer ist status weiterhin verpflichtend typisiert. Bitte synchronisieren oder den zugehörigen PR verlinken.
5. Testabdeckung fehlt. Bitte ergänzt mindestens Tests für:
   • Request ohne status
   • gültigen und ungültigen status
   • explizites status: null
   • partielle Updates von result, comment und sequenceNumber
   • Persistenz: Ein ausgelassener Status darf den bestehenden Status nicht verändern
6. Audit-Logging fehlt. Jede über diesen Admin-/Support-Endpoint ausgelöste Datenbankänderung muss nachvollziehbar protokolliert werden: handelnder Benutzer, Zeitpunkt, betroffene Entität/ID sowie alte und neue Werte. Ein aktualisierter updated-Zeitstempel allein genügt hierfür nicht.
7. Git-/PR-Konventionen sind nicht eingehalten. Der Branch feature/NOTASK-... entspricht nicht den vorgeschriebenen Mustern feat/<scope>-<topic> bzw. fix/<scope>-<topic>. Da beim Squash nur der PR-Titel erhalten bleibt, sollte auch dieser als sauberer, imperativer Commit-Titel formuliert sein.
8. Bitte weist vor dem Merge außerdem die in CONTRIBUTING.md verlangten Prüfungen format, lint und type-check nach.

Ohne nachvollziehbare Begründung, passende Consumer-Änderungen, Tests und revisionssicheres Audit-Logging ist dieser PR nicht mergefähig.