Feat: Adding Contact options to Store Details
This commit is contained in:
+57
@@ -0,0 +1,57 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
use Illuminate\Database\Migrations\Migration;
|
||||||
|
use Illuminate\Database\Schema\Blueprint;
|
||||||
|
use Illuminate\Support\Facades\DB;
|
||||||
|
use Illuminate\Support\Facades\Schema;
|
||||||
|
use Lunar\Models\Language;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* contact_email/mail_from_name join tax_identifier/registration_number/
|
||||||
|
* phone as plain, non-locale-dependent strings on the store_details
|
||||||
|
* singleton (see that table's own migration docblock).
|
||||||
|
*
|
||||||
|
* legal_name is locale-keyed JSON instead — same shape/resolution as
|
||||||
|
* name/address/bank_transfer_instructions (HasTranslations, see
|
||||||
|
* StoreDetails's own docblock) — since a registered company name can
|
||||||
|
* legitimately differ per locale (e.g. a transliterated/translated legal
|
||||||
|
* form). Distinct from the storefront-facing brand name in `name`.
|
||||||
|
*
|
||||||
|
* Backfills every translatable column (name/address/
|
||||||
|
* bank_transfer_instructions/legal_name) with an empty per-locale array
|
||||||
|
* on any row that's still genuinely NULL there — StoreDetailsService::
|
||||||
|
* firstOrCreate() only seeds columns on INSERT, so an existing singleton
|
||||||
|
* row (or one created before a Language row existed, leaving
|
||||||
|
* emptyPerLocale() an empty array at the time) could otherwise keep a
|
||||||
|
* translatable column NULL forever. That's the exact condition
|
||||||
|
* TranslatedText breaks on (see StoreDetailsService's own docblock: a
|
||||||
|
* NULL-starting translatable field silently drops every keystroke and
|
||||||
|
* never persists) — backfilled here for all four columns, not just the
|
||||||
|
* new one, so the same fix covers however the existing row got there.
|
||||||
|
*/
|
||||||
|
return new class extends Migration
|
||||||
|
{
|
||||||
|
public function up(): void
|
||||||
|
{
|
||||||
|
Schema::table('store_details', function (Blueprint $table) {
|
||||||
|
$table->string('contact_email')->nullable()->after('phone');
|
||||||
|
$table->string('mail_from_name')->nullable()->after('contact_email');
|
||||||
|
$table->json('legal_name')->nullable()->after('registration_number');
|
||||||
|
});
|
||||||
|
|
||||||
|
$emptyPerLocale = json_encode(
|
||||||
|
Language::query()->pluck('code')->mapWithKeys(fn (string $code) => [$code => ''])->all()
|
||||||
|
);
|
||||||
|
|
||||||
|
foreach (['name', 'address', 'bank_transfer_instructions', 'legal_name'] as $column) {
|
||||||
|
DB::table('store_details')->whereNull($column)->update([$column => $emptyPerLocale]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public function down(): void
|
||||||
|
{
|
||||||
|
Schema::table('store_details', function (Blueprint $table) {
|
||||||
|
$table->dropColumn(['contact_email', 'mail_from_name', 'legal_name']);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
@@ -80,10 +80,19 @@ class ManageStoreDetails extends Page implements HasForms
|
|||||||
TextInput::make('phone')
|
TextInput::make('phone')
|
||||||
->label('Phone')
|
->label('Phone')
|
||||||
->tel(),
|
->tel(),
|
||||||
|
TextInput::make('contact_email')
|
||||||
|
->label('Contact email')
|
||||||
|
->email()
|
||||||
|
->helperText('Receives the contact form, and is shown to customers as the store\'s contact email.'),
|
||||||
|
TextInput::make('mail_from_name')
|
||||||
|
->label('Mail from name')
|
||||||
|
->helperText('The sender name on outgoing emails (MAIL_FROM_NAME).'),
|
||||||
]),
|
]),
|
||||||
Section::make('Legal')
|
Section::make('Legal')
|
||||||
->description('Shown on invoices and terms pages.')
|
->description('Shown on invoices and terms pages.')
|
||||||
->schema([
|
->schema([
|
||||||
|
TranslatedText::make('legal_name')
|
||||||
|
->label('Legal name'),
|
||||||
TextInput::make('tax_identifier')
|
TextInput::make('tax_identifier')
|
||||||
->label('Tax ID (ΑΦΜ)'),
|
->label('Tax ID (ΑΦΜ)'),
|
||||||
TextInput::make('registration_number')
|
TextInput::make('registration_number')
|
||||||
|
|||||||
@@ -21,5 +21,6 @@ class StoreDetails extends Model
|
|||||||
'name' => 'array',
|
'name' => 'array',
|
||||||
'address' => 'array',
|
'address' => 'array',
|
||||||
'bank_transfer_instructions' => 'array',
|
'bank_transfer_instructions' => 'array',
|
||||||
|
'legal_name' => 'array',
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -67,6 +67,7 @@ class StoreDetailsService
|
|||||||
'name' => $this->emptyPerLocale(),
|
'name' => $this->emptyPerLocale(),
|
||||||
'address' => $this->emptyPerLocale(),
|
'address' => $this->emptyPerLocale(),
|
||||||
'bank_transfer_instructions' => $this->emptyPerLocale(),
|
'bank_transfer_instructions' => $this->emptyPerLocale(),
|
||||||
|
'legal_name' => $this->emptyPerLocale(),
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user