Remove manual open graph asset

README.md

@@ -34,7 +34,6 @@ To start a PHP× group:
     "region": "<<short city or region name>>", // can be null if airport code is good enough
     "description": "<<short description>>",
     "timezone": "<<php-compatible timezone ID>>",
-    "og_asset": "<<include image in public/og/ in pr>>", // can be null
     "bsky_url": "https://bsky.app/profile/<<group profile>>" // can be null
 },
 ```

app/Actions/ConfigureGroup.php

@@ -24,7 +24,6 @@ public function handle(
 		string $timezone = 'America/New_York', // One true timezone
 		?string $bsky_url = null,
 		?string $meetup_url = null,
-		?string $og_asset = null,
 	): Group {
 		$group = Group::updateOrCreate([
 			'domain' => $domain,
@@ -35,7 +34,6 @@ public function handle(
 			'timezone' => $timezone,
 			'bsky_url' => $bsky_url ?: null,
 			'meetup_url' => $meetup_url ?: null,
-			'og_asset' => $og_asset ?: null,
 		]);
 
 		Cache::clear();
@@ -73,15 +71,13 @@ public function asCommand(Command $command): int
 		$timezone = suggest('Timezone', DateTimeZone::listIdentifiers(), default: str($existing->timezone), required: true);
 		$bsky_url = text('Is there a Bluesky URL?', default: str($existing->bsky_url));
 		$meetup_url = text('Is there a Meetup URL?', default: str($existing->meetup_url));
-		$og_asset = text('What is the open graph image name?', default: str($existing->og_asset));
 
 		table(['Option', 'Value'], [
 			['Name', $name],
 			['Description', $description],
 			['Timezone', $timezone],
 			['Bluesky', $bsky_url],
 			['Meetup', $meetup_url],
-			['Open Graph Image', $og_asset],
 		]);
 
 		if (confirm('Is this correct?')) {
@@ -93,7 +89,6 @@ public function asCommand(Command $command): int
 				timezone: $timezone,
 				bsky_url: $bsky_url,
 				meetup_url: $meetup_url,
-				og_asset: $og_asset,
 			);
 
 			$command->info($group->wasRecentlyCreated

app/Actions/SyncGroups.php

@@ -72,7 +72,6 @@ protected function syncConfigWithGroup(string $domain, array $config): Group
 			'timezone',
 			'bsky_url',
 			'meetup_url',
-			'og_asset',
 		]));
 
 		return $group;

app/Models/Group.php

@@ -4,6 +4,7 @@
 
 use Glhd\Bits\Database\HasSnowflakes;
 use Illuminate\Container\Container;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
@@ -12,6 +13,7 @@
 use Illuminate\Routing\UrlGenerator;
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Cache;
+use Illuminate\Support\Stringable;
 use Revolution\Bluesky\Contracts\Factory;
 use Revolution\Bluesky\Facades\Bluesky;
 use Spatie\MailcoachSdk\Mailcoach;
@@ -118,5 +120,26 @@ public function mailcoach_transactional_emails(): HasMany
 	{
 		return $this->hasMany(MailcoachTransactionalEmail::class);
 	}
+
+	protected function airportCode(): Attribute
+	{
+		return Attribute::get(
+			fn(): Stringable => str($this->name)->afterLast('×')->trim()->upper(),
+		);
+	}
+
+	protected function openGraphImageUrl(): Attribute
+	{
+		return Attribute::get(function() {
+			$filename = $this->airport_code->lower()->finish('.png');
+			$path = public_path("og/{$filename}");
+
+			if (file_exists($path)) {
+				return asset("og/{$filename}").'?t='.filemtime($path);
+			}
+
+			return null;
+		});
+	}
 }
 

database/migrations/2024_11_15_124406_remove_og_asset.php

@@ -0,0 +1,21 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration {
+	public function up(): void
+	{
+		Schema::table('groups', function(Blueprint $table) {
+			$table->dropColumn('og_asset');
+		});
+	}
+
+	public function down(): void
+	{
+		Schema::table('groups', function(Blueprint $table) {
+			$table->string('og_asset')->nullable();
+		});
+	}
+};

database/seeders/GroupSeeder.php

@@ -16,7 +16,6 @@ public function run(): void
 			'twitter_url' => 'https://twitter.com/joetannenbaum',
 			'meetup_url' => 'https://www.meetup.com/php-nyc/',
 			'description' => 'A fresh PHP meetup for NYC area devs.',
-			'og_asset' => 'nyc.png',
 		]);
 
 		app()->instance('group:phpxnyc.com', $nyc);
@@ -28,7 +27,6 @@ public function run(): void
 			'twitter_url' => 'https://twitter.com/inxilpro',
 			'meetup_url' => 'https://www.meetup.com/php-philly/',
 			'description' => 'A Philly-area PHP meetup for web artisans who want to learn and connect.',
-			'og_asset' => 'philly.png',
 		]);
 
 		app()->instance('group:phpxphilly.com', $philly);
@@ -40,7 +38,6 @@ public function run(): void
 			'twitter_url' => 'https://twitter.com/skylerkatz',
 			'meetup_url' => 'https://www.meetup.com/php-stl/',
 			'description' => 'A St. Louis-area PHP meetup for web artisans who want to learn and connect.',
-			'og_asset' => 'stl.png',
 		]);
 
 		app()->instance('group:phpxstl.com', $stl);

groups.json

@@ -5,7 +5,6 @@
 		"region": null,
 		"description": "A Philly-area PHP meetup for web artisans who want to learn and connect.",
 		"timezone": "America/New_York",
-		"og_asset": "philly.png",
 		"bsky_url": "https://bsky.app/profile/phpxphilly.com"
 	},
 	"upstatephp.com": {

resources/markdown/organizers.md

@@ -16,7 +16,6 @@ To start a PHP× group:
     "region": "<<short city or region name>>", // can be null if airport code is good enough
     "description": "<<short description>>",
     "timezone": "<<php-compatible timezone ID>>",
-    "og_asset": "<<include image in public/og/ in pr>>", // can be null
     "bsky_url": "https://bsky.app/profile/<<group profile>>" // can be null
 },
 ```

resources/views/components/layout.blade.php

@@ -31,8 +31,8 @@
 				<meta name="description" content="{{ $group->description }}" />
 				<meta property="og:description" content="{{ $group->description }}" />
 			@endif
-			@if($group->og_asset)
-				<meta property="og:image" content="{{ asset("og/{$group->og_asset}") }}" />
+			@if($group->open_graph_image_url)
+				<meta property="og:image" content="{{ $group->open_graph_image_url }}" />
 			@endif
 		@endif
 	@endisset

tests/Feature/GroupsJsonTest.php

@@ -30,10 +30,6 @@ protected function assertValidGroup(string $domain, array $config): void
 		Assert::assertNotEmpty(data_get($config, 'description'));
 		Assert::assertContains(data_get($config, 'timezone'), \DateTimeZone::listIdentifiers());
 
-		if ($og_asset = data_get($config, 'og_asset')) {
-			Assert::assertFileExists(public_path("og/{$og_asset}"));
-		}
-
 		if ($bsky_url = data_get($config, 'bsky_url')) {
 			Assert::assertEquals(200, Http::get($bsky_url)->status());
 		}