Merge pull request #260 from icefoganalytics/test

STA Fix and documention fixes
ytgov · Jun 19, 2024 · 13776a4 · 13776a4
2 parents f729d26 + 544c9cc
commit 13776a4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/api/repositories/assessment/assessment-sta-repository.ts b/src/api/repositories/assessment/assessment-sta-repository.ts
@@ -47,6 +47,12 @@ export class AssessmentSTA extends AssessmentBaseRepository {
       initValues.home_city_id || 0,
       initValues.destination_city_id || 0,
     ]);
+
+    // clear travel if by correspondence
+    if (this.application.is_correspondence) {
+      initValues.travel_allowance = 0;
+    }
+
     initValues.weekly_amount = await this.getScalarValue<number>("fn_get_weekly_amount_sta", [application_id]);
 
     if (disbursementList.length) {
@@ -159,6 +165,12 @@ export class AssessmentSTA extends AssessmentBaseRepository {
       refreshData.home_city_id || 0,
       refreshData.destination_city_id || 0,
     ]);
+
+    // clear travel if by correspondence
+    if (this.application.is_correspondence) {
+      refreshData.travel_allowance = 0;
+    }
+
     refreshData.weekly_amount = await this.getScalarValue<number>("fn_get_weekly_amount_sta", [application_id]);
 
     refreshData.assessed_amount = await this.getScalarValue<number>("fn_get_other_inst_total_sta", [

diff --git a/src/api/services/shared/documentation-service.ts b/src/api/services/shared/documentation-service.ts
@@ -201,6 +201,12 @@ export class DocumentationService {
         case "Spouse as Dependent":
           // I don't currently know how to handle this...
           break;
+        case "Permanent Resident":
+          if (app.statistical.citizenship != "3") doc.meets_conditions = false;
+          break;
+        case "Second Residence":
+          if (app.funding_sources.second_residence != true) doc.meets_conditions = false;
+          break;
         case "Yukon and Previous CSL":
           // I don't currently know how to handle this...
           // But I belive it's handled by not having the requirement