WS-5: Buchungs-Gate launch-ready – proaktiver Dashboard-Hinweis
Das Abo-/Bestandsschutz-Gate (User::hasActiveBooking, BookingRequiredException,
Submit-Modal, API 402/422) war bereits implementiert und getestet. Ergänzt wird
die proaktive Block-UX:
- Kunden-Dashboard zeigt bei scharfem Gate (billing.enforce_booking=true) und
fehlender aktiver Buchung einen Banner "Buchung erforderlich, um zu
veröffentlichen" mit Link zur Buchungsseite. Bestandskunden (grandfathered)
sehen den Hinweis nicht; bei offenem Gate bleibt er unsichtbar.
- Tests: Banner aus (Gate offen) / an (Gate scharf, keine Buchung) /
Ausnahme für grandfathered.
Launch-Flip bleibt env-gesteuert (BILLING_ENFORCE_BOOKING), Dev-Default false –
.env.example und Detailplan (WS-5 ✅) dokumentieren den Schalter.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
parent
afefa86711
commit
a0547208d3
4 changed files with 94 additions and 4 deletions
|
|
@ -72,3 +72,9 @@ VITE_APP_NAME="${APP_NAME}"
|
|||
GOOGLE_CLIENT_ID=
|
||||
GOOGLE_CLIENT_SECRET=
|
||||
GOOGLE_REDIRECT_URI=https://pressekonto.com/auth/google/callback
|
||||
|
||||
# Abo-/Bestandsschutz-Gate (WS-5). Ist der Schalter aus, darf jeder einreichen
|
||||
# (Dev-Default). Für den Launch auf true setzen: Einreichen zur Prüfung erfordert
|
||||
# dann ein aktives Abo, einen bezahlten Einzelkauf oder Bestandsschutz
|
||||
# (grandfathered). Siehe User::hasActiveBooking().
|
||||
BILLING_ENFORCE_BOOKING=false
|
||||
|
|
|
|||
|
|
@ -92,11 +92,16 @@ Alle hinter WS-2 **oder** regulärem Login.
|
|||
- **Tests:** gemeinsame Abrechnung zweier Posten; portalübergreifende Geltung; Worker erzeugt Rechnung zum Periodenende.
|
||||
- **Aufwand:** ~2–3 Tage. Parallel zu WS-1/2 möglich.
|
||||
|
||||
### WS-5 — Abo-/Bestandsschutz-Gate scharf schalten (Merkliste 6)
|
||||
### WS-5 — Abo-/Bestandsschutz-Gate scharf schalten (Merkliste 6) — ✅ erledigt 16.06.2026
|
||||
**Scope:** „User ohne aktives Abo oder Bestandsschutz dürfen keine PM schreiben."
|
||||
- Mechanik existiert (`hasActiveBooking()` inkl. `grandfathered`, `BookingRequiredException`). Nötig: `billing.enforce_booking` für Launch auf `true`, Block-UX prüfen, Bestandsschutz-Fall absichern.
|
||||
- **Tests:** User ohne Buchung → blockiert; grandfathered → erlaubt; aktives Abo → erlaubt.
|
||||
- **Aufwand:** ~½–1 Tag. **Abhängigkeit:** WS-4.
|
||||
- Mechanik existiert (`hasActiveBooking()` inkl. `grandfathered`, `BookingRequiredException`) und ist mit `enforce_booking=true` voll getestet:
|
||||
- **Backend-Gate:** `PressReleaseService::submitForReview()` wirft `BookingRequiredException`/`QuotaExceededException`.
|
||||
- **API:** `POST /api/v1/press-releases/{id}/submit` → `402` (keine Buchung) bzw. `422` (Kontingent erschöpft).
|
||||
- **Block-UX am Einreichpunkt:** `press-release-submit-modal.blade.php` zeigt ohne Buchung statt des Prüf-Flows den Hinweis „Buchung erforderlich" + Button zur Buchungsseite; alle Volt-Caller (show/edit/create/index) fangen die Exceptions und zeigen einen Toast.
|
||||
- **Proaktiver Hinweis:** Kunden-Dashboard zeigt bei scharfem Gate ohne aktive Buchung einen Banner „Buchung erforderlich, um zu veröffentlichen" → `me.bookings.index`. Bestandskunden (grandfathered) sehen ihn nicht.
|
||||
- **Launch-Flip:** Kein Code-Default ändern — der Schalter bleibt in `config/billing.php` auf `false` (Dev-Default). Scharfschalten ausschließlich per Prod-Env `BILLING_ENFORCE_BOOKING=true` (→ WS-7-Checkliste).
|
||||
- **Tests:** `HasActiveBookingTest`, `PressReleaseSubmitApiTest` (402/422), `DashboardTest` (Banner an/aus + grandfathered-Ausnahme), `LegacyBundleTest` (grandfathered zählt portalübergreifend).
|
||||
- **Aufwand:** erledigt. **Abhängigkeit:** WS-4 (erledigt).
|
||||
|
||||
### WS-6 — Login/Registrierung: Flow + E-Mail-Verifizierung + Google-Login (Merkliste 7 + Zweig 3)
|
||||
**Scope nach Entscheidung 2 & 3:**
|
||||
|
|
|
|||
|
|
@ -56,6 +56,10 @@ new #[Layout('components.layouts.app'), Title('Mein Dashboard')] class extends C
|
|||
return [
|
||||
'user' => $user,
|
||||
'selectedCompany' => $selectedCompany,
|
||||
// Launch-Gate (WS-5): Solange billing.enforce_booking aus ist, ist
|
||||
// dies immer false und der Hinweis bleibt unsichtbar. Bei scharfem
|
||||
// Gate sehen Bestandskunden (grandfathered) den Banner nicht.
|
||||
'bookingRequired' => config('billing.enforce_booking') && ! $user->hasActiveBooking(),
|
||||
'stats' => [
|
||||
'total' => (int) ($stats->total ?? 0),
|
||||
'published' => (int) ($stats->published ?? 0),
|
||||
|
|
@ -236,6 +240,33 @@ new #[Layout('components.layouts.app'), Title('Mein Dashboard')] class extends C
|
|||
</div>
|
||||
</header>
|
||||
|
||||
{{-- ============== BUCHUNGS-GATE (WS-5) ============== --}}
|
||||
@if ($bookingRequired)
|
||||
<section class="rounded-[6px] border border-[color:var(--color-warn)]/40 bg-[color:var(--color-warn-soft)] px-5 py-4">
|
||||
<div class="flex flex-col gap-4 sm:flex-row sm:items-center sm:justify-between">
|
||||
<div class="flex items-start gap-3 min-w-0">
|
||||
<span class="flex-shrink-0 mt-0.5 w-9 h-9 rounded-[5px] flex items-center justify-center
|
||||
bg-white/60 border border-[color:var(--color-warn)]/30 text-[color:var(--color-accent-deep)]">
|
||||
<flux:icon.lock-closed class="size-[17px]" />
|
||||
</span>
|
||||
<div class="min-w-0">
|
||||
<div class="text-[13.5px] font-semibold text-[color:var(--color-ink)]">
|
||||
{{ __('Buchung erforderlich, um zu veröffentlichen') }}
|
||||
</div>
|
||||
<p class="text-[12px] leading-[1.55] mt-1 m-0 text-[color:var(--color-ink-2)]">
|
||||
{{ __('Sie können Pressemitteilungen wie gewohnt erstellen und speichern. Zum Einreichen zur Prüfung wird eine aktive Buchung benötigt.') }}
|
||||
</p>
|
||||
</div>
|
||||
</div>
|
||||
<div class="flex-shrink-0">
|
||||
<flux:button variant="primary" icon="credit-card" href="{{ route('me.bookings.index') }}" wire:navigate>
|
||||
{{ __('Buchung auswählen') }}
|
||||
</flux:button>
|
||||
</div>
|
||||
</div>
|
||||
</section>
|
||||
@endif
|
||||
|
||||
{{-- ============== STAT-CARDS — KPI-Reihe ============== --}}
|
||||
<section class="grid grid-cols-2 gap-4 xl:grid-cols-4">
|
||||
<x-portal.stat-card variant="primary" :label="__('Gesamt')" :value="$stats['total']">
|
||||
|
|
|
|||
|
|
@ -1,11 +1,13 @@
|
|||
<?php
|
||||
|
||||
use App\Enums\PressReleaseStatus;
|
||||
use App\Enums\UserPaymentOptionStatus;
|
||||
use App\Models\BillingAddress;
|
||||
use App\Models\Company;
|
||||
use App\Models\PressRelease;
|
||||
use App\Models\Profile;
|
||||
use App\Models\User;
|
||||
use App\Models\UserPaymentOption;
|
||||
use Database\Seeders\RolesAndPermissionsSeeder;
|
||||
use Livewire\Volt\Volt as LivewireVolt;
|
||||
use Tests\TestCase;
|
||||
|
|
@ -123,6 +125,52 @@ test('profile completeness hint with percentage appears for partial profiles', f
|
|||
->assertSee('33');
|
||||
});
|
||||
|
||||
test('the booking-required banner stays hidden while the gate is open', function () {
|
||||
/** @var TestCase $this */
|
||||
config()->set('billing.enforce_booking', false);
|
||||
|
||||
$customer = User::factory()->create(['is_active' => true]);
|
||||
$customer->assignRole('customer');
|
||||
|
||||
$this->actingAs($customer);
|
||||
|
||||
LivewireVolt::test('customer.dashboard')
|
||||
->assertDontSee(__('Buchung erforderlich, um zu veröffentlichen'));
|
||||
});
|
||||
|
||||
test('the booking-required banner appears when the gate is enforced and no booking exists', function () {
|
||||
/** @var TestCase $this */
|
||||
config()->set('billing.enforce_booking', true);
|
||||
|
||||
$customer = User::factory()->create(['is_active' => true]);
|
||||
$customer->assignRole('customer');
|
||||
|
||||
$this->actingAs($customer);
|
||||
|
||||
LivewireVolt::test('customer.dashboard')
|
||||
->assertSee(__('Buchung erforderlich, um zu veröffentlichen'))
|
||||
->assertSee(route('me.bookings.index'), false);
|
||||
});
|
||||
|
||||
test('a grandfathered bestandskunde does not see the booking-required banner', function () {
|
||||
/** @var TestCase $this */
|
||||
config()->set('billing.enforce_booking', true);
|
||||
|
||||
$customer = User::factory()->create(['is_active' => true]);
|
||||
$customer->assignRole('customer');
|
||||
|
||||
UserPaymentOption::factory()->create([
|
||||
'user_id' => $customer->id,
|
||||
'status' => UserPaymentOptionStatus::Grandfathered->value,
|
||||
'stripe_subscription_id' => null,
|
||||
]);
|
||||
|
||||
$this->actingAs($customer);
|
||||
|
||||
LivewireVolt::test('customer.dashboard')
|
||||
->assertDontSee(__('Buchung erforderlich, um zu veröffentlichen'));
|
||||
});
|
||||
|
||||
test('billing address hint disappears once address is complete', function () {
|
||||
/** @var TestCase $this */
|
||||
$customer = User::factory()->create(['is_active' => true]);
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue