[AIDAPP-1595]: Improve the user interface of advisories in the portal

.cleanup-tasks/2026_10_09_advisory_severity_sort_feature.md

@@ -0,0 +1,20 @@
+---
+title: Advisory Severity Sort Feature
+created: 2026-10-09
+---
+
+## Feature Flags
+
+- App\Features\AdvisorySeveritySortFeature
+
+## Temporary Migrations
+
+## Additional Cleanup
+
+<!--
+Only list cleanup that has no home in code. Do NOT restate obvious feature-flag
+removals (keep the active path, drop the inactive path), and do NOT include file
+paths or line numbers. For non-obvious changes, put a `TODO: Cleanup Task (<tag>)`
+comment at the change site and reference the tag here — e.g.
+"Search for `TODO: Cleanup Task (some-feature)`".
+-->

app-modules/knowledge-base/src/Filament/Resources/KnowledgeBaseItems/Pages/EditKnowledgeBaseItem.php

@@ -70,16 +70,7 @@ class EditKnowledgeBaseItem extends EditRecord
     protected static string $resource = KnowledgeBaseItemResource::class;
 
     #[Url]
-    public ?string $tab = null;
-
-    public function mount(int | string $record): void
-    {
-        parent::mount($record);
-
-        if (! in_array($this->tab, ['resource', 'properties'])) {
-            $this->tab = 'resource';
-        }
-    }
+    public ?string $tab = 'resource';
 
     public function form(Schema $schema): Schema
     {

app-modules/knowledge-base/src/Filament/Resources/KnowledgeBaseItems/Pages/ViewKnowledgeBaseItem.php

@@ -59,16 +59,7 @@ class ViewKnowledgeBaseItem extends ViewRecord
     protected static string $resource = KnowledgeBaseItemResource::class;
 
     #[Url]
-    public ?string $tab = null;
-
-    public function mount(int | string $record): void
-    {
-        parent::mount($record);
-
-        if (! in_array($this->tab, ['resource', 'properties', 'concerns', 'health'])) {
-            $this->tab = 'resource';
-        }
-    }
+    public ?string $tab = 'resource';
 
     public function getTitle(): string|Htmlable
     {

app-modules/portal/routes/api.php

@@ -56,6 +56,7 @@
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\LicenseManagementPortalController;
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\ServiceMonitorStatusController;
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\ServiceRequestTypesController;
+use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\ShowAdvisoryController;
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\ShowPortalProjectController;
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\ShowServiceMonitorStatusController;
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\StoreKnowledgeBaseArticleVoteController;
@@ -64,6 +65,7 @@
 use AidingApp\Portal\Http\Controllers\KnowledgeManagementPortal\StoreServiceRequestUpdateController;
 use AidingApp\Portal\Http\Middleware\EnsureKnowledgeManagementPortalIsEmbeddableAndAuthorized;
 use AidingApp\Portal\Http\Middleware\EnsureKnowledgeManagementPortalIsEnabled;
+use AidingApp\Portal\Http\Middleware\EnsureKnowledgeManagementPortalServiceManagementIsEnabled;
 use AidingApp\ServiceManagement\Http\Controllers\ServiceRequestMediaDownloadController;
 use App\Http\Middleware\TrackContactPresence;
 use Illuminate\Support\Facades\Route;
@@ -124,43 +126,48 @@
                 Route::get('/articles/{article}', [KnowledgeManagementPortalArticleController::class, 'show'])
                     ->name('article.show-canonical');
 
-                Route::get('/service-request-type/select', [ServiceRequestTypesController::class, 'index'])
-                    ->name('service-request-type.index');
+                Route::get('/tags', GetKnowledgeManagementPortalTagsController::class)
+                    ->name('tags.index');
 
-                Route::get('/service-requests', [GetServiceRequestsController::class, 'index'])
-                    ->name('service-request.index');
+                Route::middleware(EnsureKnowledgeManagementPortalServiceManagementIsEnabled::class)->group(function (): void {
+                    Route::get('/service-request-type/select', [ServiceRequestTypesController::class, 'index'])
+                        ->name('service-request-type.index');
 
-                Route::get('/service-request/create/{type}', GetServiceRequestFormController::class)
-                    ->name('service-request.create');
+                    Route::get('/service-requests', [GetServiceRequestsController::class, 'index'])
+                        ->name('service-request.index');
 
-                Route::post('/service-request/create/{type}', StoreServiceRequestController::class)
-                    ->middleware(['auth:sanctum'])
-                    ->name('service-request.store');
+                    Route::get('/service-request/create/{type}', GetServiceRequestFormController::class)
+                        ->name('service-request.create');
 
-                Route::post('/service-request/create/{type}/generate-questions', GenerateServiceRequestQuestionsController::class)
-                    ->middleware(['auth:sanctum'])
-                    ->name('service-request.generate-questions');
+                    Route::post('/service-request/create/{type}', StoreServiceRequestController::class)
+                        ->middleware(['auth:sanctum'])
+                        ->name('service-request.store');
 
-                Route::post('/service-request/create/{type}/evaluate-ai-resolution', EvaluateServiceRequestAiResolutionController::class)
-                    ->middleware(['auth:sanctum'])
-                    ->name('service-request.evaluate-ai-resolution');
+                    Route::post('/service-request/create/{type}/generate-questions', GenerateServiceRequestQuestionsController::class)
+                        ->middleware(['auth:sanctum'])
+                        ->name('service-request.generate-questions');
 
-                Route::get('/service-request/request-upload-url', GetServiceRequestUploadUrl::class)
-                    ->middleware(['auth:sanctum'])
-                    ->name('service-request.request-upload-url');
+                    Route::post('/service-request/create/{type}/evaluate-ai-resolution', EvaluateServiceRequestAiResolutionController::class)
+                        ->middleware(['auth:sanctum'])
+                        ->name('service-request.evaluate-ai-resolution');
 
-                Route::post('/service-request/store-secret', StoreServiceRequestSecretController::class)
-                    ->middleware(['auth:sanctum', 'throttle:service-request-secrets'])
-                    ->name('service-request.store-secret');
+                    Route::get('/service-request/request-upload-url', GetServiceRequestUploadUrl::class)
+                        ->middleware(['auth:sanctum'])
+                        ->name('service-request.request-upload-url');
+
+                    Route::post('/service-request/store-secret', StoreServiceRequestSecretController::class)
+                        ->middleware(['auth:sanctum', 'throttle:service-request-secrets'])
+                        ->name('service-request.store-secret');
+
+                    Route::get('/service-request/{serviceRequest}', [GetServiceRequestsController::class, 'show'])
+                        ->middleware(['auth:sanctum'])
+                        ->name('service-request.show');
+                    Route::post('/service-request-update/store', StoreServiceRequestUpdateController::class)
+                        ->name('service-request-update.storeServiceRequestUpdate');
+                    Route::get('/service-request/media/{media}/download', ServiceRequestMediaDownloadController::class)
+                        ->name('service-request.media.download');
+                });
 
-                Route::get('/tags', GetKnowledgeManagementPortalTagsController::class)
-                    ->name('tags.index');
-                Route::get('/service-request/{serviceRequest}', [GetServiceRequestsController::class, 'show'])
-                    ->name('service-request.show');
-                Route::post('/service-request-update/store', StoreServiceRequestUpdateController::class)
-                    ->name('service-request-update.storeServiceRequestUpdate');
-                Route::get('/service-request/media/{media}/download', ServiceRequestMediaDownloadController::class)
-                    ->name('service-request.media.download');
                 Route::get('/knowledge-base-article/media/{media}/download', KnowledgeBaseArticleMediaDownloadController::class)
                     ->name('knowledge-base-article.media.download');
                 Route::post('/knowledge_base_article_vote/store', StoreKnowledgeBaseArticleVoteController::class)
@@ -173,6 +180,10 @@
                 Route::get('/advisories', AdvisoryController::class)
                     ->middleware(['auth:sanctum'])
                     ->name('advisories');
+                Route::get('/advisories/{advisory}', ShowAdvisoryController::class)
+                    ->middleware(['auth:sanctum'])
+                    ->whereUuid('advisory')
+                    ->name('advisories.show');
                 Route::get('/status', ServiceMonitorStatusController::class)
                     ->middleware(['auth:sanctum'])
                     ->name('status');

app-modules/portal/routes/web.php

@@ -100,6 +100,8 @@
                 ->name('service');
             Route::get('/advisories', RenderKnowledgeManagementPortal::class)
                 ->name('advisories');
+            Route::get('/advisories/{advisory}', RenderKnowledgeManagementPortal::class)
+                ->name('advisories.show');
             Route::get('/assets', RenderKnowledgeManagementPortal::class)
                 ->name('assets');
             Route::get('/licenses', RenderKnowledgeManagementPortal::class)

portals/knowledge-management/src/Components/ui/BaseTableEmptyState.vue → app-modules/portal/src/DataTransferObjects/AdvisoryData.php

@@ -1,4 +1,6 @@
-<!--
+<?php
+
+/*
 <COPYRIGHT>
 
     Copyright © 2016-2026, Canyon GBS Inc. All rights reserved.
@@ -30,32 +32,39 @@
     <https://www.canyongbs.com> or contact us via email at legal@canyongbs.com.
 
 </COPYRIGHT>
--->
-<template>
-    <div
-        role="status"
-        class="flex flex-col items-center justify-center gap-4 py-20 rounded-[var(--rounding-lg)] border border-dashed border-gray-200 bg-white text-center"
-    >
-        <slot name="icon">
-            <span class="flex h-16 w-16 items-center justify-center rounded-2xl bg-gray-100">
-                <svg
-                    class="h-8 w-8 text-gray-300"
-                    fill="none"
-                    viewBox="0 0 24 24"
-                    stroke="currentColor"
-                    stroke-width="1.5"
-                    aria-hidden="true"
-                >
-                    <path
-                        stroke-linecap="round"
-                        stroke-linejoin="round"
-                        d="M20.25 7.5l-.625 10.632a2.25 2.25 0 01-2.247 2.118H6.622a2.25 2.25 0 01-2.247-2.118L3.75 7.5M10 11.25h4M3.375 7.5h17.25c.621 0 1.125-.504 1.125-1.125v-1.5c0-.621-.504-1.125-1.125-1.125H3.375c-.621 0-1.125.504-1.125 1.125v1.5c0 .621.504 1.125 1.125 1.125z"
-                    />
-                </svg>
-            </span>
-        </slot>
-        <div>
-            <slot />
-        </div>
-    </div>
-</template>
+*/
+
+namespace AidingApp\Portal\DataTransferObjects;
+
+use AidingApp\ServiceManagement\Models\Advisory;
+use Spatie\LaravelData\Attributes\MapOutputName;
+use Spatie\LaravelData\Data;
+use Spatie\LaravelData\Mappers\SnakeCaseMapper;
+
+#[MapOutputName(SnakeCaseMapper::class)]
+class AdvisoryData extends Data
+{
+    public function __construct(
+        public string $id,
+        public string $title,
+        public string $description,
+        public ?AdvisorySeverityData $severity,
+        public ?AdvisoryStatusData $status,
+        public string $lastUpdatedAt,
+    ) {}
+
+    /**
+     * Requires the `WithLastUpdatedAt` scope to have been applied to the query.
+     */
+    public static function fromModel(Advisory $advisory): self
+    {
+        return new self(
+            id: $advisory->getKey(),
+            title: $advisory->title,
+            description: $advisory->description,
+            severity: $advisory->severity ? AdvisorySeverityData::fromModel($advisory->severity) : null,
+            status: $advisory->status ? AdvisoryStatusData::fromModel($advisory->status) : null,
+            lastUpdatedAt: $advisory->getLastUpdatedAt()->toIso8601String(),
+        );
+    }
+}

portals/knowledge-management/src/Components/ui/BaseStatusPill.vue → app-modules/portal/src/DataTransferObjects/AdvisoryDetailData.php

@@ -1,4 +1,6 @@
-<!--
+<?php
+
+/*
 <COPYRIGHT>
 
     Copyright © 2016-2026, Canyon GBS Inc. All rights reserved.
@@ -30,56 +32,47 @@
     <https://www.canyongbs.com> or contact us via email at legal@canyongbs.com.
 
 </COPYRIGHT>
--->
-<script setup>
-    import { computed } from 'vue';
-
-    const props = defineProps({
-        tone: {
-            type: String,
-            default: 'neutral',
-            validator: (v) => ['success', 'warning', 'danger', 'neutral'].includes(v),
-        },
-        pulse: {
-            type: Boolean,
-            default: false,
-        },
-    });
-
-    const dotClasses = computed(() => {
-        const classes = {
-            success: 'bg-green-500',
-            warning: 'bg-orange-500',
-            danger: 'bg-red-500',
-            neutral: 'bg-gray-500',
-        };
+*/
 
-        return classes[props.tone] ?? classes.neutral;
-    });
+namespace AidingApp\Portal\DataTransferObjects;
 
