PM-Vorschau: echtes Portal-Design, Direkt-Button und Portal-Switch
Der Vorschau-Link (/pm-vorschau/{token}) renderte eine schlichte
Standalone-Seite, die dem spaeteren Erscheinungsbild nicht entsprach.
Jetzt rendert er die echte Detailseite (web.release-detail) im Theme
des Zielportals:
- Controller wendet Domain-/Theme-Konfig des Zielportals an (Sprache
der PM, Links zeigen auf die Portal-Domain), kein Hit-Zaehler
- Gelbes Vorschau-Banner mit Status und Link-Gueltigkeit; noindex per
Meta-Tag + X-Robots-Tag-Header, Canonical entfaellt bei noindex
(neue robots-Section im Web-Layout)
- Teilen-Buttons teilen den Vorschau-Link selbst (oeffentliche URL
existiert vor Veroeffentlichung noch nicht)
- PMs fuer beide Portale: ?portal=... waehlt die Ansicht, Umschalter
im Banner; im Kundenpanel je Portal ein Vorschau-Button, sonst ein
einzelner "Vorschau oeffnen"-Button (Link-Feld jetzt kopierbar)
- Alte press-release-preview-View entfernt
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
parent
9a95a6937d
commit
090a893cc6
8 changed files with 306 additions and 149 deletions
|
|
@ -1,11 +1,29 @@
|
|||
<?php
|
||||
|
||||
use App\Enums\Portal;
|
||||
use App\Enums\PressReleaseStatus;
|
||||
use App\Models\PressRelease;
|
||||
use App\Models\User;
|
||||
use App\Services\Auth\MagicLinkGenerator;
|
||||
use Database\Seeders\RolesAndPermissionsSeeder;
|
||||
use Livewire\Volt\Volt as LivewireVolt;
|
||||
use Tests\TestCase;
|
||||
|
||||
function makePreviewCustomerWithPressRelease(Portal $portal): array
|
||||
{
|
||||
test()->seed(RolesAndPermissionsSeeder::class);
|
||||
|
||||
$customer = User::factory()->create(['is_active' => true]);
|
||||
$customer->assignRole('customer');
|
||||
|
||||
$pr = PressRelease::factory()->forPortal($portal)->create([
|
||||
'user_id' => $customer->id,
|
||||
'status' => PressReleaseStatus::Draft->value,
|
||||
]);
|
||||
|
||||
return [$customer, $pr];
|
||||
}
|
||||
|
||||
test('preview link returns press release for valid unexpired token', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
|
|
@ -24,6 +42,97 @@ test('preview link returns press release for valid unexpired token', function ()
|
|||
->assertSee('Vertraulicher Inhalt');
|
||||
});
|
||||
|
||||
test('preview renders the public release detail layout with preview banner and noindex', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Presseecho)->create([
|
||||
'user_id' => $owner->id,
|
||||
'status' => PressReleaseStatus::Draft->value,
|
||||
'title' => 'Entwurf im Portal-Design',
|
||||
]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token']]))
|
||||
->assertOk()
|
||||
->assertViewIs('web.release-detail')
|
||||
->assertHeader('X-Robots-Tag', 'noindex, nofollow')
|
||||
->assertSee('<meta name="robots" content="noindex,nofollow">', false)
|
||||
->assertDontSee('rel="canonical"', false)
|
||||
->assertSeeText('Vorschau')
|
||||
->assertSeeText('So erscheint die Pressemitteilung nach der Veröffentlichung.')
|
||||
->assertSeeText('Status: Entwurf')
|
||||
->assertSeeText('Entwurf im Portal-Design');
|
||||
});
|
||||
|
||||
test('preview applies the theme of the target portal', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Businessportal24)->create([
|
||||
'user_id' => $owner->id,
|
||||
'status' => PressReleaseStatus::Draft->value,
|
||||
]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token']]));
|
||||
|
||||
expect(config('app.theme'))->toBe('businessportal24');
|
||||
});
|
||||
|
||||
test('preview of a single portal release shows no portal switch', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Presseecho)->create(['user_id' => $owner->id]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token']]))
|
||||
->assertOk()
|
||||
->assertDontSee('?portal=businessportal24', false);
|
||||
});
|
||||
|
||||
test('preview of a both portals release defaults to presseecho and shows the portal switch', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Both)->create(['user_id' => $owner->id]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token']]))
|
||||
->assertOk()
|
||||
->assertSee('?portal=presseecho', false)
|
||||
->assertSee('?portal=businessportal24', false);
|
||||
|
||||
expect(config('app.theme'))->toBe('presseecho');
|
||||
});
|
||||
|
||||
test('preview of a both portals release can be switched to businessportal24', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Both)->create(['user_id' => $owner->id]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token'], 'portal' => 'businessportal24']))
|
||||
->assertOk();
|
||||
|
||||
expect(config('app.theme'))->toBe('businessportal24');
|
||||
});
|
||||
|
||||
test('portal query parameter is ignored for single portal releases', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
$pr = PressRelease::factory()->forPortal(Portal::Presseecho)->create(['user_id' => $owner->id]);
|
||||
|
||||
$share = app(MagicLinkGenerator::class)->createPressReleaseShareLink($pr, $owner);
|
||||
|
||||
$this->get(route('press-releases.preview', ['token' => $share['plain_token'], 'portal' => 'businessportal24']))
|
||||
->assertOk();
|
||||
|
||||
expect(config('app.theme'))->toBe('presseecho');
|
||||
});
|
||||
|
||||
test('preview link allows access to draft press release without authentication', function () {
|
||||
/** @var TestCase $this */
|
||||
$owner = User::factory()->create();
|
||||
|
|
@ -72,3 +181,28 @@ test('preview link for soft deleted press release returns 404', function () {
|
|||
$this->get(route('press-releases.preview', ['token' => $share['plain_token']]))
|
||||
->assertStatus(404);
|
||||
});
|
||||
|
||||
test('share panel offers a direct open button for single portal releases', function () {
|
||||
/** @var TestCase $this */
|
||||
[$customer, $pr] = makePreviewCustomerWithPressRelease(Portal::Presseecho);
|
||||
$this->actingAs($customer);
|
||||
|
||||
LivewireVolt::test('customer.press-releases.show', ['id' => $pr->id])
|
||||
->call('generateShareLink')
|
||||
->assertSee('Vorschau öffnen')
|
||||
->assertDontSee('Vorschau presseecho')
|
||||
->assertDontSee('?portal=', false);
|
||||
});
|
||||
|
||||
test('share panel offers one open button per portal for both portal releases', function () {
|
||||
/** @var TestCase $this */
|
||||
[$customer, $pr] = makePreviewCustomerWithPressRelease(Portal::Both);
|
||||
$this->actingAs($customer);
|
||||
|
||||
LivewireVolt::test('customer.press-releases.show', ['id' => $pr->id])
|
||||
->call('generateShareLink')
|
||||
->assertSee('Vorschau presseecho')
|
||||
->assertSee('Vorschau businessportal24')
|
||||
->assertSee('?portal=presseecho', false)
|
||||
->assertSee('?portal=businessportal24', false);
|
||||
});
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue