De-fork POS chrome/assets and import CRM products only
Deploy Ladill POS / deploy (push) Successful in 32s
Deploy Ladill POS / deploy (push) Successful in 32s
Finish removing the upstream platform fork: drop the Hosting/SSL/Domain notification classes + their mail views, the email/mailbox/domains/hosting view trees, the Mini/Afia/search/paystack partials and unused components, and the now-orphan support helpers (DomainConfig, MailboxPricing, ResellerClubLegacy), SmsService, and fork configs (afia, hosting, domain, email, mailbox, qr, mail_brands, notifications, ...). Reparent the notifications page off the hosting layout onto the POS app layout (it had referenced undefined hosting./mini. routes). Computed the removable set from a reachability scan of the POS view graph; kept views reference no removed routes/services. Branding: favicon now uses the updated favicon.ico (favicon.svg removed); signed-out fallback logo points at the POS logo. CRM import: constrain the catalogue pull to type=product so CRM services are never imported as POS products (server-side filter + defensive client skip). Test suite green (8 passed, 29 assertions); all routes resolve and every remaining Blade compiles. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 4.8
parent
46b5091b1e
commit
b7c7a32ee6
@@ -64,10 +64,19 @@ class PosCommerceTest extends TestCase
|
||||
[
|
||||
'name' => 'Imported Mug',
|
||||
'sku' => 'MUG-1',
|
||||
'type' => 'product',
|
||||
'unit_price_minor' => 2500,
|
||||
'currency' => 'GHS',
|
||||
'active' => true,
|
||||
],
|
||||
[
|
||||
'name' => 'Consulting Hour',
|
||||
'sku' => 'SVC-1',
|
||||
'type' => 'service',
|
||||
'unit_price_minor' => 9000,
|
||||
'currency' => 'GHS',
|
||||
'active' => true,
|
||||
],
|
||||
],
|
||||
], 200),
|
||||
]);
|
||||
@@ -83,6 +92,13 @@ class PosCommerceTest extends TestCase
|
||||
$this->assertNotNull($product);
|
||||
$this->assertSame('Imported Mug', $product->name);
|
||||
$this->assertSame(2500, $product->price_minor);
|
||||
|
||||
// Services must never be imported as POS products.
|
||||
$this->assertNull(PosProduct::owned($user->public_id)->where('sku', 'SVC-1')->first());
|
||||
|
||||
// The CRM request must constrain to products only.
|
||||
Http::assertSent(fn ($request) => str_contains($request->url(), 'crm.test/api/products')
|
||||
&& str_contains($request->url(), 'type=product'));
|
||||
}
|
||||
|
||||
public function test_paid_sale_links_to_invoice_prefill(): void
|
||||
|
||||
@@ -46,7 +46,7 @@ class PosRegisterTest extends TestCase
|
||||
->get(route('pos.dashboard'))
|
||||
->assertOk()
|
||||
->assertSee('Overview')
|
||||
->assertSee('favicon.svg', false);
|
||||
->assertSee('favicon.ico', false);
|
||||
}
|
||||
|
||||
public function test_register_renders_products(): void
|
||||
|
||||
Reference in New Issue
Block a user