-    const toneClasses = computed(() => {
-        const classes = {
-            success: 'bg-green-100 text-green-800 ring-green-600/20',
-            warning: 'bg-orange-100 text-orange-800 ring-orange-600/20',
-            danger: 'bg-red-100 text-red-800 ring-red-600/20',
-            neutral: 'bg-gray-100 text-gray-700 ring-gray-500/20',
-        };
+use AidingApp\ServiceManagement\Models\Advisory;
+use AidingApp\ServiceManagement\Models\AdvisoryUpdate;
+use Spatie\LaravelData\Attributes\MapOutputName;
+use Spatie\LaravelData\Data;
+use Spatie\LaravelData\Mappers\SnakeCaseMapper;
 
-        return classes[props.tone] ?? classes.neutral;
-    });
-</script>
+#[MapOutputName(SnakeCaseMapper::class)]
+class AdvisoryDetailData extends Data
+{
+    /**
+     * @param list<AdvisoryUpdateData> $updates The advisory's updates, newest first.
+     */
+    public function __construct(
+        public string $id,
+        public string $title,
+        public string $description,
+        public ?AdvisorySeverityData $severity,
+        public ?AdvisoryStatusData $status,
+        public ?string $departmentName,
+        public array $updates,
+    ) {}
 
-<template>
-    <span
-        :class="[
-            'inline-flex items-center gap-1.5 px-2.5 py-1 text-xs font-semibold ring-1 ring-inset rounded-[var(--rounding-md)]',
-            toneClasses,
-        ]"
-    >
-        <span
-            :class="['h-1.5 w-1.5 shrink-0 rounded-full', dotClasses, pulse ? 'animate-pulse' : '']"
-            aria-hidden="true"
-        />
-        <slot />
-    </span>
-</template>
+    /**
+     * Expects the advisory's updates to have been loaded newest first.
+     */
+    public static function fromModel(Advisory $advisory): self
+    {
+        return new self(
+            id: $advisory->getKey(),
+            title: $advisory->title,
+            description: $advisory->description,
+            severity: $advisory->severity ? AdvisorySeverityData::fromModel($advisory->severity) : null,
+            status: $advisory->status ? AdvisoryStatusData::fromModel($advisory->status) : null,
+            departmentName: $advisory->assignedDepartment?->name,
+            updates: $advisory->advisoryUpdates
+                ->map(fn (AdvisoryUpdate $advisoryUpdate): AdvisoryUpdateData => AdvisoryUpdateData::fromModel($advisoryUpdate))
+                ->all(),
+        );
+    }
+}