Adapt documents component to ownership docs

- Update responsible parties to meet interface reqs for upload dialog/document list
   - Add certificate of title to properties
   - Add corporate summary to responsible parties
- Update C&E docs to accept property and responsible party
- Add new section for ownership docs to draft
- Integrate updated docs component into new section

Author: trslater

alcs-frontend/src/app/features/compliance-and-enforcement/documents/documents.component.ts

@@ -5,10 +5,7 @@ import { MatSort } from '@angular/material/sort';
 import { MatTableDataSource } from '@angular/material/table';
 import { FILE_NAME_TRUNCATE_LENGTH } from '../../../shared/constants';
 import { DOCUMENT_SYSTEM } from '../../../shared/document/document.dto';
-import {
-  DocumentUploadDialogData,
-  DocumentUploadDialogOptions,
-} from '../../../shared/document-upload-dialog/document-upload-dialog.interface';
+import { DocumentUploadDialogData } from '../../../shared/document-upload-dialog/document-upload-dialog.interface';
 import { DocumentUploadDialogComponent } from '../../../shared/document-upload-dialog/document-upload-dialog.component';
 import { MatDialog } from '@angular/material/dialog';
 import {
@@ -31,7 +28,7 @@ export class ComplianceAndEnforcementDocumentsComponent implements OnInit, OnDes
 
   @Input() title?: string;
   @Input() fileNumber?: string;
-  @Input() options?: DocumentUploadDialogOptions;
+  @Input() options?: DocumentUploadDialogData;
   @Input() section?: Section;
 
   displayedColumns: string[] = ['source', 'type', 'fileName', 'uploadedAt', 'actions'];

alcs-frontend/src/app/features/compliance-and-enforcement/draft/draft.component.html

@@ -25,11 +25,18 @@ <h2>C&E File ID: {{ file?.fileNumber }}</h2>
     </section>
 
     <section class="form-section">
-      <app-compliance-and-enforcement-responsible-parties 
+      <app-compliance-and-enforcement-responsible-parties
         #responsiblePartiesComponent
         [fileUuid]="file?.uuid"
         [isPropertyCrown]="isPropertyCrown"
       />
+      <app-compliance-and-enforcement-documents
+        #ownershipDocumentsComponent
+        [title]="'Property Ownership Documents'"
+        [fileNumber]="fileNumber"
+        [options]="ownershipDocumentOptions"
+        [section]="Section.OWNERSHIP"
+      />
     </section>
 
     <div class="button-container">

alcs-frontend/src/app/features/compliance-and-enforcement/draft/draft.component.ts

@@ -1,6 +1,17 @@
 import { AfterViewInit, Component, OnDestroy, OnInit, ViewChild } from '@angular/core';
 import { ActivatedRoute } from '@angular/router';
-import { catchError, debounceTime, EMPTY, filter, firstValueFrom, skip, Subject, switchMap, takeUntil, tap } from 'rxjs';
+import {
+  catchError,
+  debounceTime,
+  EMPTY,
+  filter,
+  firstValueFrom,
+  skip,
+  Subject,
+  switchMap,
+  takeUntil,
+  tap,
+} from 'rxjs';
 import { C_E_AUTOSAVE_DEBOUNCE_MS } from '../constants';
 import {
   ComplianceAndEnforcementDto,
@@ -17,9 +28,10 @@ import { PropertyComponent, cleanPropertyUpdate } from '../property/property.com
 import { ComplianceAndEnforcementPropertyDto } from '../../../services/compliance-and-enforcement/property/property.dto';
 import { ComplianceAndEnforcementPropertyService } from '../../../services/compliance-and-enforcement/property/property.service';
 import { DOCUMENT_SOURCE, DOCUMENT_TYPE } from '../../../shared/document/document.dto';
-import { DocumentUploadDialogOptions } from '../../../shared/document-upload-dialog/document-upload-dialog.interface';
+import { DocumentUploadDialogData } from '../../../shared/document-upload-dialog/document-upload-dialog.interface';
 import { Section } from '../../../services/compliance-and-enforcement/documents/document.service';
 import { ResponsiblePartiesComponent } from '../responsible-parties/responsible-parties.component';
+import { ResponsiblePartiesService } from '../../../services/compliance-and-enforcement/responsible-parties/responsible-parties.service';
 
 @Component({
   selector: 'app-compliance-and-enforcement-draft',
@@ -29,7 +41,9 @@ import { ResponsiblePartiesComponent } from '../responsible-parties/responsible-
 export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
   Section = Section;
 
-  submissionDocumentOptions: DocumentUploadDialogOptions = {
+  submissionDocumentOptions: DocumentUploadDialogData = {
+    // A necessary hack to make this work without rewriting lots of code
+    fileId: '',
     allowedVisibilityFlags: [],
     allowsFileEdit: true,
     allowedDocumentSources: [
@@ -42,6 +56,29 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
     allowedDocumentTypes: [DOCUMENT_TYPE.COMPLAINT, DOCUMENT_TYPE.REFERRAL],
   };
 
+  ownershipDocumentOptions: DocumentUploadDialogData = {
+    // A necessary hack to make this work without rewriting lots of code
+    fileId: '',
+    parcelService: this.propertyService,
+    submissionService: this.responsiblePartyService,
+    allowedVisibilityFlags: [],
+    allowsFileEdit: true,
+    allowedDocumentSources: [
+      DOCUMENT_SOURCE.PUBLIC,
+      DOCUMENT_SOURCE.LFNG,
+      DOCUMENT_SOURCE.BC_GOVERNMENT,
+      DOCUMENT_SOURCE.OTHER_AGENCY,
+      DOCUMENT_SOURCE.ALC,
+    ],
+    defaultDocumentSource: DOCUMENT_SOURCE.ALC,
+    allowedDocumentTypes: [
+      DOCUMENT_TYPE.CERTIFICATE_OF_TITLE,
+      DOCUMENT_TYPE.CORPORATE_SUMMARY,
+      DOCUMENT_TYPE.BC_ASSESSMENT_REPORT,
+      DOCUMENT_TYPE.SURVEY_PLAN,
+    ],
+  };
+
   $destroy = new Subject<void>();
 
   fileNumber?: string;
@@ -64,6 +101,8 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
     private readonly complianceAndEnforcementPropertyService: ComplianceAndEnforcementPropertyService,
     private readonly route: ActivatedRoute,
     private readonly toastService: ToastService,
+    private readonly propertyService: ComplianceAndEnforcementPropertyService,
+    private readonly responsiblePartyService: ResponsiblePartiesService,
   ) {}
 
   ngOnInit(): void {
@@ -75,7 +114,12 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
   }
 
   ngAfterViewInit(): void {
-    if (!this.overviewComponent || !this.submitterComponent || !this.propertyComponent || !this.responsiblePartiesComponent) {
+    if (
+      !this.overviewComponent ||
+      !this.submitterComponent ||
+      !this.propertyComponent ||
+      !this.responsiblePartiesComponent
+    ) {
       console.warn('Not all form sections component not initialized');
       return;
     }
@@ -139,18 +183,18 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
         switchMap((property) => {
           // Only auto-save if there are meaningful changes (non-empty fields)
           const cleanedProperty = cleanPropertyUpdate(property);
-          
+
           // Check if this is a meaningful ownership type change
           const currentOwnership = this.property?.ownershipTypeCode || 'SMPL';
           const newOwnership = cleanedProperty.ownershipTypeCode;
           const isOwnershipChange = newOwnership !== undefined && newOwnership !== currentOwnership;
-          
+
           // For non-ownership changes, check if there are other meaningful changes
           const propertyForCheck = { ...cleanedProperty };
           if (propertyForCheck.ownershipTypeCode === 'SMPL') {
             delete propertyForCheck.ownershipTypeCode;
           }
-          
+
           const hasOtherActualData = Object.values(propertyForCheck).some(
             (value) => value !== null && value !== undefined && value !== '' && value !== 0,
           );
@@ -188,23 +232,19 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
       });
 
     // Watch for property ownership changes to update Crown status
-    this.propertyComponent.$changes
-      .pipe(
-        takeUntil(this.$destroy),
-      )
-      .subscribe(async (propertyUpdate) => {
-        if (propertyUpdate.ownershipTypeCode) {
-          const wasCrown = this.isPropertyCrown;
-          this.isPropertyCrown = propertyUpdate.ownershipTypeCode === 'CRWN';
-          
-          // If Crown status changed, update responsible parties component
-          if (wasCrown !== this.isPropertyCrown && this.responsiblePartiesComponent) {
-            this.responsiblePartiesComponent.isPropertyCrown = this.isPropertyCrown;
-            
-            await this.responsiblePartiesComponent.loadResponsibleParties();
-          }
+    this.propertyComponent.$changes.pipe(takeUntil(this.$destroy)).subscribe(async (propertyUpdate) => {
+      if (propertyUpdate.ownershipTypeCode) {
+        const wasCrown = this.isPropertyCrown;
+        this.isPropertyCrown = propertyUpdate.ownershipTypeCode === 'CRWN';
+
+        // If Crown status changed, update responsible parties component
+        if (wasCrown !== this.isPropertyCrown && this.responsiblePartiesComponent) {
+          this.responsiblePartiesComponent.isPropertyCrown = this.isPropertyCrown;
+
+          await this.responsiblePartiesComponent.loadResponsibleParties();
         }
-      });
+      }
+    });
   }
 
   async loadFile(fileNumber: string) {
@@ -216,18 +256,20 @@ export class DraftComponent implements OnInit, AfterViewInit, OnDestroy {
       // Load property data
       if (this.file.uuid) {
         try {
-          this.property = await this.complianceAndEnforcementPropertyService.fetchByFileUuid(this.file.uuid);
-          
+          const properties = await this.complianceAndEnforcementPropertyService.fetchParcels(this.file.uuid);
+          // There should only ever be one while in draft
+          this.property = properties[0];
+
           if (this.propertyComponent && this.property) {
             this.propertyComponent.property = this.property;
           }
           // Check if property is Crown ownership
           this.isPropertyCrown = this.property?.ownershipTypeCode === 'CRWN';
-          
+
           // Set the Crown status on the responsible parties component and load parties
           if (this.responsiblePartiesComponent) {
             this.responsiblePartiesComponent.isPropertyCrown = this.isPropertyCrown;
-            // Manually trigger loading since the component's ngOnInit ran before file was loaded up 
+            // Manually trigger loading since the component's ngOnInit ran before file was loaded up
             if (this.file?.uuid) {
               this.responsiblePartiesComponent.fileUuid = this.file.uuid;
               await this.responsiblePartiesComponent.loadResponsibleParties();

alcs-frontend/src/app/services/compliance-and-enforcement/documents/document.dto.ts

@@ -19,9 +19,12 @@ export interface UpdateComplianceAndEnforcementDocumentDto {
 
   source?: DOCUMENT_SOURCE;
   fileName?: string;
-  section?: Section;
+
+  parcelUuid?: string;
+  ownerUuid?: string;
 }
 
 export interface CreateComplianceAndEnforcementDocumentDto extends UpdateComplianceAndEnforcementDocumentDto {
+  section?: Section;
   file: File;
 }

alcs-frontend/src/app/services/compliance-and-enforcement/documents/document.service.ts

@@ -1,5 +1,5 @@
 import { HttpClient, HttpParams } from '@angular/common/http';
-import { Injectable } from '@angular/core';
+import { CSP_NONCE, Injectable } from '@angular/core';
 import { firstValueFrom } from 'rxjs';
 import { environment } from '../../../../environments/environment';
 import {
@@ -12,6 +12,7 @@ import { downloadFileFromUrl, openFileInline } from '../../../shared/utils/file'
 
 export enum Section {
   SUBMISSION = 'Submission',
+  OWNERSHIP = 'Ownership',
 }
 
 @Injectable({
@@ -96,6 +97,12 @@ export class ComplianceAndEnforcementDocumentService {
     if (dto.section) {
       formData.append('section', dto.section);
     }
+    if (dto.parcelUuid) {
+      formData.append('parcelUuid', dto.parcelUuid);
+    }
+    if (dto.ownerUuid) {
+      formData.append('ownerUuid', dto.ownerUuid);
+    }
 
     return formData;
   }

alcs-frontend/src/app/services/compliance-and-enforcement/property/property.dto.ts

@@ -10,13 +10,14 @@ export interface ComplianceAndEnforcementPropertyDto {
   latitude: number;
   longitude: number;
   ownershipTypeCode: string;
-  pid?: string | null;
+  pid?: string;
   pin?: string | null;
   areaHectares: number;
   alrPercentage: number;
   alcHistory: string;
   localGovernment?: BaseCodeDto;
   ownershipType?: BaseCodeDto;
+  certificateOfTitleUuid?: string;
 }
 
 export interface UpdateComplianceAndEnforcementPropertyDto {
@@ -33,6 +34,7 @@ export interface UpdateComplianceAndEnforcementPropertyDto {
   alrPercentage?: number | null;
   alcHistory?: string | null;
   fileUuid?: string | null;
+  certificateOfTitleUuid?: string;
 }
 
 export interface CreateComplianceAndEnforcementPropertyDto {
@@ -49,4 +51,5 @@ export interface CreateComplianceAndEnforcementPropertyDto {
   alrPercentage: number;
   alcHistory: string;
   fileUuid: string;
+  certificateOfTitleUuid?: string;
 }

alcs-frontend/src/app/services/compliance-and-enforcement/property/property.service.ts

@@ -2,10 +2,7 @@ import { HttpClient } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { firstValueFrom, Observable } from 'rxjs';
 import { environment } from '../../../../environments/environment';
-import {
-  ComplianceAndEnforcementPropertyDto,
-  UpdateComplianceAndEnforcementPropertyDto,
-} from './property.dto';
+import { ComplianceAndEnforcementPropertyDto, UpdateComplianceAndEnforcementPropertyDto } from './property.dto';
 
 @Injectable({
   providedIn: 'root',
@@ -15,15 +12,11 @@ export class ComplianceAndEnforcementPropertyService {
 
   constructor(private readonly http: HttpClient) {}
 
-  async fetchByFileUuid(fileUuid: string): Promise<ComplianceAndEnforcementPropertyDto> {
-    return await firstValueFrom(
-      this.http.get<ComplianceAndEnforcementPropertyDto>(`${this.url}/${fileUuid}`),
-    );
+  async fetchParcels(fileNumber: string): Promise<ComplianceAndEnforcementPropertyDto[]> {
+    return await firstValueFrom(this.http.get<ComplianceAndEnforcementPropertyDto[]>(`${this.url}/${fileNumber}`));
   }
 
-  create(
-    createDto: UpdateComplianceAndEnforcementPropertyDto,
-  ): Observable<ComplianceAndEnforcementPropertyDto> {
+  create(createDto: UpdateComplianceAndEnforcementPropertyDto): Observable<ComplianceAndEnforcementPropertyDto> {
     return this.http.post<ComplianceAndEnforcementPropertyDto>(this.url, createDto);
   }
 

alcs-frontend/src/app/services/compliance-and-enforcement/responsible-parties/responsible-parties.dto.ts

@@ -30,7 +30,7 @@ export interface ResponsiblePartyDto {
   individualTelephone?: string;
   individualEmail?: string;
   individualNote?: string;
-  
+
   // Organization fields
   organizationName?: string;
   organizationTelephone?: string;
@@ -68,7 +68,7 @@ export interface CreateResponsiblePartyDto {
   individualTelephone?: string;
   individualEmail?: string;
   individualNote?: string;
-  
+
   // Organization fields
   organizationName?: string;
   organizationTelephone?: string;
@@ -91,7 +91,7 @@ export interface UpdateResponsiblePartyDto {
   individualTelephone?: string;
   individualEmail?: string;
   individualNote?: string;
-  
+
   // Organization fields
   organizationName?: string;
   organizationTelephone?: string;