Ready to publish

Added new Email Template and new Modal form for expired TECNs.
Updated Email Template and new Modal form for TECNs.
Added more margin on all modal dialogs

.vscode/settings.json

@@ -420,6 +420,5 @@
     "windowsphone",
     "Winsock",
     "worlflow"
-  ],
-  "dotnet.preferCSharpExtension": true
+  ]
 }

.vscode/tasks.json

@@ -2,7 +2,7 @@
     "version": "2.0.0",
     "tasks": [
         {
-            "label": "MSBuild-Debug",
+            "label": "MSBuild",
             "command": "C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe",
             "type": "process",
             "args": [
@@ -16,21 +16,6 @@
             ],
             "problemMatcher": "$msCompile"
         },
-        {
-            "label": "MSBuild-Release",
-            "command": "C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe",
-            "type": "process",
-            "args": [
-                "/target:Build",
-                "/restore:True",
-                "/p:RestoreSources=https://artifactory.intra.infineon.com/artifactory/api/nuget/ngt-fi-package-main-vir/%3Bhttps://packagemanagement.eu.infineon.com:4430/api/v2/%3Bhttps://api.nuget.org/v3/index.json",
-                "/detailedsummary",
-                "/consoleloggerparameters:PerformanceSummary;ErrorsOnly;",
-                "/property:Configuration=Release;TargetFrameworkVersion=v4.8",
-                "Fab2ApprovalSystem/Fab2ApprovalSystem.csproj"
-            ],
-            "problemMatcher": "$msCompile"
-        },
         {
             "label": "Fab2ApprovalMKLink-User Secrets Init",
             "command": "dotnet",

Fab2ApprovalMKLink/.editorconfig

@@ -367,7 +367,6 @@ dotnet_diagnostic.IDE0065.severity = none # Question -
 dotnet_diagnostic.IDE0066.severity = none # Question - Use 
 dotnet_diagnostic.IDE0078.severity = none # Question - Use pattern matching (may change code meaning)
 dotnet_diagnostic.IDE0090.severity = warning # Question - Simplify new expression
-dotnet_diagnostic.IDE0100.severity = error # Question - Remove redundant equality
 dotnet_diagnostic.IDE0160.severity = warning # Question - Use block-scoped namespace
 dotnet_diagnostic.IDE0161.severity = warning # Question - Namespace declaration preferences
 dotnet_diagnostic.IDE0270.severity = none # Question - Null check can be simplified

Fab2ApprovalMKLink/Fab2ApprovalMKLink.csproj

@@ -18,15 +18,9 @@
   <PropertyGroup Condition="'$(TargetFramework)' == 'net8.0'">
     <DefineConstants>NET8</DefineConstants>
   </PropertyGroup>
-  <ItemGroup>
-    <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
-      <_Parameter1>Fab2ApprovalTests</_Parameter1>
-    </AssemblyAttribute>
-  </ItemGroup>
   <ItemGroup>
     <PackageReference Include="Dapper.Contrib" Version="2.0.78" />
     <PackageReference Include="Dapper" Version="2.1.44" />
-    <PackageReference Include="DocumentFormat.OpenXml" Version="3.3.0" />
     <PackageReference Include="EntityFramework" Version="6.5.1" />
     <PackageReference Include="ExcelDataReader" Version="3.7.0" />
     <PackageReference Include="jQuery" Version="3.7.1" />

Fab2ApprovalSystem-Development.yml

@@ -0,0 +1,105 @@
+trigger:
+  branches:
+    include:
+      - Development
+  paths:
+    include:
+      - "Fab2ApprovalSystem/*"
+    exclude:
+      - "**/*.yaml"
+      - "**/*.yml"
+      - "SQL/*"
+      - "references/*"
+      - "packages/*"
+      - "Kendo/*"
+
+pool:
+  name: Mesa-IIS
+  demands: Fab2ApprovalSystem-Development
+
+variables:
+  # solution: '**/*.sln'
+  # buildPlatform: 'Any CPU'
+  buildConfiguration: "Debug"
+  ASPNETCORE_ENVIRONMENT: "Development"
+
+steps:
+  - script: |
+      set assemblyTitle=Fab2ApprovalSystem
+      echo %assemblyTitle%
+      echo ##vso[task.setvariable variable=AssemblyTitle;]%assemblyTitle%
+      echo $(AssemblyTitle)
+    displayName: AssemblyTitle
+
+  - script: |
+      set targetFrameworkVersion=v4.8
+      echo %targetFrameworkVersion%
+      echo ##vso[task.setvariable variable=TargetFrameworkVersion;]%targetFrameworkVersion%
+      echo $(TargetFrameworkVersion)
+    displayName: TargetFrameworkVersion
+
+  - script: |
+      set coreVersion=na
+      echo %coreVersion%
+      echo ##vso[task.setvariable variable=CoreVersion;]%coreVersion%
+      echo $(CoreVersion)
+    displayName: CoreVersion
+
+  - script: |
+      set configuration=Debug
+      echo %configuration%
+      echo ##vso[task.setvariable variable=Configuration;]%configuration%
+      echo $(Configuration)
+    displayName: Configuration
+
+  - script: |
+      REM set nugetSource=https://messa017.infineon.com/v3/index.json
+      set nugetSource=https://eaf-prod.mes.infineon.com/v3/index.json
+      echo %nugetSource%
+      echo ##vso[task.setvariable variable=NugetSource;]%nugetSource%
+      echo $(NugetSource)
+    displayName: NugetSource
+
+  - script: |
+      set gitCommit=$(Build.SourceVersion)
+      set gitCommitSeven=%gitCommit:~0,7%
+      echo %gitCommitSeven%
+      echo ##vso[task.setvariable variable=GitCommitSeven;]%gitCommitSeven%
+      echo $(GitCommitSeven)
+    displayName: GitCommitSeven
+
+  - script: |
+      echo $(AssemblyTitle)
+      echo $(Build.BuildId)
+      echo $(Build.Reason)
+      echo $(Build.Repository.Id)
+      echo $(Build.Repository.Name)
+      echo $(Build.SourceVersion)
+      echo $(CoreVersion)
+      echo $(Configuration)
+      echo $(NugetSource)
+      echo $(GitCommitSeven)
+      echo $(TargetFrameworkVersion)
+      REM echo $(pipelinePassword)
+    displayName: "Echo Check"
+
+  - script: '"C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=$(NugetSource) $(AssemblyTitle).csproj'
+    workingDirectory: Fab2ApprovalSystem
+    displayName: "Framework Restore"
+
+  - script: '"C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
+    workingDirectory: Fab2ApprovalSystem
+    displayName: "Framework Build"
+
+  - script: '"C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedBuildConfiguration=$(Configuration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath="D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(GitCommitSeven)-$(Build.BuildId)-$(Build.Repository.Name)-$(Configuration)" /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
+    workingDirectory: Fab2ApprovalSystem
+    displayName: "Framework Pack"
+
+  - script: '"C:/Program Files (x86)/IIS/Microsoft Web Deploy V3/MSDeploy.exe" -AllowUntrusted -dest:auto -disableLink:AppPoolExtension -disableLink:CertificateExtension -disableLink:ContentExtension -setParam:name="IIS Web Application Name",value=$(Build.Repository.Name) -setParamFile:"D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(GitCommitSeven)-$(Build.BuildId)-$(Build.Repository.Name)-$(Configuration)" "/_PublishedWebsites/$(Build.Repository.Name)_Package/$(Build.Repository.Name).SetParameters.xml" -source:package="D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(GitCommitSeven)-$(Build.BuildId)-$(Build.Repository.Name)-$(Configuration)" "/_PublishedWebsites/$(AssemblyTitle)_Package/$(AssemblyTitle).zip" -verb:sync'
+    workingDirectory: Fab2ApprovalSystem
+    displayName: "Framework Deploy"
+    enabled: false
+
+  - script: 'echo $(Build.SourceVersion)-$(Build.BuildId)>bin_x_x_\$(Configuration)\$(CoreVersion)\win-x64\$(Build.Repository.Name).txt'
+    displayName: "Force Fail"
+    enabled: false

Fab2ApprovalSystem.sln

@@ -6,9 +6,6 @@ MinimumVisualStudioVersion = 10.0.40219.1
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Fab2ApprovalSystem", "Fab2ApprovalSystem\Fab2ApprovalSystem.csproj", "{AAE52608-4DD1-4732-92BD-CC8915DEC71E}"
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MesaFabApproval.API", "MesaFabApproval.API\MesaFabApproval.API.csproj", "{852E528D-015A-43B5-999D-F281E3359E5E}"
-	ProjectSection(ProjectDependencies) = postProject
-		{2C16014D-B04E-46AF-AB4C-D2691D44A339} = {2C16014D-B04E-46AF-AB4C-D2691D44A339}
-	EndProjectSection
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MesaFabApproval.Shared", "MesaFabApproval.Shared\MesaFabApproval.Shared.csproj", "{2C16014D-B04E-46AF-AB4C-D2691D44A339}"
 EndProject
@@ -17,18 +14,6 @@ Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MesaFabApproval.Client", "M
 		{2C16014D-B04E-46AF-AB4C-D2691D44A339} = {2C16014D-B04E-46AF-AB4C-D2691D44A339}
 	EndProjectSection
 EndProject
-Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MesaFabApproval.API.Test", "MesaFabApproval.API.Test\MesaFabApproval.API.Test.csproj", "{D03AB305-BA29-4EB1-AC66-ABBF76FBF5C1}"
-	ProjectSection(ProjectDependencies) = postProject
-		{2C16014D-B04E-46AF-AB4C-D2691D44A339} = {2C16014D-B04E-46AF-AB4C-D2691D44A339}
-		{852E528D-015A-43B5-999D-F281E3359E5E} = {852E528D-015A-43B5-999D-F281E3359E5E}
-	EndProjectSection
-EndProject
-Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MesaFabApproval.Client.Test", "MesaFabApproval.Client.Test\MesaFabApproval.Client.Test.csproj", "{A0E5BD7D-3910-43BD-BBA3-3820AD524423}"
-	ProjectSection(ProjectDependencies) = postProject
-		{2C16014D-B04E-46AF-AB4C-D2691D44A339} = {2C16014D-B04E-46AF-AB4C-D2691D44A339}
-		{34D52F44-A81F-4247-8180-16E204824A07} = {34D52F44-A81F-4247-8180-16E204824A07}
-	EndProjectSection
-EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -51,14 +36,6 @@ Global
 		{34D52F44-A81F-4247-8180-16E204824A07}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{34D52F44-A81F-4247-8180-16E204824A07}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{34D52F44-A81F-4247-8180-16E204824A07}.Release|Any CPU.Build.0 = Release|Any CPU
-		{D03AB305-BA29-4EB1-AC66-ABBF76FBF5C1}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
-		{D03AB305-BA29-4EB1-AC66-ABBF76FBF5C1}.Debug|Any CPU.Build.0 = Debug|Any CPU
-		{D03AB305-BA29-4EB1-AC66-ABBF76FBF5C1}.Release|Any CPU.ActiveCfg = Release|Any CPU
-		{D03AB305-BA29-4EB1-AC66-ABBF76FBF5C1}.Release|Any CPU.Build.0 = Release|Any CPU
-		{A0E5BD7D-3910-43BD-BBA3-3820AD524423}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
-		{A0E5BD7D-3910-43BD-BBA3-3820AD524423}.Debug|Any CPU.Build.0 = Debug|Any CPU
-		{A0E5BD7D-3910-43BD-BBA3-3820AD524423}.Release|Any CPU.ActiveCfg = Release|Any CPU
-		{A0E5BD7D-3910-43BD-BBA3-3820AD524423}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE

Fab2ApprovalSystem/Controllers/AuditController.cs

@@ -58,7 +58,6 @@ public class AuditController : Controller {
         try {
             bool isAdmin = (bool)Session[GlobalVars.IS_ADMIN];
             int userId = (int)Session[GlobalVars.SESSION_USERID];
-            audit = auditDMO.GetAuditItem(issueID, userId);
             AuditEdit auditEdit = auditDMO.GetAuditEdit(issueID, audit, isAdmin, userId);
             if (auditEdit.RedirectToAction)
                 return RedirectToAction("ReadOnlyAudit", new { auditNo = audit.AuditNo });

Fab2ApprovalSystem/Controllers/ChangeControlController.cs

@@ -52,7 +52,7 @@ public class ChangeControlController : Controller {
         string encodedJwt = System.Net.WebUtility.UrlEncode(jwt);
         string refreshToken = Session["RefreshToken"].ToString();
         string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
-        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=pcrb/{issueID}";
+        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=/pcrb/{issueID}";
 
         return Redirect(mrbUrl);
     }
@@ -62,7 +62,7 @@ public class ChangeControlController : Controller {
         string encodedJwt = System.Net.WebUtility.UrlEncode(jwt);
         string refreshToken = Session["RefreshToken"].ToString();
         string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
-        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=pcrb/{issueID}";
+        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=/pcrb/{issueID}";
 
         return Redirect(mrbUrl);
     }

Fab2ApprovalSystem/Controllers/ECNController.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Configuration;
-using System.IO;
 using System.Linq;
 using System.Web;
 using System.Web.Mvc;
@@ -22,7 +21,7 @@ namespace Fab2ApprovalSystem.Controllers;
 [Authorize]
 [OutputCache(NoStore = true, Duration = 0, VaryByParam = "*")]
 [SessionExpireFilter]
-public class ECNController : Controller {
+public class ECNController : PdfViewController {
 
     private const string ECN_PREFIX = "ECN_";
     private const string TECN_PREFIX = "TECN_";
@@ -670,6 +669,9 @@ public class ECNController : Controller {
         }
     }
 
+    /// <summary>
+    /// Get a list of Approvers and the status
+    /// </summary>
     public ActionResult GetApproversList([DataSourceRequest] DataSourceRequest request, int issueID, byte step, bool isTECN, bool isEmergrncyTECN) {
         int isITARCompliant = 0;
         ECN ecn = new ECN();
@@ -881,8 +883,6 @@ public class ECNController : Controller {
 
             string outputFileName = "";
             ecn = ecnDMO.GetECNPdf(ecnNumber);
-            ViewBag.Category = ecnDMO.GetCategoryID(ecn);
-            ViewBag.TrainingNotificationTo = ecnDMO.GetTrainingNotificationTo(ecn, trainingDMO);
             outputFileName = ecnNumber.ToString() + ".pdf";
 
             string ecnFolderPath = _AppSettings.AttachmentFolder + "ECN\\" + ecnNumber.ToString();
@@ -891,12 +891,10 @@ public class ECNController : Controller {
             if (!di.Exists)
                 di.Create();
 
-            string htmlText;
-            string pageTitle = string.Empty;
-            htmlText = RenderViewToString("ECNPdf", ecn);
-            StandardPdfRenderer.WritePortableDocumentFormatToFile(pageTitle, htmlText, $"{ecnFolderPath}\\ECNForm_{outputFileName}");
-            htmlText = RenderViewToString("ECNApprovalPdf", ecn);
-            StandardPdfRenderer.WritePortableDocumentFormatToFile(pageTitle, htmlText, $"{ecnFolderPath}\\ECNApprovalLog_{outputFileName}");
+            // To render a PDF instead of an HTML, all we need to do is call ViewPdf instead of View. This
+            // requires the controller to be inherited from MyController instead of MVC's Controller.
+            SavePdf(ecnFolderPath + "\\ECNForm_" + outputFileName, "ECNPdf", ecn);
+            SavePdf(ecnFolderPath + "\\ECNApprovalLog_" + outputFileName, "ECNApprovalPdf", ecn);
         } catch (Exception ex) {
             EventLogDMO.Add(new WinEventLog() { IssueID = ecnNumber, UserID = @User.Identity.Name, DocumentType = "ECN", OperationType = "Generate PDF", Comments = ex.Message });
             ecn = null;
@@ -906,32 +904,12 @@ public class ECNController : Controller {
         return true;
     }
 
-    private string RenderViewToString(string viewName, ECNPdf ecnPdf) {
-        string result;
-        ViewData.Model = ecnPdf;
-        using (StringWriter writer = new()) {
-            try {
-                ViewEngineResult viewResult = ViewEngines.Engines.FindView(ControllerContext, viewName, string.Empty);
-                if (viewResult is null)
-                    return $"A view with the name '{viewName}' could not be found";
-                ViewContext viewContext = new(ControllerContext, viewResult.View, ViewData, TempData, writer);
-                viewResult.View.Render(viewContext, writer);
-                result = writer.GetStringBuilder().ToString();
-            } catch (Exception ex) {
-                result = $"Failed - {ex.Message}";
-            }
-        }
-        return result;
-    }
-
     public bool GenerateECNPdfDifferentLocation(int ecnNumber, int folderName) {
         ECNPdf ecn = new ECNPdf();
         try {
             string outputFileName = "";
 
             ecn = ecnDMO.GetECNPdf(ecnNumber);
-            ViewBag.Category = ecnDMO.GetCategoryID(ecn);
-            ViewBag.TrainingNotificationTo = ecnDMO.GetTrainingNotificationTo(ecn, trainingDMO);
             outputFileName = ecnNumber.ToString() + ".pdf";
 
             string ecnFolderPath = _AppSettings.AttachmentFolder + "ECN\\" + folderName.ToString();
@@ -941,9 +919,9 @@ public class ECNController : Controller {
             if (!di.Exists)
                 di.Create();
 
-            string pageTitle = string.Empty;
-            string htmlText = RenderViewToString("ECNPdf", ecn);
-            StandardPdfRenderer.WritePortableDocumentFormatToFile(pageTitle, htmlText, $"{ecnFolderPath}\\ECNForm_{outputFileName}");
+            // To render a PDF instead of an HTML, all we need to do is call ViewPdf instead of View. This
+            // requires the controller to be inherited from MyController instead of MVC's Controller.
+            SavePdf(ecnFolderPath + "\\ECNForm_" + outputFileName, "ECNPdf", ecn);
         } catch (Exception ex) {
             EventLogDMO.Add(new WinEventLog() { IssueID = ecnNumber, UserID = @User.Identity.Name, DocumentType = "ECN", OperationType = "Generate PDF", Comments = ex.Message });
             ecn = null;
@@ -957,20 +935,13 @@ public class ECNController : Controller {
         ECNPdf ecn;
         try {
             ecn = ecnDMO.GetECNPdf(ecnNumber);
-            ViewBag.Category = ecnDMO.GetCategoryID(ecn);
-            ViewBag.TrainingNotificationTo = ecnDMO.GetTrainingNotificationTo(ecn, trainingDMO);
-            string pageTitle = string.Empty;
-            string htmlText = RenderViewToString("ECNPdf", ecn);
-            if (System.Diagnostics.Debugger.IsAttached) {
-                return Content(htmlText, "text/html");
-            } else {
-                byte[] buffer = StandardPdfRenderer.GetPortableDocumentFormatBytes(pageTitle, htmlText);
-                return new BinaryContentResult(buffer, "application/pdf");
-            }
+            // To render a PDF instead of an HTML, all we need to do is call ViewPdf instead of View. This
+            // requires the controller to be inherited from MyController instead of MVC's Controller.
+            return this.ViewPdf("", "ECNPdf", ecn);
         } catch (Exception ex) {
             EventLogDMO.Add(new WinEventLog() { IssueID = ecnNumber, UserID = @User.Identity.Name, DocumentType = "ECN", OperationType = "Print PDF", Comments = ex.Message });
             ecn = null;
-            return Content("An unexpected error has occurred!");
+            return Content("");
         }
     }
 
@@ -1188,8 +1159,8 @@ public class ECNController : Controller {
     public ActionResult CancelDocument(int ecnNumber, byte currentStep, int documentType, string ecnTypeString, string comments = "") {
         ECN ecn = ecnDMO.GetECN(ecnNumber);
         bool lastApproverAndLastStep = false;
-        if (ecn.SubmitedDate is not null && currentStep >= 1)
-            wfDMO.Approve(_AppSettings, ecnNumber, currentStep, comments, out bool lastStep, (int)Session[GlobalVars.SESSION_USERID], documentType, ecn.WorkFlowNumber);
+        if (ecn.IsTECN && ecn.SubmitedDate is not null && currentStep >= 1)
+            Approve(ecnNumber, currentStep, comments, documentType, ecnTypeString);
         int appoverCount = ecnDMO.SubmitForCancellation(ecnNumber, (byte)GlobalVars.WorkFLowStepNumber.Step1, (int)Session[GlobalVars.SESSION_USERID], documentType, ecnTypeString, (int)GlobalVars.TECNExpirationCancellation.Cancellation);
         if (appoverCount > 0) {
             NotifyApproversForCancellation(ecnNumber, ecn, currentStep, documentType, ecnTypeString);
@@ -1250,12 +1221,6 @@ public class ECNController : Controller {
             try {
                 lastApproverAndLastStep = true;
 
-                ecn.CancellationDate = DateTime.Now;
-                ecn.CancellationApprovalDate = DateTime.Now;
-                ecn.CancellationApproved = true;
-                ecn.Cancelled = true;
-                ecnDMO.UpdateECN(ecn);
-
                 ECNPdf ecnPDF = new ECNPdf();
                 GenerateECNPdf(ecnNumber, out ecnPDF);
 

Fab2ApprovalSystem/Controllers/LotTravelerController.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Configuration;
-using System.IO;
 using System.Linq;
 using System.Threading;
 using System.Web;
@@ -21,7 +20,7 @@ namespace Fab2ApprovalSystem.Controllers;
 [Authorize]
 [OutputCache(NoStore = true, Duration = 0, VaryByParam = "*")]
 [SessionExpireFilter]
-public class LotTravelerController : Controller {
+public class LotTravelerController : PdfViewController {
 
     LotTravelerDMO LotTravDMO = new LotTravelerDMO();
     string docTypeString = "LotTraveler";
@@ -200,14 +199,9 @@ public class LotTravelerController : Controller {
         try {
             workRequest = LotTravDMO.GetLTWorkRequestItemPDF(workRequestID);
 
-            string pageTitle = string.Empty;
-            string htmlText = RenderViewToString("WorkRequestPDF", workRequest);
-            if (System.Diagnostics.Debugger.IsAttached) {
-                return Content(htmlText, "text/html");
-            } else {
-                byte[] buffer = StandardPdfRenderer.GetPortableDocumentFormatBytes(pageTitle, htmlText);
-                return new BinaryContentResult(buffer, "application/pdf");
-            }
+            // To render a PDF instead of an HTML, all we need to do is call ViewPdf instead of View. This
+            // requires the controller to be inherited from MyController instead of MVC's Controller.
+            return this.ViewPdf("", "WorkRequestPDF", workRequest);
         } catch (Exception ex) {
             Functions.WriteEvent(_AppSettings, @User.Identity.Name + "\r\n DisplayWorkRequestPDF - LotTraveler\r\n" + ex.InnerException.ToString(), System.Diagnostics.EventLogEntryType.Error);
             EventLogDMO.Add(new WinEventLog() { IssueID = workRequest.SWRNumber, UserID = @User.Identity.Name, DocumentType = "LotTravler", OperationType = "Generate PDF", Comments = ex.Message });
@@ -216,24 +210,6 @@ public class LotTravelerController : Controller {
         }
     }
 
-    private string RenderViewToString(string viewName, object model) {
-        string result;
-        ViewData.Model = model;
-        using (StringWriter writer = new()) {
-            try {
-                ViewEngineResult viewResult = ViewEngines.Engines.FindView(ControllerContext, viewName, string.Empty);
-                if (viewResult is null)
-                    return $"A view with the name '{viewName}' could not be found";
-                ViewContext viewContext = new(ControllerContext, viewResult.View, ViewData, TempData, writer);
-                viewResult.View.Render(viewContext, writer);
-                result = writer.GetStringBuilder().ToString();
-            } catch (Exception ex) {
-                result = $"Failed - {ex.Message}";
-            }
-        }
-        return result;
-    }
-
     public ActionResult WorkRequestRevision(int workRequestID) {
         int isITARCompliant = 1;
         LTWorkRequest workRequest = new LTWorkRequest();
@@ -275,7 +251,9 @@ public class LotTravelerController : Controller {
 
         return Content("Successfully Saved");
     }
-
+    /// <summary>
+    /// 
+    /// </summary>
     public JsonResult GetBaseFlowLocations(string baseFlow) {
         List<BaseFlowLocation> loclist = LotTravDMO.GetBaseFlowLocations(baseFlow);
         return Json(loclist, JsonRequestBehavior.AllowGet);
@@ -354,6 +332,9 @@ public class LotTravelerController : Controller {
         return Content(newWorkRequestID.ToString());
     }
 
+    /// <summary>
+    /// For the Revison
+    /// </summary>
     public ActionResult UpdateMaterialDetailRevision(LTWorkRequest model) {
         var modelMaterialDetail = model.LTMaterial;
         int previousMaterialID = model.LTMaterial.ID;
@@ -1251,19 +1232,15 @@ public class LotTravelerController : Controller {
         LotTravDMO.DeleteLot(ltLotID);
     }
 
+    ///
     public ActionResult DisplayLotTravlerPdf(int ltLotID, int revisionNumber) {
         LotTravelerPdf traveler = new LotTravelerPdf();
         try {
             traveler = LotTravDMO.GetLotTravlerPdf(ltLotID, revisionNumber);
 
-            string pageTitle = string.Empty;
-            string htmlText = RenderViewToString("LotTravelerPDF", traveler);
-            if (System.Diagnostics.Debugger.IsAttached) {
-                return Content(htmlText, "text/html");
-            } else {
-                byte[] buffer = StandardPdfRenderer.GetPortableDocumentFormatBytes(pageTitle, htmlText);
-                return new BinaryContentResult(buffer, "application/pdf");
-            }
+            // To render a PDF instead of an HTML, all we need to do is call ViewPdf instead of View. This
+            // requires the controller to be inherited from MyController instead of MVC's Controller.
+            return this.ViewPdf("", "LotTravelerPDF", traveler);
         } catch (Exception ex) {
             EventLogDMO.Add(new WinEventLog() { IssueID = traveler.SWRNumber, UserID = @User.Identity.Name, DocumentType = "LotTraveler", OperationType = "Generate PDF", Comments = ex.Message });
             traveler = null;

Fab2ApprovalSystem/Controllers/MRBController.cs

@@ -79,7 +79,7 @@ public class MRBController : Controller {
         string encodedJwt = System.Net.WebUtility.UrlEncode(jwt);
         string refreshToken = Session["RefreshToken"].ToString();
         string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
-        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=mrb/{issueID}";
+        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=/mrb/{issueID}";
 
         return Redirect(mrbUrl);
     }
@@ -102,7 +102,7 @@ public class MRBController : Controller {
         string encodedJwt = System.Net.WebUtility.UrlEncode(jwt);
         string refreshToken = Session["RefreshToken"].ToString();
         string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
-        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=mrb/{issueID}";
+        string mrbUrl = $"{GlobalVars.AppSettings.WasmClientUrl}/redirect?jwt={encodedJwt}&refreshToken={encodedRefreshToken}&redirectPath=/mrb/{issueID}";
 
         return Redirect(mrbUrl);
     }

Fab2ApprovalSystem/DMO/AdminDMO.cs

@@ -37,7 +37,7 @@ public class AdminDMO {
             foreach (SubRole sr in r.SubRoles) {
                 if (sr.Inactive) {
                     // hide inactive roles unless parameter says otherwise
-                    if (!showInactiveRoles.Equals("true"))
+                    if (showInactiveRoles.Equals("true") == false)
                         continue;
                 }
 

Fab2ApprovalSystem/DMO/AuditDMO.cs

@@ -401,6 +401,7 @@ public class AuditDMO {
             result.Is8DQA = "true";
         }
 
+        audit = GetAuditItem(issueID, userId);
         // transform audit users from string to list, delimited by a comma.
         if (audit.Auditees == null) {
             result.AuditeeNames = new List<string>();
@@ -413,7 +414,7 @@ public class AuditDMO {
         if (audit.OriginatorID == userId) {
             result.IsSubmitter = true;
         }
-        if (!isAdmin) {
+        if (isAdmin != true) {
             result.IsAdmin = false;
         } else {
             result.IsAdmin = true;
@@ -423,7 +424,7 @@ public class AuditDMO {
         {
             result.RedirectToAction = true;
         }
-        if (!result.IsAdmin && !result.IsSubmitter) {
+        if (result.IsAdmin == false && result.IsSubmitter == false) {
             result.RedirectToAction = true;
         } else {
             result.UserList = GetUserList();

Fab2ApprovalSystem/DMO/CorrectiveActionDMO.cs

@@ -324,7 +324,7 @@ public class CorrectiveActionDMO {
         bool isAssignee = false;
         int aiIndex = 0;
         List<D5D6CorrectivetAction> actionItems = GetD5D6CorrectivetActions(caId).ToList();
-        while (!isAssignee && aiIndex < actionItems.Count) {
+        while (isAssignee == false && aiIndex < actionItems.Count) {
             D5D6CorrectivetAction actionItem = actionItems[aiIndex];
             if (actionItem.ResponsibilityOwnerID == userId && actionItem.ImplementedDate == null && actionItem.Approved) {
                 isAssignee = true;

Fab2ApprovalSystem/DMO/ECN_DMO.cs

@@ -42,9 +42,6 @@ public class ECN_DMO {
             parameters.Add("@ExpirationDate", ecn.ExpirationDate);
             parameters.Add("@ExtensionDate", ecn.ExtensionDate);
             parameters.Add("@CancellationDate", ecn.CancellationDate);
-            parameters.Add("@CancellationApprovalDate", ecn.CancellationApprovalDate);
-            parameters.Add("@CancellationApproved", ecn.CancellationApproved);
-            parameters.Add("@Cancelled", ecn.Cancelled);
             parameters.Add("@AcknowledgementRequired", ecn.AcknowledgementRequired);
             parameters.Add("@TrainingRequired", ecn.TrainingRequired);
             parameters.Add("@AreaID", ecn.AreaID);
@@ -356,17 +353,14 @@ public class ECN_DMO {
             List<string> affectedAreas = multipleResultItems.Read<string>().ToList();
             List<string> affectedTechnologies = multipleResultItems.Read<string>().ToList();
             List<string> acknowledgementBy = multipleResultItems.Read<string>().ToList();
-            List<int> trainingby = multipleResultItems.Read<int>().ToList();
-            if (ecnItem != null && trainingby != null) {
-                if (trainingby.Count > 0)
-                    ecnItem.TrainingByIDs.AddRange(trainingby);
-            }
+            List<string> trainingBy = multipleResultItems.Read<string>().ToList();
             List<string> productfamilies = multipleResultItems.Read<string>().ToList();
 
             ecnItem.AffectedModules = string.Join(", ", modules);
             ecnItem.AffectedDepartments = string.Join(", ", departments);
             ecnItem.AffectedAreas = string.Join(",", affectedAreas);
             ecnItem.AffectedTechnologies = string.Join(",", affectedTechnologies);
+            ecnItem.TrainingBy = string.Join(",", trainingBy);
             ecnItem.AcknowledgementBy = string.Join(",", acknowledgementBy);
             ecnItem.AffectedProductFamilies = string.Join(",", productfamilies);
 
@@ -396,7 +390,7 @@ public class ECN_DMO {
 
     internal List<ECNCategory> GetCategories() {
         List<ECNCategory> r = db.Query<ECNCategory>("ECNGetCategories", null, commandType: CommandType.StoredProcedure).ToList();
-        return r.OrderBy(l => l.CategoryName).ToList();
+        return r;
     }
 
     internal List<ECNArea> GetECNAreas() {
@@ -717,26 +711,4 @@ public class ECN_DMO {
         return r;
     }
 
-    internal string GetCategoryID(ECNPdf ecn) {
-        string result;
-        if (ecn.CategoryID is null) {
-            result = string.Empty;
-        } else {
-            List<ECNCategory> categories = GetCategories();
-            result = (from l in categories where l.CategoryID == ecn.CategoryID.Value select l.CategoryName).FirstOrDefault();
-        }
-        return result;
-    }
-
-    internal string GetTrainingNotificationTo(ECNPdf ecn, TrainingDMO trainingDMO) {
-        string result;
-        if (ecn.TrainingByIDs is null) {
-            result = string.Empty;
-        } else {
-            List<TrainingGroup> trainingGroups = trainingDMO.GetTrainingGroups();
-            result = string.Join(", ", (from l in trainingGroups where ecn.TrainingByIDs.Contains(l.TrainingGroupID) select l.TrainingGroupName).ToArray());
-        }
-        return result;
-    }
-
 }

Fab2ApprovalSystem/DMO/MRB_DMO.cs

@@ -266,9 +266,9 @@ public class MRB_DMO {
         DateTime? issueStartDate = tempMRBInfo.IssueStartDate;
         DateTime? issueEndDate = tempMRBInfo.IssueEndDate;
 
-        if (!issueStartDate.HasValue)
+        if (issueStartDate.HasValue == false)
             throw new Exception("MRB Issue Start Date cannot be blank");
-        if (!issueEndDate.HasValue)
+        if (issueEndDate.HasValue == false)
             throw new Exception("MRB Issue End Date cannot be blank");
 
         MRB mrbData = new() { MRBNumber = mrbNumber, ToolCSV = tempMRBInfo.ToolCSV, IssueStartDate = tempMRBInfo.IssueStartDate, IssueEndDate = tempMRBInfo.IssueEndDate };

Fab2ApprovalSystem/DMO/WorkflowDMO.cs

@@ -166,7 +166,7 @@ public class WorkflowDMO {
             sqlString.Append("SELECT COUNT(*) FROM LotDisposition LD INNER JOIN Lot L  ON LD.IssueID = L.IssueID ");
             sqlString.Append("WHERE (PERequired = 1 OR Location = 'QDB' OR Location = 'EDB' ) AND LD.IssueID = @IssueID ");
             recordCount = db.Query<int>(sqlString.ToString(), new { IssueID = issueID }).Single();
-            if ((recordCount > 0) && (!allLotsAreScrap)) {
+            if ((recordCount > 0) && (allLotsAreScrap == false)) {
                 // check if there any IG Medical products
                 if (recordCountForIG_MA > 0) {
                     sqlString = new StringBuilder();

Fab2ApprovalSystem/EmailTemplates/TECNCancelled.txt

@@ -2,7 +2,7 @@
 *****Please DO NOT reply to this email***** 
 <br/><br/>
 {3}# {0} has been Cancelled. Please remove posted TECN from point of use. The cancellation date is {4} 
-Please review comments below and ensure process has been returned to normal.
+Please review the cancelled TECN form in the attachment.
 <br/><br/>
 https://messa016ec.infineon.com/ECN/Edit?issueID={1}
 <br/><br/>

Fab2ApprovalSystem/EmailTemplates/TECNReturnedToProcess.txt

@@ -2,7 +2,7 @@
 *****Please DO NOT reply to this email***** 
 <br/><br/>
 {3}# {0} has been returned to process. Please remove posted TECN from point of use. The returned to process date is {4} 
-Please review comments below and ensure process has been returned to normal.
+Please review the returned to process TECN form in the attachment.
 <br/><br/>
 https://messa016ec.infineon.com/ECN/Edit?issueID={1}
 <br/><br/>

Fab2ApprovalSystem/Fab2ApprovalSystem.csproj

@@ -325,6 +325,9 @@
     <Compile Include="Models\WinEventLogModel.cs" />
     <Compile Include="Models\WorkFlowModels.cs" />
     <Compile Include="PdfGenerator\BinaryContentResult.cs" />
+    <Compile Include="PdfGenerator\FakeView.cs" />
+    <Compile Include="PdfGenerator\HtmlViewRenderer.cs" />
+    <Compile Include="PdfGenerator\PdfViewController.cs" />
     <Compile Include="PdfGenerator\PrintHeaderFooter.cs" />
     <Compile Include="PdfGenerator\StandardPdfRenderer.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />

Fab2ApprovalSystem/Misc/ExcelData.cs

@@ -1,109 +1,104 @@
-#if NET8
-
-using ExcelDataReader;
-
-#else 
+#if !NET8
 
 using Excel;
 
-#endif
-
 using System;
 using System.Collections.Generic;
 using System.Data;
 using System.IO;
 using System.Linq;
+using System.Web;
 
-namespace Fab2ApprovalSystem.Misc;
+namespace Fab2ApprovalSystem.Misc {
 
-public class ExcelData {
-
-    private readonly string _Path;
-
-    public ExcelData(string path) {
-        _Path = path;
-    }
-
-    public IExcelDataReader getExcelReader() {
-        FileStream stream = File.Open(_Path, FileMode.Open, FileAccess.Read);
-        IExcelDataReader reader = null;
-        try {
-            if (_Path.EndsWith(".xls")) {
-                reader = ExcelReaderFactory.CreateBinaryReader(stream);
-            }
-            if (_Path.EndsWith(".xlsx")) {
-                reader = ExcelReaderFactory.CreateOpenXmlReader(stream);
-            }
-            return reader;
-        } catch (Exception) {
-            throw;
-        }
-    }
-
-    public class ExcelLotInfo {
-        public string? LotNo { get; set; }
-        public string? LotDispo { get; set; }
-    }
-
-    public IEnumerable<ExcelLotInfo> ReadData() {
-        List<ExcelLotInfo> r = new();
-        ExcelData excelData = new(_Path);
-        List<DataRow> lots = excelData.getData().ToList();
-
-        int lotDispoColumnIndex = -1;
-        foreach (DataColumn col in lots[0].Table.Columns) {
-            if (col.ColumnName.ToLower().Contains("dispo")) {
-                lotDispoColumnIndex = col.Ordinal;
-                break;
-            }
+    public class ExcelData {
+        string _path;
+        public ExcelData(string path) {
+            _path = path;
         }
 
-        foreach (DataRow row in lots) {
-            string temValue = row[0].ToString();
-            if (temValue.Trim().Length > 0 && temValue.Trim().Length <= 10) {
-                r.Add(new ExcelLotInfo() {
-                    LotNo = row[0].ToString(),
-                    LotDispo = (lotDispoColumnIndex >= 0 ? row[lotDispoColumnIndex].ToString() : "")
-                });
+        public IExcelDataReader getExcelReader() {
+            // ExcelDataReader works with the binary Excel file, so it needs a FileStream
+            // to get started. This is how we avoid dependencies on ACE or Interop:
+
+            FileStream stream = File.Open(_path, FileMode.Open, FileAccess.Read);
+            // We return the interface, so that 
+            IExcelDataReader reader = null;
+
+            try {
+                if (_path.EndsWith(".xls")) {
+                    reader = ExcelReaderFactory.CreateBinaryReader(stream);
+                }
+                if (_path.EndsWith(".xlsx")) {
+                    reader = ExcelReaderFactory.CreateOpenXmlReader(stream);
+                }
+                return reader;
+            } catch (Exception) {
+                throw;
             }
 
         }
-        return r;
-    }
 
-    public IEnumerable<string> ReadQDBFlagData() {
-        List<string> s = new();
-        ExcelData excelData = new(_Path);
-        IEnumerable<DataRow> lotNos = excelData.getData();
-        foreach (DataRow row in lotNos) {
-            string temValue = row[0].ToString();
-            if (temValue.Trim().Length > 0 && temValue.Trim().Length == 9) {
-                if (row[2].ToString().ToUpper() != "YES" && row[2].ToString().ToUpper() != "NO") {
-                    throw new Exception("Invalid data in the file");
-                } else {
-                    s.Add(row[0].ToString() + "~" + row[1] + "~" + row[2]);
+        public class ExcelLotInfo {
+            public string LotNo { get; set; }
+            public string LotDispo { get; set; }
+        }
+
+        public IEnumerable<ExcelLotInfo> ReadData() {
+            var r = new List<ExcelLotInfo>();
+            var excelData = new ExcelData(_path);
+            var lots = excelData.getData().ToList();
+
+            int lotDispoColumnIndex = -1;
+            foreach (DataColumn col in lots[0].Table.Columns) {
+                if (col.ColumnName.ToLower().Contains("dispo")) {
+                    lotDispoColumnIndex = col.Ordinal;
+                    break;
                 }
             }
 
+            foreach (var row in lots) {
+                string temValue = row[0].ToString();
+                if (temValue.Trim().Length > 0 && temValue.Trim().Length <= 10) {
+                    r.Add(new ExcelLotInfo() {
+                        LotNo = row[0].ToString(),
+                        LotDispo = (lotDispoColumnIndex >= 0 ? row[lotDispoColumnIndex].ToString() : "")
+                    });
+                }
+
+            }
+            return r;
         }
-        return s;
-    }
-
-#if NET8
-
-    public IEnumerable<DataRow> getData(bool firstRowIsColumnNames = true) =>
-        throw new NotImplementedException();
-
-#else
-
-    public IEnumerable<DataRow> getData(bool firstRowIsColumnNames = true) {
-        IExcelDataReader reader = getExcelReader();
-        reader.IsFirstRowAsColumnNames = firstRowIsColumnNames;
-        var workSheet = reader.AsDataSet().Tables[0];
-        var rows = from DataRow a in workSheet.Rows select a;
-        return rows;
+
+        public IEnumerable<string> ReadQDBFlagData() {
+            List<string> s = new List<string>();
+            // We return the interface, so that         
+            var excelData = new ExcelData(_path);
+            var lotNos = excelData.getData();
+            foreach (var row in lotNos) {
+                string temValue = row[0].ToString();
+                if (temValue.Trim().Length > 0 && temValue.Trim().Length == 9) {
+                    if (row[2].ToString().ToUpper() != "YES" && row[2].ToString().ToUpper() != "NO") {
+                        throw new Exception("Invalid data in the file");
+                    } else {
+                        s.Add(row[0].ToString() + "~" + row[1] + "~" + row[2]);
+                    }
+                }
+
+            }
+            return s;
+        }
+
+        public IEnumerable<DataRow> getData(bool firstRowIsColumnNames = true) {
+            var reader = this.getExcelReader();
+            reader.IsFirstRowAsColumnNames = firstRowIsColumnNames;
+            var workSheet = reader.AsDataSet().Tables[0];
+            var rows = from DataRow a in workSheet.Rows select a;
+            return rows;
+
+        }
+
     }
+}
 
 #endif
-
-}

Fab2ApprovalSystem/Misc/LotDispositionHelper.cs

@@ -117,8 +117,8 @@ public class LotDispositionHelper {
     public static void AttachSave(AppSettings appSettings, LotDispositionDMO lotDispositionDMO, int issueID, int userId, string fullFileName, Stream stream) {
         // Some browsers send file names with full path.
         // We are only interested in the file name.
-        string fileName = Path.GetFileName(fullFileName);
-        string physicalPath = Path.Combine(appSettings.AttachmentFolder + "LotDisposition", fileName);
+        var fileName = Path.GetFileName(fullFileName);
+        var physicalPath = Path.Combine(appSettings.AttachmentFolder + "LotDisposition", fileName);
 
         using (FileStream fileStream = new(physicalPath, FileMode.Create, FileAccess.Write)) {
             stream.CopyTo(fileStream);
@@ -134,7 +134,7 @@ public class LotDispositionHelper {
     public static string ExcelLotOpen(LotDispositionDMO lotDispositionDMO, int issueID, string userIdentityName, string lotTempPipeLine, string fullFileName, Stream stream) {
         string physicalPath;
 
-        string fileExtension = Path.GetExtension(fullFileName);
+        var fileExtension = Path.GetExtension(fullFileName);
         string fName = userIdentityName + DateTime.Now.Hour.ToString() + DateTime.Now.Minute.ToString() + DateTime.Now.Second.ToString();
 
         physicalPath = Path.Combine(lotTempPipeLine, fName + "." + fileExtension);
@@ -142,18 +142,20 @@ public class LotDispositionHelper {
             stream.CopyTo(fileStream);
         }
 
-        ExcelData x = new (physicalPath);
-        IEnumerable<ExcelData.ExcelLotInfo> lotNumbers = x.ReadData();
+#if !NET8
+        ExcelData x = new ExcelData(physicalPath);
+        var lotNumbers = x.ReadData();
 
-        foreach (ExcelData.ExcelLotInfo lotInfo in lotNumbers) {
-            Lot l = new();
-            l.LotNumber = lotInfo.LotNo ?? string.Empty;
+        foreach (var lotInfo in lotNumbers) {
+            Lot l = new Lot();
+            l.LotNumber = lotInfo.LotNo;
             l.IssueID = issueID;
             if (l.LotStatusOptionID == 0)
                 l.LotStatusOption.LotStatusOptionID = (int)GlobalVars.LotStatusOption.Release;
 
             lotDispositionDMO.InsertLot(l, true);
         }
+#endif
 
         FileInfo f = new(physicalPath);
         if (f.Exists)

Fab2ApprovalSystem/Misc/MRBHelper.cs

@@ -46,12 +46,14 @@ public class MRBHelper {
             stream.CopyTo(fileStream);
         }
 
-        ExcelData x = new(physicalPath);
+#if !NET8
+        ExcelData x = new ExcelData(physicalPath);
         lotDataList = x.ReadQDBFlagData();
 
         foreach (string lotData in lotDataList) {
             mrbDMO.InsertMRB_QDB_HoldFlag(guid, lotData, operation);
         }
+#endif
 
         FileInfo f = new(physicalPath);
         if (f.Exists)
@@ -168,12 +170,14 @@ public class MRBHelper {
             stream.CopyTo(fileStream);
         }
 
-        ExcelData x = new(physicalPath);
+#if !NET8
+        ExcelData x = new ExcelData(physicalPath);
         lotDataList = x.ReadQDBFlagData();
 
         foreach (string lotData in lotDataList) {
             mrbDMO.InsertMRB_QDB_HoldFlag(guid, lotData, operation);
         }
+#endif
 
         FileInfo f = new(physicalPath);
         if (f.Exists)
@@ -242,16 +246,17 @@ public class MRBHelper {
         string fName = userIdentityName + DateTime.Now.Hour.ToString() + DateTime.Now.Minute.ToString() + DateTime.Now.Second.ToString();
         string physicalPath = Path.Combine(lotTempPipeLine, fName + "." + fileExtension);
 
+#if !NET8
         IEnumerable<ExcelData.ExcelLotInfo> lotNumbers;
 
         try {
-            using (FileStream fileStream = new(physicalPath, FileMode.Create, FileAccess.Write)) {
+            using (var fileStream = new FileStream(physicalPath, FileMode.Create, FileAccess.Write)) {
                 stream.CopyTo(fileStream);
             }
-            ExcelData x = new(physicalPath);
+            ExcelData x = new ExcelData(physicalPath);
             lotNumbers = x.ReadData();
         } catch (Exception ex) {
-            throw new Exception(string.Format("Invalid file format for {0}: {1}", fileName, ex.Message));
+            throw new Exception(String.Format("Invalid file format for {0}: {1}", fileName, ex.Message));
         }
 
         // Get Tool, Issue Start and End Date
@@ -261,7 +266,7 @@ public class MRBHelper {
         foreach (var lotInfo in lotNumbers) {
             if (lotInfo.LotDispo.Length == 1) {
                 if (dispos.Count(d => d.DispositionType.Trim().ToUpper() == lotInfo.LotDispo.Trim().ToUpper()) == 0) {
-                    throw new Exception(string.Format("Invalid lot disposition {0} for lot no {1}",
+                    throw new Exception(String.Format("Invalid lot disposition {0} for lot no {1}",
                         lotInfo.LotDispo, lotInfo.LotNo));
                 }
             }
@@ -271,8 +276,8 @@ public class MRBHelper {
         if (!mrbInfo.ToolCSV.ToUpper().Equals("NA")) {
             foreach (var lotInfo in lotNumbers) {
                 bool existingLotUpdated;
-                Lot l = new();
-                l.LotNumber = lotInfo.LotNo ?? string.Empty;
+                Lot l = new Lot();
+                l.LotNumber = lotInfo.LotNo;
                 if (lotInfo.LotDispo.Length == 1) {
                     l.DispoType = lotInfo.LotDispo[0];
                 }
@@ -293,8 +298,8 @@ public class MRBHelper {
             // Only find the child Splits when a Tool or a list of Tools is provided
             foreach (var lotInfo in lotNumbers) {
                 bool existingLotUpdated;
-                Lot l = new();
-                l.LotNumber = lotInfo.LotNo ?? string.Empty;
+                Lot l = new Lot();
+                l.LotNumber = lotInfo.LotNo;
                 if (lotInfo.LotDispo.Length == 1) {
                     l.DispoType = lotInfo.LotDispo[0];
                 }
@@ -303,6 +308,7 @@ public class MRBHelper {
                 mrbDMO.InsertLot(l, true, out existingLotUpdated);
             }
         }
+#endif
 
         FileInfo f = new(physicalPath);
         if (f.Exists)

Fab2ApprovalSystem/PdfGenerator/FakeView.cs

@@ -0,0 +1,28 @@
+#if !NET8
+
+// --------------------------------------------------------------------------------------------------------------------
+// <copyright file="FakeView.cs" company="SemanticArchitecture">
+//   http://www.SemanticArchitecture.net pkalkie@gmail.com
+// </copyright>
+// <summary>
+//   Defines the FakeView type.
+// </summary>
+// --------------------------------------------------------------------------------------------------------------------
+
+namespace Fab2ApprovalSystem.PdfGenerator {
+    using System;
+    using System.IO;
+    using System.Web.Mvc;
+
+    public class FakeView : IView {
+        #region IView Members
+
+        public void Render(ViewContext viewContext, TextWriter writer) {
+            throw new NotImplementedException();
+        }
+
+        #endregion
+    }
+}
+
+#endif

Fab2ApprovalSystem/PdfGenerator/HtmlViewRenderer.cs

@@ -0,0 +1,49 @@
+#if !NET8
+
+// --------------------------------------------------------------------------------------------------------------------
+// <copyright file="HtmlViewRenderer.cs" company="SemanticArchitecture">
+//   http://www.SemanticArchitecture.net pkalkie@gmail.com
+// </copyright>
+// <summary>
+//   This class is responsible for rendering a HTML view to a string.
+// </summary>
+// --------------------------------------------------------------------------------------------------------------------
+
+namespace Fab2ApprovalSystem.PdfGenerator {
+    using System.IO;
+    using System.Text;
+    using System.Web;
+    using System.Web.Mvc;
+    using System.Web.Mvc.Html;
+
+    /// <summary>
+    /// This class is responsible for rendering a HTML view into a string. 
+    /// </summary>
+    public class HtmlViewRenderer {
+        public string RenderViewToString(Controller controller, string viewName, object viewData) {
+            var renderedView = new StringBuilder();
+            using (var responseWriter = new StringWriter(renderedView)) {
+                var fakeResponse = new HttpResponse(responseWriter);
+                var fakeContext = new HttpContext(HttpContext.Current.Request, fakeResponse);
+                var fakeControllerContext = new ControllerContext(new HttpContextWrapper(fakeContext), controller.ControllerContext.RouteData, controller.ControllerContext.Controller);
+
+                var oldContext = HttpContext.Current;
+                HttpContext.Current = fakeContext;
+
+                using (var viewPage = new ViewPage()) {
+                    var html = new HtmlHelper(CreateViewContext(responseWriter, fakeControllerContext), viewPage);
+                    html.RenderPartial(viewName, viewData);
+                    HttpContext.Current = oldContext;
+                }
+            }
+
+            return renderedView.ToString();
+        }
+
+        private static ViewContext CreateViewContext(TextWriter responseWriter, ControllerContext fakeControllerContext) {
+            return new ViewContext(fakeControllerContext, new FakeView(), new ViewDataDictionary(), new TempDataDictionary(), responseWriter);
+        }
+    }
+}
+
+#endif

Fab2ApprovalSystem/PdfGenerator/PdfViewController.cs

@@ -0,0 +1,54 @@
+#if !NET8
+
+// --------------------------------------------------------------------------------------------------------------------
+// <copyright file="PdfViewController.cs" company="SemanticArchitecture">
+//   http://www.SemanticArchitecture.net pkalkie@gmail.com
+// </copyright>
+// <summary>
+//   Extends the controller with functionality for rendering PDF views
+// </summary>
+// --------------------------------------------------------------------------------------------------------------------
+
+namespace Fab2ApprovalSystem.PdfGenerator {
+    using System.Web.Mvc;
+    using System.IO;
+
+    /// <summary>
+    /// Extends the controller with functionality for rendering PDF views
+    /// </summary>
+    public class PdfViewController : Controller {
+        private readonly HtmlViewRenderer htmlViewRenderer;
+        private readonly StandardPdfRenderer standardPdfRenderer;
+
+        public PdfViewController() {
+            this.htmlViewRenderer = new HtmlViewRenderer();
+            this.standardPdfRenderer = new StandardPdfRenderer();
+        }
+
+        protected ActionResult ViewPdf(string pageTitle, string viewName, object model) {
+            // Render the view html to a string.
+            string htmlText = this.htmlViewRenderer.RenderViewToString(this, viewName, model);
+
+            // Let the html be rendered into a PDF document through iTextSharp.
+            byte[] buffer = standardPdfRenderer.Render(htmlText, pageTitle);
+
+            // Return the PDF as a binary stream to the client.
+            return new BinaryContentResult(buffer, "application/pdf");
+        }
+
+        protected void SavePdf(string fileName, string viewName, object model) {
+            // Render the view html to a string.
+            string htmlText = this.htmlViewRenderer.RenderViewToString(this, viewName, model);
+
+            // Let the html be rendered into a PDF document through iTextSharp.
+            byte[] buffer = standardPdfRenderer.Render(htmlText, "");
+
+            using (FileStream fs = new FileStream(fileName, FileMode.Create)) {
+                fs.Write(buffer, 0, buffer.Length);
+            }
+
+        }
+    }
+}
+
+#endif

Fab2ApprovalSystem/PdfGenerator/StandardPdfRenderer.cs

@@ -1,38 +1,27 @@
+#if !NET8
 using iTextSharp.text;
 using iTextSharp.text.html.simpleparser;
 using iTextSharp.text.pdf;
+#endif
 
 using System.IO;
 
 namespace Fab2ApprovalSystem.PdfGenerator;
 
+/// <summary>
+/// This class is responsible for rendering a html text string to a PDF document using the html renderer of iTextSharp.
+/// </summary>
 public class StandardPdfRenderer {
-
     private const int HorizontalMargin = 40;
     private const int VerticalMargin = 40;
 
-    public static byte[] GetPortableDocumentFormatBytes(string pageTitle, string htmlText) {
-        byte[] results;
-        using (MemoryStream memoryStream = GetPortableDocumentFormat(pageTitle, htmlText)) {
-            results = new byte[memoryStream.Position];
-            memoryStream.Position = 0;
-            memoryStream.Read(results, 0, results.Length);
-        }
-        return results;
-    }
+    public byte[] Render(string htmlText, string pageTitle) {
+        byte[] renderedBuffer;
 
-    public static void WritePortableDocumentFormatToFile(string pageTitle, string htmlText, string path) {
-        using (MemoryStream memoryStream = GetPortableDocumentFormat(pageTitle, htmlText)) {
-            using (FileStream fileStream = new(path, FileMode.Create)) {
-                memoryStream.CopyTo(fileStream);
-            }
-        }
-    }
-
-    public static MemoryStream GetPortableDocumentFormat(string pageTitle, string htmlText) {
-        MemoryStream result = new();
-        using (Document pdfDocument = new Document(PageSize.A4, HorizontalMargin, HorizontalMargin, VerticalMargin, VerticalMargin)) {
-            using (PdfWriter pdfWriter = PdfWriter.GetInstance(pdfDocument, result)) {
+        using (MemoryStream outputMemoryStream = new()) {
+#if !NET8
+            using (Document pdfDocument = new Document(PageSize.A4, HorizontalMargin, HorizontalMargin, VerticalMargin, VerticalMargin)) {
+                PdfWriter pdfWriter = PdfWriter.GetInstance(pdfDocument, outputMemoryStream);
                 pdfWriter.CloseStream = false;
                 pdfWriter.PageEvent = new PrintHeaderFooter { Title = pageTitle };
                 pdfDocument.Open();
@@ -41,9 +30,15 @@ public class StandardPdfRenderer {
                         htmlWorker.Parse(htmlViewReader);
                     }
                 }
-            }
-        }
-        return result;
-    }
 
+            }
+#endif
+
+            renderedBuffer = new byte[outputMemoryStream.Position];
+            outputMemoryStream.Position = 0;
+            outputMemoryStream.Read(renderedBuffer, 0, renderedBuffer.Length);
+        }
+
+        return renderedBuffer;
+    }
 }

Fab2ApprovalSystem/ViewModels/PdfViewModel.cs

@@ -30,7 +30,7 @@ public class ECNPdf {
     public string AffectedDepartments { get; set; }
     public string AffectedAreas { get; set; }
     public string AffectedTechnologies { get; set; }
-    public List<int> TrainingByIDs { get; set; }
+    public string TrainingBy { get; set; }
     public string AcknowledgementBy { get; set; }
     public bool IsECN { get; set; }
     public bool IsTECN { get; set; }
@@ -79,7 +79,6 @@ public class ECNPdf {
     public int? ConvertedToNumber { get; set; }
     public int? ConvertedFromNumber { get; set; }
     public int WorkFlowNumber { get; set; }
-    public int? CategoryID { get; set; }
     public bool FIChangeRequired { get; set; }
     public string NumberOfLotsAffected { get; set; }
     public string RecipeChange { get; set; }
@@ -88,7 +87,6 @@ public class ECNPdf {
     public ECNPdf() {
         Approvalog = new List<ECNApprovalLog>();
         Attachments = new List<string>();
-        TrainingByIDs = new List<int>();
     }
 
 }

Fab2ApprovalSystem/Views/ECN/Acknowledge.cshtml

@@ -210,10 +210,10 @@
                             @Html.TextBoxFor(model => model.Title, new { id = "txtTitle", @class = "k-textbox", style = "width:100%", disabled = "disabled" })
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            * (DO NOT USE Rev I, O, S, X, and Z)
+                            *(DO NOT USE Rev I, O, Q, S, X, and Z)
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            Revision Y is followed by AA-AY, AY is followed by BA-BY etc. YY is followed by AAA
+                            Revision Y is followed by AA. YY is followed by AAA
                         </div>
                     </div>
                     <div class="form-group">

Fab2ApprovalSystem/Views/ECN/ECNPdf.cshtml

@@ -1,506 +1,497 @@
 @model Fab2ApprovalSystem.ViewModels.ECNPdf
-@{
-    Layout = null;
-}
-
-<!DOCTYPE html>
-
-<head>
-    <meta charset="utf-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <meta http-equiv="X-UA-Compatible" content="IE=edge">
-    <title></title>
-</head>
-
-<body class="navbar-inner">
-    <table style="width:100%;">
-        <tr>
-            <td>
-                <table cellpadding="3" cellspacing="3" border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td colspan="2">
-                            @Model.Title
-                        </td>
-
-                    </tr>
-                    <tr>
-                        <td>
-                            <table border="0">
-                                @if (@Model.ConvertedFromNumber != null)
-                                {
-                                    <tr>
-                                        <td>
-                                            ECN# @Model.ECNNumber (Converted from TECN#:@Model.ConvertedFromNumber)
-                                        </td>
-                                    </tr>
-                                }
-                                @if (@Model.ConvertedToNumber != null)
-                                {
-                                    <tr>
-                                        <td>
-                                            ECN# @Model.ECNNumber (Converted to ECN#:@Model.ConvertedToNumber)
-                                        </td>
-                                    </tr>
-                                }
-                                @if (@Model.ConvertedFromNumber == null && @Model.ConvertedToNumber == null)
-                                {
-                                    <tr>
-                                        <td>
-                                            ECN# @Model.ECNNumber
-                                        </td>
-                                    </tr>
-                                }
+<table style="width:100%;">
+    <tr>
+        <td>
+            <table cellpadding="3" cellspacing="3" border="1">
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td colspan="2">
+                        @Model.Title
+                    </td>
 
+                </tr>
+                <tr>
+                    <td>
+                        <table border="0">
+                            @if (@Model.ConvertedFromNumber != null)
+                            {
                                 <tr>
                                     <td>
-                                        <font size="2">
-                                            Submit Date#:&nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @Model.SubmitedDate
-                                        </font>
+                                        ECN# @Model.ECNNumber (Converted from TECN#:@Model.ConvertedFromNumber)
                                     </td>
                                 </tr>
+                            }
+                            @if (@Model.ConvertedToNumber != null)
+                            {
                                 <tr>
                                     <td>
-                                        <font size="2">
-                                            Originator Name:&nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @Model.OriginatorName
-                                        </font>
+                                        ECN# @Model.ECNNumber (Converted to ECN#:@Model.ConvertedToNumber)
                                     </td>
                                 </tr>
-
-                            </table>
-                        </td>
-                        <td>
-                            <table border="0">
-                                @*<tr>
+                            }
+                            @if (@Model.ConvertedFromNumber == null && @Model.ConvertedToNumber == null)
+                            { 
+                            <tr>
                                 <td>
-                                <font size="2">
-                                Affected Department:&nbsp;
-                                </font>
-                                <font size="1">
-                                @Model.AffectedDepartments
-                                </font>
+                                    ECN# @Model.ECNNumber
+                                </td>
+                            </tr>
+                            }
+
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Submit Date#:&nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @Model.SubmitedDate
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Originator Name:&nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @Model.OriginatorName
+                                    </font>
+                                </td>
+                            </tr>
+                           
+                        </table>
+                    </td>
+                    <td>
+                        <table border="0">
+                            @*<tr>
+                                <td>
+                                    <font size="2">
+                                        Affected Department:&nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @Model.AffectedDepartments
+                                    </font>
                                 </td>
 
-                                </tr>*@
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Affected Area:&nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @Model.AffectedModules
-                                            @*@Model.AffectedAreas*@
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Category:&nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @(ViewBag.Category)
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            ITAR/EC:&nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @(Model.IsDocEC ? "Yes" : "No")
-                                        </font>
-                                    </td>
-                                </tr>
-                            </table>
-                        </td>
-                    </tr>
-                </table>
-            </td>
-        </tr>
-
-        <tr>
-            <td>
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td> ECN - TECN Details</td>
-                    </tr>
-                    <tr>
-                        <td>
-                            <table border="0">
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            ECN: &nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @(Model.IsECN ? "Yes" : "No")
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            TECN:&nbsp;
-                                            @(Model.IsTECN ? "Yes" : "No")
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            Emergency: &nbsp;
-                                        </font>
-                                        <font size="1">
-                                            @(Model.IsEmergencyTECN ? "Yes" : "No")
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Expiration Date:
-                                        </font>
-                                        <font size="1">
-
-                                            @Convert.ToString(string.Format("{0:MM/dd/yyyy}", Model.ExpirationDate))
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            Extension Date:
-                                        </font>
-                                        <font size="1">
-
-                                            @Convert.ToString(string.Format("{0:MM/dd/yyyy}", @Model.ExtensionDate))
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            Cancellation Date:
-                                        </font>
-                                        <font size="1">
-                                            @Model.CancellationApprovalDate
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            # lots affected:
-                                        </font>
-                                        <font size="1">
-                                            @Model.NumberOfLotsAffected
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            Recipe Or/And Flow Change:
-                                        </font>
-                                        <font size="1">
-                                            @Model.RecipeChange
-                                        </font>
-                                    </td>
-                                    <td>
-                                        <font size="2">
-                                            Affected product families:
-                                        </font>
-                                        <font size="1">
-                                            @Model.AffectedProductFamilies
-                                        </font>
-                                    </td>
-                                </tr>
-                            </table>
-                        </td>
-                    </tr>
-                    <tr>
-                        <td>
-                            <table border="0">
-                                <tr>
-                                    <td colspan="3" style="background-color:ActiveCaption">
-                                        <font size="2">
-                                            Affected Documents:&nbsp;
-                                        </font>
-
-                                    </td>
-                                </tr>
-                                <tr>
-                                    @foreach (string attachmentName in Model.Attachments)
-                                    {
-                                        <td colspan="3">
-                                            <font size="1">
-                                                @attachmentName
-                                            </font>
-                                        </td>
-                                    }
-                                </tr>
-                            </table>
-                        </td>
-                    </tr>
-
-                </table>
-            </td>
-        </tr>
-
-        <tr>
-            <td>
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td> Description of Change</td>
-                        <td> Reason for Change</td>
-                    </tr>
-                    <tr>
-                        <td>
-                            <font size="1">
-                                @Html.Raw(@Model.DescriptionOfChange)
-                            </font>
-                        </td>
-                        <td>
-                            <font size="1">
-                                @Html.Raw(@Model.ReasonForChange)
-                            </font>
-                        </td>
-                    </tr>
-                </table>
-            </td>
-        </tr>
-        <tr>
-            <td>
-
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td colspan="5">Systems</td>
-                    </tr>
-                    <tr style="display:block;">
-                        <td>
-                            <font size="2">
-                                PCRB:
-                            </font>
-                            <font size="1">
-                                @(Model.PCRBRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                PCRB#: &nbsp;
-                                @Model.PCRBNumber
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                Metrology Change: &nbsp;&nbsp;
-                            </font>
-                            <font size="1">
-                                @(Model.MetrologyChangeRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                SPC Change:
-                            </font>
-                            <font size="1">
-                                @(Model.SPCChangeRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                FI Change:
-                            </font>
-                            <font size="1">
-                                @(Model.FIChangeRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                OI Change:
-                            </font>
-                            <font size="1">
-                                @(Model.SPNChangeRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                    </tr>
-                </table>
-
-            </td>
-        </tr>
-        <tr>
-            <td>
-
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td colspan="1">Training Notification</td>
-                    </tr>
-                    <tr style="display:block;">
-                        <td>
-                            <font size="2">
-                                Training:
-                            </font>
-                            <font size="1">
-                                @(Model.TrainingRequired ? "Yes" : "No")
-                            </font>
-                        </td>
-                        <td>
-                            <font size="2">
-                                Training Notification to: &nbsp;
-                                @(ViewBag.TrainingNotificationTo)
-                            </font>
-                        </td>
-                    </tr>
-                </table>
-
-            </td>
-        </tr>
-        <tr>
-            <td>
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#000000">
-                        <td colspan="4"> Change Impact</td>
-                    </tr>
-                    <tr>
-                        <td>
-                            <table border="0">
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Impact On ESH:
-                                        </font>
-                                        <font size="1">
-                                            @if (Model.ImpactOnEnvironment == 1)
-                                            {
-                                                @: Positive
-                                            }
-                                            else if (Model.ImpactOnEnvironment == 2)
-                                            {
-                                                @: Negative
-                                            }
-                                            else
-                                            {
-                                                @: N/A
-                                            }
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="1">
-                                            @Html.Raw(@Model.ImpactOnEnvironmentDescription)
-                                        </font>
-                                    </td>
-                                </tr>
-                            </table>
-                        </td>
-                        <td>
-                            <table border="0">
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Impact On Capacity:
-                                        </font>
-                                        <font size="1">
-                                            @(Model.ImpactOnCapacity == 1 ? "Yes" : "No")
-                                        </font>
-                                    </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="1">
-                                            @Html.Raw(@Model.ImpactOnCapacityDescription)
-                                        </font>
-                                    </td>
-                                </tr>
-                            </table>
-                        </td>
-                        <td>
-                            @*<table border="0">
+                            </tr>*@
                             <tr>
-                            <td>
-                            <font size="2">
-                            RH(ITAR):
-                            </font>
-                            <font size="1">
-                            @(Model.IsRH ? "Yes" : "No")
-                            </font>
-                            </td>
+                                <td>
+                                    <font size="2">
+                                        Affected Area:&nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @Model.AffectedModules
+                                        @*@Model.AffectedAreas*@
+                                    </font>
+                                </td>
                             </tr>
-                            </table>*@
-                        </td>
-                        <td>
-                            <table border="0">
-                                <tr>
-                                    <td>
-                                        <font size="2">
-                                            Material Consumption Change:
-                                        </font>
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        ITAR/EC:&nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @(Model.IsDocEC ? "Yes" : "No")
+                                    </font>
+                                </td>
+                            </tr>
+                        </table>
+                    </td>
+                </tr>
+                
+            </table>
+        </td>
+    </tr>
+       
+    <tr>
+        <td>
+            <table border="1">
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td> ECN - TECN Details</td>
+                </tr>
+                <tr>
+                    <td>
+                        <table border="0">
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        ECN: &nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @(Model.IsECN ? "Yes" : "No")
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        TECN:&nbsp;
+                                        @(Model.IsTECN ? "Yes" : "No")
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        Emergency: &nbsp;
+                                    </font>
+                                    <font size="1">
+                                        @(Model.IsEmergencyTECN ? "Yes" : "No")
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Expiration Date:
+                                    </font>
+                                    <font size="1">
+                                        
+                                        @Convert.ToString(string.Format("{0:MM/dd/yyyy}", Model.ExpirationDate))
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        Extension Date:
+                                    </font>
+                                    <font size="1">
+                                        
+                                        @Convert.ToString(string.Format("{0:MM/dd/yyyy}", @Model.ExtensionDate))
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        Cancellation Date:
+                                    </font>
+                                    <font size="1">
+                                        @Model.CancellationApprovalDate
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        # lots affected:
+                                    </font>
+                                    <font size="1">
+                                        @Model.NumberOfLotsAffected
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        Recipe Or/And Flow Change:
+                                    </font>
+                                    <font size="1">
+                                        @Model.RecipeChange
+                                    </font>
+                                </td>
+                                <td>
+                                    <font size="2">
+                                        Affected product families:
+                                    </font>
+                                    <font size="1">
+                                        @Model.AffectedProductFamilies
+                                    </font>
+                                </td>
+                            </tr>
+                        </table>
+                    </td>
+                </tr>                
+                <tr>
+                    <td>
+                        <table border="0">
+                            <tr>
+                                <td colspan="3" style="background-color:ActiveCaption">
+                                    <font size="2">
+                                        Affected Documents:&nbsp;
+                                    </font>
+
+                                </td>
+                            </tr>
+                            <tr>
+                                @foreach (string attachmentName in Model.Attachments)
+                                {
+                                    <td colspan="3">
                                         <font size="1">
-                                            @(Model.MaterialConsumptionChangeRequired == 1 ? "Yes" : "No")
+                                            @attachmentName
                                         </font>
                                     </td>
-                                </tr>
-                                <tr>
-                                    <td>
-                                        <font size="1">
-                                            @Html.Raw(@Model.MaterialConsumptionChangeDescription)
+                                }
+                            </tr>
+                        </table>
+                    </td>
+                </tr>
+               
+                
+            </table>
+        </td>
+    </tr>
+
+    <tr>
+        <td>
+            <table border="1" >
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td> Description of Change</td>
+                    <td> Reason for Change</td>
+                </tr>
+                <tr>
+                    <td>
+                        <font size="1">
+                            @Html.Raw(@Model.DescriptionOfChange)
+                        </font>
+                    </td>
+                    <td>
+                        <font size="1">
+                            @Html.Raw(@Model.ReasonForChange)
+                        </font>
+                    </td>
+                </tr>
+            </table>
+        </td>
+    </tr>
+    @*<tr>
+        <td>
+
+            <table border="1">
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td colspan="3">Training Notification</td>                    
+                </tr>
+                <tr>
+                    <td>
+                       <table border="0">
+                           <tr>
+                               <td>
+                                   <font size="1">
+                                       Training:
+                                       @(Model.TrainingRequired ? "Yes" : "No")
+                                   </font>
+                              
+                                   <font size="1">
+                                       @(Model.TrainingBy.Length > 0 ? "(" + Model.TrainingBy + ")" : Model.TrainingBy)                                       
+                                   </font>
+                               </td>
+                           </tr>
+                       </table>
+                    </td>
+                </tr>
+            </table>
+
+        </td>
+    </tr>*@
+    <tr>
+        <td>
+
+            <table border="1">
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td colspan="5">Systems</td>
+                </tr>
+                <tr style="display:block;">
+                    <td>
+                        <font size="2">
+                            PCRB:
+                        </font>
+                        <font size="1">
+                            @(Model.PCRBRequired ? "Yes": "No")
+                        </font>
+                    </td>
+                    <td>
+                        <font size="2">
+                            PCRB#: &nbsp;
+                            @Model.PCRBNumber
+                        </font>
+                    </td>
+                    <td>
+                        <font size="2">
+                            Metrology Change: &nbsp;&nbsp;
+                        </font>
+                        <font size="1">
+                            @(Model.MetrologyChangeRequired ? "Yes": "No")
+                        </font>
+                    </td>
+                    <td>
+                        <font size="2">
+                            SPC Change:
+                        </font>
+                        <font size="1">
+                            @(Model.SPCChangeRequired ? "Yes": "No")
+                        </font>
+                    </td>
+                    <td>
+                        <font size="2">
+                            FI Change:
+                        </font>
+                        <font size="1">
+                           @(Model.FIChangeRequired ? "Yes" : "No")
+                        </font>
+                    </td>
+                    <td>
+                        <font size="2">
+                            OI Change:
+                        </font>
+                        <font size="1">
+                            @(Model.SPNChangeRequired ? "Yes" : "No")
+                        </font>
+                    </td>
+                    <td></td>
+                </tr>
+                
+            </table>
+
+
+        </td>
+    </tr>
+    <tr>
+        <td>
+            <table border="1">
+                <tr bgcolor="#c4baba" color="#000000">
+                    <td colspan="4"> Change Impact</td>
+                </tr>
+                <tr>
+                    <td>
+                        <table border="0">
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Impact On ESH:
+                                    </font>
+                                    <font size="1">
+                                        @if (Model.ImpactOnEnvironment == 1)
+                                        {
+                                            @: Positive
+                                        }
+                                        else if (Model.ImpactOnEnvironment == 2)
+                                        { 
+                                            @: Negative
+                                        }
+                                        else
+                                        { 
+                                            @: N/A
+                                        }
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="1">
+                                        @Html.Raw(@Model.ImpactOnEnvironmentDescription)
                                         </font>
-                                    </td>
-                                </tr>
-                            </table>
-                        </td>
-                    </tr>
-                </table>
-            </td>
-        </tr>
-        <tr>
-            <td>
-                <table border="1">
-                    <tr bgcolor="#c4baba" color="#ffffff">
-                        <td>
-                            Full Name
-                        </td>
-                        <td>
-                            Role
-                        </td>
-                        <td>
-                            Approved/Denied
-                        </td>
-                        <td>
-                            Date Time
+                                </td>
+                            </tr>
+                        </table>
+                    </td>
+                    <td>
+                        <table border="0">
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Impact On Capacity:
+                                    </font>
+                                    <font size="1">
+                                        @(Model.ImpactOnCapacity == 1 ? "Yes" : "No")
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="1">
+                                        @Html.Raw(@Model.ImpactOnCapacityDescription)
+                                        </font>
+                                </td>
+                            </tr>
+                        </table>
+                    </td>
+                    <td>
+                        @*<table border="0">
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        RH(ITAR):
+                                    </font>
+                                    <font size="1">
+                                        @(Model.IsRH ? "Yes" : "No")
+                                    </font>
+                                </td>
+                            </tr>
+                        </table>*@
+                    </td>
+                    <td>
+                        <table border="0">
+                            <tr>
+                                <td>
+                                    <font size="2">
+                                        Material Consumption Change:
+                                    </font>
+                                    <font size="1">
+                                        @(Model.MaterialConsumptionChangeRequired == 1 ? "Yes" : "No")
+                                    </font>
+                                </td>
+                            </tr>
+                            <tr>
+                                <td>
+                                    <font size="1">
+                                        @Html.Raw(@Model.MaterialConsumptionChangeDescription)
+                                    </font>
+                                </td>
+                            </tr>
+                        </table>
+                    </td>
+                </tr>
+            </table>
+        </td>
+    </tr>
+    <tr>
+        <td>
+            <table border="1">
+                <tr bgcolor="#c4baba" color="#ffffff">
+                    <td>
+                        Full Name
+                    </td>
+                    <td>
+                        Role
+                    </td>
+                    <td>
+                        Approved/Denied
+                    </td>
+                    <td>
+                        Date Time
+                    </td>
+                </tr>
+
+                @foreach (Fab2ApprovalSystem.ViewModels.ECNApprovalLog ecnApp in Model.Approvalog)
+                {
+                    <tr>
+                        <td>
+                            <font size="1">
+                                @ecnApp.FullName
+                            </font>
+                        </td>
+                        <td>
+                            <font size="1">
+                                @ecnApp.SubRole
+                            </font>
+                        </td>
+                        <td>
+                            <font size="1">
+                                @ecnApp.Operation
+                            </font>
+                        </td>
+
+                        <td>
+                            <font size="1">
+                                @ecnApp.OperationTime
+                            </font>
                         </td>
                     </tr>
+                }
+
+
+            </table>
+        </td>
+    </tr>
+    
+
+</table> <!--main table -->
+
+
+
+
     
-                    @foreach (Fab2ApprovalSystem.ViewModels.ECNApprovalLog ecnApp in Model.Approvalog)
-                    {
-                        <tr>
-                            <td>
-                                <font size="1">
-                                    @ecnApp.FullName
-                                </font>
-                            </td>
-                            <td>
-                                <font size="1">
-                                    @ecnApp.SubRole
-                                </font>
-                            </td>
-                            <td>
-                                <font size="1">
-                                    @ecnApp.Operation
-                                </font>
-                            </td>
 
-                            <td>
-                                <font size="1">
-                                    @ecnApp.OperationTime
-                                </font>
-                            </td>
-                        </tr>
-                    }
    
-                </table>
-            </td>
-        </tr>
     
-    </table>
-</body>
     
-</html>

Fab2ApprovalSystem/Views/ECN/Edit.cshtml

@@ -154,10 +154,10 @@
                             @Html.TextBoxFor(model => model.Title, new { id = "txtTitle", @class = "k-textbox", style = "width:100%" })
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            * (DO NOT USE Rev I, O, S, X, and Z)
+                            *(DO NOT USE Rev I, O, Q, S, X, and Z)
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            Revision Y is followed by AA-AY, AY is followed by BA-BY etc. YY is followed by AAA
+                            Revision Y is followed by AA. YY is followed by AAA
                         </div>
                     </div>
                     <div class="form-group">

Fab2ApprovalSystem/Views/ECN/EditApproval.cshtml

@@ -260,10 +260,10 @@
                             @Html.TextBoxFor(model => model.Title, new { id = "txtTitle", @class = "k-textbox", style = "width:100%", disabled = "disabled" })
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            * (DO NOT USE Rev I, O, S, X, and Z)
+                            *(DO NOT USE Rev I, O, Q, S, X, and Z)
                         </div>
                         <div class="col-sm-6 col-sm-offset-4" style="color: red;">
-                            Revision Y is followed by AA-AY, AY is followed by BA-BY etc. YY is followed by AAA
+                            Revision Y is followed by AA. YY is followed by AAA
                         </div>
                     </div>
                     <div class="form-group">

Fab2ApprovalSystem/Views/Home/_HomeLayout.cshtml

@@ -90,16 +90,16 @@
                             string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
                             string wasmClientUrl = Environment.GetEnvironmentVariable("FabApprovalWasmClientUrl") ??
                                 "https://localhost:7255";
-                            string mrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=mrb/new";
+                            string mrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=/mrb/new";
                             <li><a href="@mrbUrl">Create MRB</a></li>
-                            string pcrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=pcrb/new";
+                            string pcrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=/pcrb/new";
                             <li><a href="@pcrbUrl">Create PCRB</a></li>
                         } else {
                             string wasmClientUrl = Environment.GetEnvironmentVariable("FabApprovalWasmClientUrl") ??
                                                         "https://localhost:7255";
-                            string mrbUrl = wasmClientUrl + "/redirect?redirectPath=mrb/new";
+                            string mrbUrl = wasmClientUrl + "/redirect?redirectPath=/mrb/new";
                             <li><a href="@mrbUrl">Create MRB</a></li>
-                            string pcrbUrl = wasmClientUrl + "/redirect?redirectPath=pcrb/new";
+                            string pcrbUrl = wasmClientUrl + "/redirect?redirectPath=/pcrb/new";
                             <li><a href="@pcrbUrl">Create PCRB</a></li>
                         }
                         @*<li><a href=@Url.Action("CreateWorkRequest", "LotTraveler")>Create Special Work Request</a></li>*@
@@ -150,9 +150,9 @@
             string encodedRefreshToken = System.Net.WebUtility.UrlEncode(refreshToken);
             string wasmClientUrl = Environment.GetEnvironmentVariable("FabApprovalWasmClientUrl") ??
                 "https://localhost:7255";
-            string mrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=mrb/all";
+            string mrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=/mrb/all";
             menu.Add().Text("MRB").Url(mrbUrl);
-            string pcrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=pcrb/all";
+            string pcrbUrl = wasmClientUrl + "/redirect?jwt=" + encodedJwt + "&refreshToken=" + encodedRefreshToken + "&redirectPath=/pcrb/all";
             menu.Add().Text("PCRB").Url(pcrbUrl);
             //menu.Add().Text("Special Work Requests").Action("SpecialWorkRequestList", "Home");
             //menu.Add().Text("PCRB").Action("ChangeControlList", "Home");

Fab2ApprovalSystem/pipelines-manual.yml

@@ -1,232 +0,0 @@
-trigger:
-  branches:
-    include:
-      - master
-  paths:
-    include:
-      - Fab2ApprovalSystem
-
-variables:
-  coreVersion: "net8.0"
-  targetFrameworkVersion: "v4.8"
-  nugetSource: "https://artifactory.intra.infineon.com/artifactory/api/nuget/ngt-fi-package-main-vir/"
-  msBuild: "C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe"
-
-stages:
-  - stage: Development
-    displayName: Development
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem-Dev
-    variables:
-      ASPNETCORE_ENVIRONMENT: "Development"
-      assemblyTitle: "Fab2ApprovalSystem"
-      configuration: "Debug"
-    jobs:
-      - job: DebugDotnet
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: |
-              mklink /J ".vscode\.UserSecrets" "%AppData%\Microsoft\UserSecrets\f2da5035-aba9-4676-9f8d-d6689f84663d"
-              mklink /J "DMO" "..\Fab2ApprovalSystem\DMO"
-              mklink /J "Jobs" "..\Fab2ApprovalSystem\Jobs"
-              mklink /J "JobSchedules" "..\Fab2ApprovalSystem\JobSchedules"
-              mklink /J "Misc" "..\Fab2ApprovalSystem\Misc"
-              mklink /J "Models" "..\Fab2ApprovalSystem\Models"
-              mklink /J "PdfGenerator" "..\Fab2ApprovalSystem\PdfGenerator"
-              mklink /J "Utilities" "..\Fab2ApprovalSystem\Utilities"
-              mklink /J "ViewModels" "..\Fab2ApprovalSystem\ViewModels"
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Symbolic Link of Type Junction"
-
-          - script: |
-              dotnet user-secrets init
-              dotnet user-secrets set BuildNumber $(Build.BuildId)
-              dotnet user-secrets set "GitCommit" "$(Build.SourceVersion)"
-              dotnet user-secrets list
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Safe storage of app secrets"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Build"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Build"
-
-          - script: dotnet test --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Test"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Clean"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Clean"
-
-      - job: DebugMicrosoftBuildEngine
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: '"$(msBuild)" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=$(NugetSource) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Restore"
-
-          - script: '"$(msBuild)" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Build"
-
-          - script: '"$(msBuild)" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedconfiguration=$(Configuration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath=bin/$(Configuration) /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Package (.zip for msdeploy.exe)"
-
-          - task: CopyFiles@2
-            displayName: 'Copy Files to: D:\'
-            inputs:
-              Contents: "*"
-              SourceFolder: '$(AssemblyTitle)\bin\$(Configuration)\_PublishedWebsites\$(AssemblyTitle)_Package'
-              TargetFolder: 'D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(Build.BuildId)\$(Configuration)-$(AssemblyTitle)-Package'
-              OverWrite: true
-
-          - script: '"$(msBuild)" /target:Clean /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Clean"
-
-  - stage: Release
-    displayName: Release
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem
-    variables:
-      ASPNETCORE_ENVIRONMENT: "Production"
-      assemblyTitle: "Fab2ApprovalSystem"
-      configuration: "Release"
-    jobs:
-      - job: ReleaseDotnet
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: |
-              mklink /J ".vscode\.UserSecrets" "%AppData%\Microsoft\UserSecrets\f2da5035-aba9-4676-9f8d-d6689f84663d"
-              mklink /J "DMO" "..\Fab2ApprovalSystem\DMO"
-              mklink /J "Jobs" "..\Fab2ApprovalSystem\Jobs"
-              mklink /J "JobSchedules" "..\Fab2ApprovalSystem\JobSchedules"
-              mklink /J "Misc" "..\Fab2ApprovalSystem\Misc"
-              mklink /J "Models" "..\Fab2ApprovalSystem\Models"
-              mklink /J "PdfGenerator" "..\Fab2ApprovalSystem\PdfGenerator"
-              mklink /J "Utilities" "..\Fab2ApprovalSystem\Utilities"
-              mklink /J "ViewModels" "..\Fab2ApprovalSystem\ViewModels"
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Symbolic Link of Type Junction"
-
-          - script: |
-              dotnet user-secrets init
-              dotnet user-secrets set BuildNumber $(Build.BuildId)
-              dotnet user-secrets set "GitCommit" "$(Build.SourceVersion)"
-              dotnet user-secrets list
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Safe storage of app secrets"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Build"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Build"
-
-          - script: dotnet test --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Test"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Clean"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Clean"
-
-      - job: ReleaseMicrosoftBuildEngine
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: '"$(msBuild)" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=$(NugetSource) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Restore"
-
-          - script: '"$(msBuild)" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Build"
-
-          - script: '"$(msBuild)" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedconfiguration=$(Configuration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath=bin/$(Configuration) /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Package (.zip for msdeploy.exe)"
-
-          - task: CopyFiles@2
-            displayName: 'Copy Files to: D:\'
-            inputs:
-              Contents: "*"
-              SourceFolder: '$(AssemblyTitle)\bin\$(Configuration)\_PublishedWebsites\$(AssemblyTitle)_Package'
-              TargetFolder: 'D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(Build.BuildId)\$(Configuration)-$(AssemblyTitle)-Package'
-              OverWrite: true
-
-          - script: '"$(msBuild)" /target:Clean /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Clean"
-
-          - script: 'echo $(Build.BuildId)-$(Build.SourceVersion)-bin_x_x_\$(Configuration)\$(CoreVersion)\win-x64\$(Build.Repository.Name).txt'
-            displayName: "Force Fail"
-            enabled: true

Fab2ApprovalSystem/pipelines.yml

@@ -1,239 +0,0 @@
-trigger:
-  branches:
-    include:
-      - master
-  paths:
-    include:
-      - "Fab2ApprovalSystem/*"
-    exclude:
-      - "**/*.yaml"
-      - "**/*.yml"
-      - "SQL/*"
-      - "references/*"
-      - "packages/*"
-      - "Kendo/*"
-
-variables:
-  coreVersion: "net8.0"
-  targetFrameworkVersion: "v4.8"
-  nugetSource: "https://artifactory.intra.infineon.com/artifactory/api/nuget/ngt-fi-package-main-vir/"
-  msBuild: "C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe"
-
-stages:
-  - stage: Development
-    displayName: Development
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem-Dev
-    variables:
-      ASPNETCORE_ENVIRONMENT: "Development"
-      assemblyTitle: "Fab2ApprovalSystem"
-      configuration: "Debug"
-    jobs:
-      - job: DebugDotnet
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: |
-              mklink /J ".vscode\.UserSecrets" "%AppData%\Microsoft\UserSecrets\f2da5035-aba9-4676-9f8d-d6689f84663d"
-              mklink /J "DMO" "..\Fab2ApprovalSystem\DMO"
-              mklink /J "Jobs" "..\Fab2ApprovalSystem\Jobs"
-              mklink /J "JobSchedules" "..\Fab2ApprovalSystem\JobSchedules"
-              mklink /J "Misc" "..\Fab2ApprovalSystem\Misc"
-              mklink /J "Models" "..\Fab2ApprovalSystem\Models"
-              mklink /J "PdfGenerator" "..\Fab2ApprovalSystem\PdfGenerator"
-              mklink /J "Utilities" "..\Fab2ApprovalSystem\Utilities"
-              mklink /J "ViewModels" "..\Fab2ApprovalSystem\ViewModels"
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Symbolic Link of Type Junction"
-
-          - script: |
-              dotnet user-secrets init
-              dotnet user-secrets set BuildNumber $(Build.BuildId)
-              dotnet user-secrets set "GitCommit" "$(Build.SourceVersion)"
-              dotnet user-secrets list
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Safe storage of app secrets"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Build"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Build"
-
-          - script: dotnet test --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Test"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Clean"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Clean"
-
-      - job: DebugMicrosoftBuildEngine
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: '"$(msBuild)" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=$(NugetSource) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Restore"
-
-          - script: '"$(msBuild)" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Build"
-
-          - script: '"$(msBuild)" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedconfiguration=$(Configuration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath=bin/$(Configuration) /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Package (.zip for msdeploy.exe)"
-
-          - task: CopyFiles@2
-            displayName: 'Copy Files to: D:\'
-            inputs:
-              Contents: "*"
-              SourceFolder: '$(AssemblyTitle)\bin\$(Configuration)\_PublishedWebsites\$(AssemblyTitle)_Package'
-              TargetFolder: 'D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(Build.BuildId)\$(Configuration)-$(AssemblyTitle)-Package'
-              OverWrite: true
-
-          - script: '"$(msBuild)" /target:Clean /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Clean"
-
-  - stage: Release
-    displayName: Release
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem
-    variables:
-      ASPNETCORE_ENVIRONMENT: "Production"
-      assemblyTitle: "Fab2ApprovalSystem"
-      configuration: "Release"
-    jobs:
-      - job: ReleaseDotnet
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: |
-              mklink /J ".vscode\.UserSecrets" "%AppData%\Microsoft\UserSecrets\f2da5035-aba9-4676-9f8d-d6689f84663d"
-              mklink /J "DMO" "..\Fab2ApprovalSystem\DMO"
-              mklink /J "Jobs" "..\Fab2ApprovalSystem\Jobs"
-              mklink /J "JobSchedules" "..\Fab2ApprovalSystem\JobSchedules"
-              mklink /J "Misc" "..\Fab2ApprovalSystem\Misc"
-              mklink /J "Models" "..\Fab2ApprovalSystem\Models"
-              mklink /J "PdfGenerator" "..\Fab2ApprovalSystem\PdfGenerator"
-              mklink /J "Utilities" "..\Fab2ApprovalSystem\Utilities"
-              mklink /J "ViewModels" "..\Fab2ApprovalSystem\ViewModels"
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Symbolic Link of Type Junction"
-
-          - script: |
-              dotnet user-secrets init
-              dotnet user-secrets set BuildNumber $(Build.BuildId)
-              dotnet user-secrets set "GitCommit" "$(Build.SourceVersion)"
-              dotnet user-secrets list
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Safe storage of app secrets"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Build"
-
-          - script: dotnet build --configuration $(Configuration) --source $(NugetSource)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Build"
-
-          - script: dotnet test --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Test"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalTests
-            displayName: "Tests - Clean"
-
-          - script: dotnet clean --configuration $(Configuration)
-            workingDirectory: Fab2ApprovalMKLink
-            displayName: "MKLink - Clean"
-
-      - job: ReleaseMicrosoftBuildEngine
-        steps:
-          - script: |
-              echo BuildId: $(Build.BuildId)
-              echo Build reason: $(Build.Reason)
-              echo Repo Id: $(Build.Repository.Id)
-              echo Repo name: $(Build.Repository.Name)
-              echo Source version: $(Build.SourceVersion)
-              echo Core version: $(CoreVersion)
-              echo Configuration: $(Configuration)
-              echo Target Framework version: $(TargetFrameworkVersion)
-              echo Assembly title: $(AssemblyTitle)
-              echo MicrosoftBuildEngine: $(msBuild)
-              echo NugetSource: $(NugetSource)
-            displayName: "Echo Check"
-
-          - script: '"$(msBuild)" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=$(NugetSource) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Restore"
-
-          - script: '"$(msBuild)" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Build"
-
-          - script: '"$(msBuild)" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedconfiguration=$(Configuration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath=bin/$(Configuration) /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Package (.zip for msdeploy.exe)"
-
-          - task: CopyFiles@2
-            displayName: 'Copy Files to: D:\'
-            inputs:
-              Contents: "*"
-              SourceFolder: '$(AssemblyTitle)\bin\$(Configuration)\_PublishedWebsites\$(AssemblyTitle)_Package'
-              TargetFolder: 'D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(Build.BuildId)\$(Configuration)-$(AssemblyTitle)-Package'
-              OverWrite: true
-
-          - script: '"$(msBuild)" /target:Clean /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
-            workingDirectory: Fab2ApprovalSystem
-            displayName: "Framework - Clean"
-
-          - script: 'echo $(Build.BuildId)-$(Build.SourceVersion)-bin_x_x_\$(Configuration)\$(CoreVersion)\win-x64\$(Build.Repository.Name).txt'
-            displayName: "Force Fail"
-            enabled: true

Fab2ApprovalTests/DMO/AuditDMOTests.cs

@@ -51,7 +51,7 @@ public class AuditDMOTests {
         try { throw new Exception(); } catch (Exception) { }
     }
 
-    private static void AuditDMO(ILogger? logger, AppSettings appSettings, int auditNo, bool isAdmin) {
+    private static void AuditDMO(ILogger? logger, AppSettings appSettings) {
 #pragma warning disable IDE0059
         SetGlobalVars(logger, appSettings);
         AuditDMO auditDMO = new(appSettings);
@@ -61,25 +61,25 @@ public class AuditDMOTests {
         AuditedArea[] auditedAreas = auditDMO.GetAuditAreaList().ToArray();
         // IEnumerable<int> GetAuditFindingCategoryIdsByFindingId(int auditFindingsID);
         // AuditFindings GetAuditFindingsByID(int auditFindingsID);
-        AuditFindings[] auditFindings = auditDMO.GetAuditFindingsList(auditNo).ToArray();
-        Audit auditB = auditDMO.GetAuditItem(auditNo, appSettings.UserId);
-        Audit auditC = new();
-        AuditEdit auditEdit = auditDMO.GetAuditEdit(auditNo, auditC, isAdmin, appSettings.UserId);
-        Audit auditD = auditDMO.GetAuditItemReadOnly(auditNo, appSettings.UserId);
+        // IEnumerable<AuditFindings> GetAuditFindingsList(int auditNo);
+        // Audit GetAuditItem(int auditNo, appSettings.UserId);
+        // Audit GetAuditItemReadOnly(int auditNo, appSettings.UserId);
         Auditor[] auditors = auditDMO.GetAuditorList().ToArray();
-        AuditReportAttachment[] auditReportAttachments = auditDMO.GetAuditReportAttachments(auditNo).ToArray();
+        // IEnumerable<AuditReportAttachment> GetAuditReportAttachments(int auditNo);
         // C_8DAuditedStandard[] c_8DAuditedStandards = auditDMO.GetAuditStandardList().ToArray();
         AuditType[] auditTypes = auditDMO.GetAuditTypeList().ToArray();
         // CAFindings GetCAFindingsItem(int caFindingsID);
         // IEnumerable<AuditReportAttachment> GetCAFindingsItemAttachments(int caFindingsID);
-        CAFindings[] caFindings = auditDMO.GetCAFindingsList(auditNo).ToArray();
+        // IEnumerable<CAFindings> GetCAFindingsList(int auditNo);
         CANoList[] cANoLists = auditDMO.GetCorrectiveActionNoList().ToArray();
-        int openCACountByAuditNo = auditDMO.GetOpenCACountByAuditNo(auditNo);
+        // int GetOpenCACountByAuditNo(int auditNo);
         CAUserList[] cAUserLists = auditDMO.GetUserList().ToArray();
+        // Audit InsertAudit(Audit audit);
         // void InsertAuditReportAttachment(AuditReportAttachment attach);
         // void InsertCAFindings(CAFindings model);
         // int IsCAAssignedToAudit(int CANo, int auditNo);
         // void ReleaseLockOnDocument(appSettings.UserId, int issueID);
+        // void UpdateAudit(Audit audit, appSettings.UserId);
         // void UpdateCAFindings(CAFindings model);
         if (auditDMO is null) { }
 #pragma warning restore IDE0059
@@ -89,14 +89,13 @@ public class AuditDMOTests {
     [Ignore]
 #endif
     [TestMethod]
-    [DataRow(250, true)]
-    public void AuditDMOIsAttachedOnly(int auditNo, bool isAdmin) {
+    public void AuditDMOIsAttachedOnly() {
         _Logger?.LogInformation("Starting Web Application");
         IServiceProvider? serviceProvider = _WebApplicationFactory?.Services.CreateScope().ServiceProvider;
         AppSettings? appSettings = serviceProvider?.GetRequiredService<AppSettings>();
         Assert.IsTrue(appSettings is not null);
         if (System.Diagnostics.Debugger.IsAttached)
-            AuditDMO(_Logger, appSettings, auditNo, isAdmin);
+            AuditDMO(_Logger, appSettings);
         _Logger?.LogInformation("{TestName} completed", _TestContext?.TestName);
         NonThrowTryCatch();
     }

Fab2ApprovalTests/DMO/EngineeringChangeNoticeDMOTests.cs

@@ -85,18 +85,6 @@ public class EngineeringChangeNoticeDMOTests {
         // int SubmitTECNExtensionDocument(int issueID, appSettings.UserId, int documentType, DateTime extensionDate);
         // void TECNExtensionLog(int ecnNumber, DateTime extensionDate);
         // void UpdateECNType(int ecnNumber, string ecnType);
-        // ECN InsertECN(ECN ecn)
-        // void UpdateECN(ECN ecn)
-        // ECN GetECN(int ecnNumber, out int isITAR, int userID)
-        // ECN GetECNForRead(int ecnNumber, out int isITAR, int userID)
-        // ECNPdf GetECNPdf(int ecnNumber)
-        ECNAffectedDeparmtent[] departments = ecnDMO.GetDepartments().ToArray();
-        ECNAffectedModule[] modules = ecnDMO.GetModules().ToArray();
-        ECNCategory[] categories = ecnDMO.GetCategories().ToArray();
-        ECNArea[] eCNAreas = ecnDMO.GetECNAreas().ToArray();
-        ECNTechnology[] eCNTechnologies = ecnDMO.GetECNTechnologies().ToArray();
-        ECNAcknowledgementTrainingBy[] eCNAcknowledgementTrainingBy = ecnDMO.GetECNAcknowledgementTrainingBy().ToArray();
-        ProductFamilies[] productFamilies = ecnDMO.GetProductFamilies().ToArray();
         if (ecnDMO is null) { }
 #pragma warning restore IDE0059
     }

MesaFabApproval.API.Test/MesaFabApproval.API.Test.csproj

@@ -1,32 +0,0 @@
-<Project Sdk="Microsoft.NET.Sdk">
-
-  <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
-    <ImplicitUsings>enable</ImplicitUsings>
-    <Nullable>enable</Nullable>
-
-    <IsPackable>false</IsPackable>
-    <IsTestProject>true</IsTestProject>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <PackageReference Include="coverlet.collector" Version="6.0.0" />
-    <PackageReference Include="Dapper" Version="2.1.66" />
-    <PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="8.0.1" />
-    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="9.0.2" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-    <PackageReference Include="Moq" Version="4.20.72" />
-    <PackageReference Include="xunit" Version="2.5.3" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.3" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <Using Include="Xunit" />
-  </ItemGroup>
-
-    <ItemGroup>
-        <ProjectReference Include="..\MesaFabApproval.Shared\MesaFabApproval.Shared.csproj" />
-        <ProjectReference Include="..\MesaFabApproval.API\MesaFabApproval.API.csproj" />
-    </ItemGroup>
-
-</Project>

MesaFabApproval.API.Test/MonInUtilsTests.cs

@@ -1,63 +0,0 @@
-using Moq;
-using Microsoft.Extensions.Logging;
-using MesaFabApproval.API.Utilities;
-using MesaFabApproval.Shared.Models;
-using MesaFabApproval.Shared.Services;
-
-namespace MesaFabApproval.API.Test;
-
-public class MonInUtilsTests {
-    private readonly Mock<IMonInWorkerClient> _mockMonInClient;
-    private readonly Mock<ILogger<MonInUtils>> _mockLogger;
-    private readonly MonInUtils _monInUtils;
-
-    public MonInUtilsTests() {
-        _mockMonInClient = new Mock<IMonInWorkerClient>();
-        _mockMonInClient
-            .Setup(client => client.PostAverage(It.IsAny<string>(), It.IsAny<double>()))
-            .Verifiable();
-
-        _mockLogger = new Mock<ILogger<MonInUtils>>();
-
-        _monInUtils = new MonInUtils(_mockMonInClient.Object, _mockLogger.Object);
-    }
-
-    [Fact]
-    public void PostMetrics_ShouldPostAverageAndStatusOk_WhenNoErrors() {
-        string metricName = "TestMetric";
-        double latency = 100;
-        bool isArgumentError = false;
-        bool isInternalError = false;
-
-        _monInUtils.PostMetrics(metricName, latency, isArgumentError, isInternalError);
-
-        _mockMonInClient.Verify(client => client.PostAverage(metricName + "Latency", latency), Times.Once);
-        _mockMonInClient.Verify(client => client.PostStatus(metricName, StatusValue.Ok), Times.Once);
-    }
-
-    [Fact]
-    public void PostMetrics_ShouldPostAverageAndStatusOk_WhenArgumentError() {
-        string metricName = "TestMetric";
-        double latency = 100;
-        bool isArgumentError = true;
-        bool isInternalError = false;
-
-        _monInUtils.PostMetrics(metricName, latency, isArgumentError, isInternalError);
-
-        _mockMonInClient.Verify(client => client.PostAverage(metricName + "Latency", latency), Times.Once);
-        _mockMonInClient.Verify(client => client.PostStatus(metricName, StatusValue.Ok), Times.Once);
-    }
-
-    [Fact]
-    public void PostMetrics_ShouldPostAverageAndStatusCritical_WhenInternalError() {
-        string metricName = "TestMetric";
-        double latency = 100;
-        bool isArgumentError = false;
-        bool isInternalError = true;
-
-        _monInUtils.PostMetrics(metricName, latency, isArgumentError, isInternalError);
-
-        _mockMonInClient.Verify(client => client.PostAverage(metricName + "Latency", latency), Times.Once);
-        _mockMonInClient.Verify(client => client.PostStatus(metricName, StatusValue.Critical), Times.Once);
-    }
-}

MesaFabApproval.API.Test/PCRBServiceTests.cs

@@ -1,414 +0,0 @@
-using MesaFabApproval.API.Services;
-using MesaFabApproval.Models;
-using MesaFabApproval.Shared.Models;
-
-using Microsoft.Extensions.Caching.Memory;
-using Microsoft.Extensions.Logging;
-
-using Moq;
-
-namespace MesaFabApproval.API.Test;
-
-public static class MockMemoryCacheService {
-    public static Mock<IMemoryCache> GetMemoryCache(object expectedValue) {
-        Mock<IMemoryCache> mockMemoryCache = new Mock<IMemoryCache>();
-        mockMemoryCache
-            .Setup(x => x.TryGetValue(It.IsAny<object>(), out expectedValue))
-            .Returns(true);
-        mockMemoryCache
-            .Setup(x => x.CreateEntry(It.IsAny<object>()))
-            .Returns(Mock.Of<ICacheEntry>());
-        return mockMemoryCache;
-    }
-}
-
-public class PCRBServiceTests {
-    private readonly Mock<ILogger<PCRBService>> _loggerMock;
-    private readonly Mock<IDalService> _dalServiceMock;
-    private Mock<IMemoryCache> _cacheMock;
-    private readonly Mock<IUserService> _userServiceMock;
-    private readonly Mock<IApprovalService> _approvalServiceMock;
-    private readonly Mock<ISmtpService> _smtpServiceMock;
-    private PCRBService _pcrbService;
-
-    private static IEnumerable<PCRB> PCRBS = new List<PCRB>() {
-        new PCRB {
-            PlanNumber = 1,
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            InsertTimeStamp = DateTime.Now,
-            LastUpdateDate = DateTime.Now,
-            Type = "Type 1"
-        }
-    };
-
-    private static IEnumerable<PCRBFollowUp> FOLLOW_UPS = new List<PCRBFollowUp>() { 
-        new PCRBFollowUp { ID = 1, PlanNumber = 1, Step = 1, FollowUpDate = DateTime.Now }
-    };
-
-    private static AppSettings appSettings = new AppSettings(
-            Company: "Infineon",
-            DbConnectionString: "connectionString",
-            JwtAudience: "audience",
-            JwtIssuer: "issuer",
-            JwtKey: "key",
-            MrbAttachmentPath: "mrbAttachmentPath",
-            PcrbAttachmentPath: "pcrbAttachmentPath",
-            ShouldSendEmail: false,
-            SiteBaseUrl: "siteBaseUrl",
-            WorkingDirectoryName: "workingDirectoryName"
-        );
-
-    public PCRBServiceTests() {
-        _loggerMock = new Mock<ILogger<PCRBService>>();
-        _dalServiceMock = new Mock<IDalService>();
-        _userServiceMock = new Mock<IUserService>();
-        _approvalServiceMock = new Mock<IApprovalService>();
-        _smtpServiceMock = new Mock<ISmtpService>();
-        _cacheMock = MockMemoryCacheService.GetMemoryCache(FOLLOW_UPS);
-
-        _pcrbService = new PCRBService(
-            _loggerMock.Object,
-            _dalServiceMock.Object,
-            _cacheMock.Object,
-            _userServiceMock.Object,
-            _approvalServiceMock.Object,
-            _smtpServiceMock.Object,
-            appSettings
-        );
-    }
-
-    [Fact]
-    public async Task CreateNewPCRB_WithValidParam_ShouldCreatePCRB() {
-        var pcrb = new PCRB {
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            InsertTimeStamp = DateTime.Now,
-            LastUpdateDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .ReturnsAsync(1);
-
-        await _pcrbService.CreateNewPCRB(pcrb);
-
-        _dalServiceMock.Verify(d => d.ExecuteAsync(It.IsAny<string>(), pcrb), Times.Once);
-    }
-
-    [Fact]
-    public async Task CreateNewPCRB_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.CreateNewPCRB(null));
-    }
-
-    [Fact]
-    public async Task CreateNewPCRB_WithDatabaseFailure_ShouldThrowException() {
-        var pcrb = new PCRB {
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            InsertTimeStamp = DateTime.Now,
-            LastUpdateDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .ReturnsAsync(0);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.CreateNewPCRB(pcrb));
-    }
-
-    [Fact]
-    public async Task CreateNewPCRB_WithDatabaseException_ShouldThrowException() {
-        var pcrb = new PCRB {
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            InsertTimeStamp = DateTime.Now,
-            LastUpdateDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.CreateNewPCRB(pcrb));
-    }
-
-    [Fact]
-    public async Task UpdatePCRB_WithValidParam_ShouldUpdatePCRB() {
-        _cacheMock = MockMemoryCacheService.GetMemoryCache(PCRBS);
-
-        _pcrbService = new PCRBService(
-            _loggerMock.Object,
-            _dalServiceMock.Object,
-            _cacheMock.Object,
-            _userServiceMock.Object,
-            _approvalServiceMock.Object,
-            _smtpServiceMock.Object,
-            appSettings
-        );
-
-        var pcrb = new PCRB {
-            PlanNumber = 1,
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            LastUpdateDate = DateTime.Now,
-            ClosedDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .ReturnsAsync(1);
-
-        await _pcrbService.UpdatePCRB(pcrb);
-
-        _dalServiceMock.Verify(d => d.ExecuteAsync(It.IsAny<string>(), pcrb), Times.Once);
-    }
-
-    [Fact]
-    public async Task UpdatePCRB_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.UpdatePCRB(null));
-    }
-
-    [Fact]
-    public async Task UpdatePCRB_WithDatabaseFailure_ShouldThrowException() {
-        var pcrb = new PCRB {
-            PlanNumber = 1,
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            LastUpdateDate = DateTime.Now,
-            ClosedDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .ReturnsAsync(0);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.UpdatePCRB(pcrb));
-    }
-
-    [Fact]
-    public async Task UpdatePCRB_WithDatabaseException_ShouldThrowException() {
-        var pcrb = new PCRB {
-            PlanNumber = 1,
-            OwnerID = 1,
-            Title = "Test Title",
-            ChangeLevel = "Level 1",
-            ReasonForChange = "Test Reason",
-            ChangeDescription = "Test Description",
-            IsITAR = false,
-            CurrentStep = 1,
-            LastUpdateDate = DateTime.Now,
-            ClosedDate = DateTime.Now,
-            Type = "Type 1"
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), pcrb))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.UpdatePCRB(pcrb));
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithValidParam_ShouldCreateFollowUp() {
-        var followUp = new PCRBFollowUp {
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .ReturnsAsync(1);
-
-        await _pcrbService.CreateFollowUp(followUp);
-
-        _dalServiceMock.Verify(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp), Times.Once);
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.CreateFollowUp(null));
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithDatabaseFailure_ShouldThrowException() {
-        var followUp = new PCRBFollowUp {
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .ReturnsAsync(0);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.CreateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithDatabaseException_ShouldThrowException() {
-        var followUp = new PCRBFollowUp {
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.CreateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithCacheBypass_ShouldReturnFollowUps() {
-        int planNumber = 1;
-
-        _dalServiceMock.Setup(d => d.QueryAsync<PCRBFollowUp>(It.IsAny<string>(), It.IsAny<object>()))
-            .ReturnsAsync(FOLLOW_UPS);
-
-        IEnumerable<PCRBFollowUp> result = await _pcrbService.GetFollowUpsByPlanNumber(planNumber, true);
-
-        Assert.NotNull(result);
-        Assert.Single(result);
-        Assert.Equal(FOLLOW_UPS, result);
-        _dalServiceMock.Verify(d => d.QueryAsync<PCRBFollowUp>(It.IsAny<string>(), It.IsAny<object>()), Times.Once);
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithCacheBypass_AndDatabaseException_ShouldThrowException() {
-        int planNumber = 1;
-
-        _dalServiceMock.Setup(d => d.QueryAsync<PCRBFollowUp>(It.IsAny<string>(), It.IsAny<object>()))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.GetFollowUpsByPlanNumber(planNumber, true));
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithoutCacheBypass_ShouldReturnFollowUps() {
-        int planNumber = 1;
-
-        IEnumerable<PCRBFollowUp> result = await _pcrbService.GetFollowUpsByPlanNumber(planNumber, false);
-
-        Assert.NotNull(result);
-        Assert.Single(result);
-        Assert.Equal(FOLLOW_UPS, result);
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithValidParam_ShouldUpdateFollowUp() {
-        var followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .ReturnsAsync(1);
-
-        await _pcrbService.UpdateFollowUp(followUp);
-
-        _dalServiceMock.Verify(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp), Times.Once);
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.UpdateFollowUp(null));
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithDatabaseFailure_ShouldThrowException() {
-        var followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .ReturnsAsync(0);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.UpdateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithDatabaseException_ShouldThrowException() {
-        var followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 1,
-            Step = 1,
-            FollowUpDate = DateTime.Now
-        };
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync<PCRBFollowUp>(It.IsAny<string>(), followUp))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.UpdateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithValidId_ShouldDeleteFollowUp() {
-        int followUpId = 1;
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), It.IsAny<object>()))
-            .ReturnsAsync(1);
-
-        await _pcrbService.DeleteFollowUp(followUpId);
-
-        _dalServiceMock.Verify(d => d.ExecuteAsync(It.IsAny<string>(), It.IsAny<object>()), Times.Once);
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithInvalidId_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentException>(() => _pcrbService.DeleteFollowUp(0));
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithDatabaseFailure_ShouldThrowException() {
-        int followUpId = 1;
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), It.IsAny<object>()))
-            .ReturnsAsync(0);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.DeleteFollowUp(followUpId));
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithDatabaseException_ShouldThrowException() {
-        int followUpId = 1;
-
-        _dalServiceMock.Setup(d => d.ExecuteAsync(It.IsAny<string>(), It.IsAny<object>()))
-            .Throws<Exception>();
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.DeleteFollowUp(followUpId));
-    }
-}

MesaFabApproval.API/Clients/SmtpClientWrapper.cs

@@ -15,8 +15,6 @@ public class SmtpClientWrapper : ISmtpClientWrapper {
     }
 
     public void Send(MailMessage message) {
-        message.Subject = message.Subject.Replace('\r', ' ').Replace('\n', ' ');
-
         _client.Send(message);
     }
 }

MesaFabApproval.API/Controllers/ECNController.cs

@@ -53,36 +53,4 @@ public class ECNController : ControllerBase {
             }
         }
     }
-
-    [HttpGet]
-    [Route("ecn/allNumbers")]
-    public async Task<IActionResult> GetAllECNNumbers() {
-        DateTime start = DateTime.Now;
-        bool isInternalError = false;
-        string errorMessage = "";
-
-        try {
-            _logger.LogInformation($"Attempting to get all ECN#s");
-            
-            IEnumerable<int> allEcnNumbers = await _ecnService.GetAllECNNumbers();
-
-            return Ok(allEcnNumbers);
-        } catch (Exception ex) {
-            isInternalError = true;
-            errorMessage = $"Cannot get all ECN#s, because {ex.Message}";
-            return Problem(errorMessage);
-        } finally {
-            string metricName = "GetAllECNNumbers";
-            DateTime end = DateTime.Now;
-            double millisecondsDiff = (end - start).TotalMilliseconds;
-            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
-
-            if (isInternalError) {
-                _logger.LogError(errorMessage);
-                _monInClient.PostStatus(metricName, StatusValue.Critical);
-            } else {
-                _monInClient.PostStatus(metricName, StatusValue.Ok);
-            }
-        }
-    }
 }

MesaFabApproval.API/Controllers/PCRBController.cs

@@ -1,5 +1,4 @@
 using MesaFabApproval.API.Services;
-using MesaFabApproval.API.Utilities;
 using MesaFabApproval.Shared.Models;
 using MesaFabApproval.Shared.Services;
 
@@ -14,12 +13,12 @@ namespace MesaFabApproval.API.Controllers;
 public class PCRBController : ControllerBase {
     private readonly ILogger<MRBController> _logger;
     private readonly IPCRBService _pcrbService;
-    private readonly IMonInUtils _monInUtils;
+    private readonly IMonInWorkerClient _monInClient;
 
-    public PCRBController(ILogger<MRBController> logger, IPCRBService pcrbService, IMonInUtils monInUtils) {
+    public PCRBController(ILogger<MRBController> logger, IPCRBService pcrbService, IMonInWorkerClient monInClient) {
         _logger = logger ?? throw new ArgumentNullException("ILogger not injected");
         _pcrbService = pcrbService ?? throw new ArgumentNullException("IPCRBService not injected");
-        _monInUtils = monInUtils ?? throw new ArgumentNullException("IMonInUtils not injected");
+        _monInClient = monInClient ?? throw new ArgumentNullException("IMonInWorkerClient not injected");
     }
 
     [HttpPost]
@@ -50,8 +49,17 @@ public class PCRBController : ControllerBase {
             string metricName = "CreateNewPCRB";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -76,8 +84,14 @@ public class PCRBController : ControllerBase {
             string metricName = "GetAllPCRBs";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, false, isInternalError);
+            if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -109,8 +123,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBbyTitle";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -142,8 +165,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBbyPlanNumber";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -175,8 +207,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UpdatePCRB";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -208,14 +249,22 @@ public class PCRBController : ControllerBase {
             string metricName = "DeletePCRB";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
     [HttpPost]
     [Route("pcrb/attachment")]
-    [ApiExplorerSettings(IgnoreApi = true)]
     public async Task<IActionResult> UploadAttachment([FromForm] IFormFile file, [FromForm] PCRBAttachment attachment) {
         DateTime start = DateTime.Now;
         bool isArgumentError = false;
@@ -244,8 +293,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UploadPCRBAttachment";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -277,8 +335,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBAttachments";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -323,8 +390,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBAttachmentFile";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -356,8 +432,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UpdatePCRBAttachment";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -389,8 +474,17 @@ public class PCRBController : ControllerBase {
             string metricName = "DeletePCRBAttachment";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -422,8 +516,17 @@ public class PCRBController : ControllerBase {
             string metricName = "CreatePCRBActionItem";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -455,8 +558,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UpdatePCRBActionItem";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -488,8 +600,17 @@ public class PCRBController : ControllerBase {
             string metricName = "DeletePCRBActionItem";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -521,8 +642,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBActionItems";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -554,8 +684,17 @@ public class PCRBController : ControllerBase {
             string metricName = "CreatePCR3Document";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -587,8 +726,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UpdatePCR3Document";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -620,8 +768,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCR3Documents";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -653,8 +810,17 @@ public class PCRBController : ControllerBase {
             string metricName = "CreatePCRBAttendee";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -686,8 +852,17 @@ public class PCRBController : ControllerBase {
             string metricName = "UpdatePCRBAttendee";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -719,8 +894,17 @@ public class PCRBController : ControllerBase {
             string metricName = "DeletePCRBAttendee";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -752,8 +936,17 @@ public class PCRBController : ControllerBase {
             string metricName = "GetPCRBAttendees";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -785,8 +978,17 @@ public class PCRBController : ControllerBase {
             string metricName = "NotifyNewPCRBApprovers";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -820,8 +1022,17 @@ public class PCRBController : ControllerBase {
             string metricName = "NotifyPCRBApprovers";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -855,8 +1066,17 @@ public class PCRBController : ControllerBase {
             string metricName = "NotifyPCRBOriginator";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 
@@ -890,140 +1110,17 @@ public class PCRBController : ControllerBase {
             string metricName = "NotifyPCRBResponsiblePerson";
             DateTime end = DateTime.Now;
             double millisecondsDiff = (end - start).TotalMilliseconds;
+            _monInClient.PostAverage(metricName + "Latency", millisecondsDiff);
 
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
-        }
-    }
-
-    [HttpPost]
-    [Route("pcrb/followUp")]
-    public async Task<IActionResult> CreateFollowUp(PCRBFollowUp followUp) {
-        DateTime start = DateTime.Now;
-        bool isArgumentError = false;
-        bool isInternalError = false;
-        string errorMessage = "";
-
-        try {
-            _logger.LogInformation("Attempting to create follow up");
-
-            if (followUp is null) throw new ArgumentNullException("follow up cannot be null");
-
-            await _pcrbService.CreateFollowUp(followUp);
-
-            return Ok();
-        } catch (ArgumentException ex) {
-            isArgumentError = true;
-            errorMessage = ex.Message;
-            return BadRequest(errorMessage);
-        } catch (Exception ex) {
-            isInternalError = true;
-            errorMessage = $"Unable to create follow up, because {ex.Message}";
-            return Problem(errorMessage);
-        } finally {
-            string metricName = "CreatePCRBFollowUp";
-            DateTime end = DateTime.Now;
-            double millisecondsDiff = (end - start).TotalMilliseconds;
-
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
-        }
-    }
-
-    [HttpGet]
-    [Route("pcrb/followUps")]
-    public async Task<IActionResult> GetFollowUpsByPlanNumber(int planNumber, bool bypassCache) {
-        DateTime start = DateTime.Now;
-        bool isArgumentError = false;
-        bool isInternalError = false;
-        string errorMessage = "";
-
-        try {
-            _logger.LogInformation($"Attempting to get attendees for plan# {planNumber}");
-
-            if (planNumber <= 0) throw new ArgumentException($"{planNumber} is not a valid PCRB Plan#");
-
-            List<PCRBFollowUp> attendees = (await _pcrbService.GetFollowUpsByPlanNumber(planNumber, bypassCache)).ToList();
-
-            return Ok(attendees);
-        } catch (ArgumentException ex) {
-            isArgumentError = true;
-            errorMessage = ex.Message;
-            return BadRequest(errorMessage);
-        } catch (Exception ex) {
-            isInternalError = true;
-            errorMessage = $"Cannot get follow ups for plan# {planNumber}, because {ex.Message}";
-            return Problem(errorMessage);
-        } finally {
-            string metricName = "GetPCRBFollowUps";
-            DateTime end = DateTime.Now;
-            double millisecondsDiff = (end - start).TotalMilliseconds;
-
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
-        }
-    }
-
-    [HttpPut]
-    [Route("pcrb/followUp")]
-    public async Task<IActionResult> UpdateFollowUp(PCRBFollowUp followUp) {
-        DateTime start = DateTime.Now;
-        bool isArgumentError = false;
-        bool isInternalError = false;
-        string errorMessage = "";
-
-        try {
-            _logger.LogInformation("Attempting to update follow up");
-
-            if (followUp is null) throw new ArgumentNullException("follow up cannot be null");
-
-            await _pcrbService.UpdateFollowUp(followUp);
-
-            return Ok();
-        } catch (ArgumentException ex) {
-            isArgumentError = true;
-            errorMessage = ex.Message;
-            return BadRequest(errorMessage);
-        } catch (Exception ex) {
-            isInternalError = true;
-            errorMessage = $"Unable to update follow up, because {ex.Message}";
-            return Problem(errorMessage);
-        } finally {
-            string metricName = "UpdatePCRBFollowUp";
-            DateTime end = DateTime.Now;
-            double millisecondsDiff = (end - start).TotalMilliseconds;
-
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
-        }
-    }
-
-    [HttpDelete]
-    [Route("pcrb/followUp")]
-    public async Task<IActionResult> DeleteFollowUp(int id) {
-        DateTime start = DateTime.Now;
-        bool isArgumentError = false;
-        bool isInternalError = false;
-        string errorMessage = "";
-
-        try {
-            _logger.LogInformation("Attempting to delete follow up");
-
-            if (id <= 0) throw new ArgumentException($"{id} is not a valid PCRB follow up ID");
-
-            await _pcrbService.DeleteFollowUp(id);
-
-            return Ok();
-        } catch (ArgumentException ex) {
-            isArgumentError = true;
-            errorMessage = ex.Message;
-            return BadRequest(errorMessage);
-        } catch (Exception ex) {
-            isInternalError = true;
-            errorMessage = $"Unable to delete follow up, because {ex.Message}";
-            return Problem(errorMessage);
-        } finally {
-            string metricName = "DeletePCRBFollowUp";
-            DateTime end = DateTime.Now;
-            double millisecondsDiff = (end - start).TotalMilliseconds;
-
-            _monInUtils.PostMetrics(metricName, millisecondsDiff, isArgumentError, isInternalError);
+            if (isArgumentError) {
+                _logger.LogWarning(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            } else if (isInternalError) {
+                _logger.LogError(errorMessage);
+                _monInClient.PostStatus(metricName, StatusValue.Critical);
+            } else {
+                _monInClient.PostStatus(metricName, StatusValue.Ok);
+            }
         }
     }
 }

MesaFabApproval.API/Program.cs

@@ -6,7 +6,6 @@ using dotenv.net;
 
 using MesaFabApproval.API.Clients;
 using MesaFabApproval.API.Services;
-using MesaFabApproval.API.Utilities;
 using MesaFabApproval.Models;
 using MesaFabApproval.Shared.Services;
 
@@ -95,7 +94,6 @@ builder.Services.AddScoped<IECNService, ECNService>();
 builder.Services.AddScoped<ISmtpService, SmtpService>();
 builder.Services.AddScoped<IUserService, UserService>();
 builder.Services.AddScoped<IMonInWorkerClient, MonInWorkerClient>();
-builder.Services.AddScoped<IMonInUtils, MonInUtils>();
 builder.Services.AddScoped<IAuthenticationService, AuthenticationService>();
 builder.Services.AddScoped<IMRBService, MRBService>();
 builder.Services.AddScoped<IPCRBService, PCRBService>();
@@ -103,6 +101,7 @@ builder.Services.AddScoped<IApprovalService, ApprovalService>();
 
 builder.Services.AddControllers();
 
+#if DEBUG
 builder.Services.AddEndpointsApiExplorer();
 builder.Services.AddSwaggerGen(c => {
     c.SwaggerDoc("v1", new OpenApiInfo {
@@ -129,9 +128,13 @@ builder.Services.AddSwaggerGen(c => {
     }
   });
 });
+#endif
 
 WebApplication app = builder.Build();
 
+if (Debugger.IsAttached)
+    app.Services.GetRequiredService<IApprovalService>();
+
 app.UseCors();
 
 // Configure the HTTP request pipeline.

MesaFabApproval.API/Services/ApprovalService.cs

@@ -219,10 +219,7 @@ public class ApprovalService : IApprovalService {
                 queryBuilder.Append($"select a.*, src.SubRoleCategoryItem from Approval a ");
                 queryBuilder.Append("join SubRole sr on a.SubRoleID=sr.SubRoleID ");
                 queryBuilder.Append("join SubRoleCategory src on sr.SubRoleCategoryID=src.SubRoleCategoryID ");
-                queryBuilder.Append($"where UserID={userId} and ItemStatus=0 and ");
-                queryBuilder.Append($"(AssignedDate is not null and ");
-                queryBuilder.Append($"AssignedDate <= '{DateTime.Now.ToString("yyyy-MM-dd HH:mm:ss")}' and ");
-                queryBuilder.Append($"AssignedDate > '{DateTimeOffset.Now.AddYears(-2).DateTime.ToString("yyyy-MM-dd HH:mm:ss")}') and ");
+                queryBuilder.Append($"where UserID={userId} and ");
                 queryBuilder.Append($"((CompletedDate >= '{DateTime.Now.ToString("yyyy-MM-dd HH:mm:ss")}') or ");
                 queryBuilder.Append($"(CompletedDate is null));");
                 string sql = queryBuilder.ToString();

MesaFabApproval.API/Services/DalService.cs

@@ -1,5 +1,4 @@
 using System.Data;
-using System.Text;
 
 using Dapper;
 
@@ -7,9 +6,7 @@ namespace MesaFabApproval.API.Services;
 
 public interface IDalService {
     Task<IEnumerable<T>> QueryAsync<T>(string sql);
-    Task<IEnumerable<T>> QueryAsync<T>(string sql, object paramaters);
     Task<int> ExecuteAsync(string sql);
-    Task<int> ExecuteAsync<T>(string sql, T paramaters);
 }
 
 public class DalService : IDalService {
@@ -58,45 +55,6 @@ public class DalService : IDalService {
         return result;
     }
 
-    public async Task<IEnumerable<T>> QueryAsync<T>(string sql, object parameters) {
-        if (sql is null) throw new ArgumentNullException("sql cannot be null");
-        if (parameters is null) throw new ArgumentNullException("parameters cannot be null");
-
-        StringBuilder logBuilder = new();
-
-        int remainingRetries = RETRIES;
-        bool queryWasSuccessful = false;
-        Exception exception = null;
-        IEnumerable<T> result = new List<T>();
-        while (!queryWasSuccessful && remainingRetries > 0) {
-            int backoffSeconds = (RETRIES - remainingRetries--) * BACKOFF_SECONDS_INTERVAL;
-            Task.Delay(backoffSeconds * 1000).Wait();
-
-            try {
-                logBuilder.Clear();
-                logBuilder.Append($"Attempting to perform query with {sql} ");
-                logBuilder.Append($"and parameters {parameters.ToString()}. ");
-                logBuilder.Append($"Remaining retries: {remainingRetries}");
-                _logger.LogInformation(logBuilder.ToString());
-
-                using (IDbConnection conn = _dbConnectionService.GetConnection()) {
-                    result = await conn.QueryAsync<T>(sql, parameters);
-                }
-
-                queryWasSuccessful = true;
-            } catch (Exception ex) {
-                _logger.LogError($"An exception occurred while attempting to perform a query. Exception: {ex.Message}");
-                exception = ex;
-            }
-        }
-
-        if (!queryWasSuccessful && exception is not null) {
-            throw exception;
-        }
-
-        return result;
-    }
-
     public async Task<int> ExecuteAsync(string sql) {
         if (sql is null) throw new ArgumentNullException("sql cannot be null");
 
@@ -128,36 +86,4 @@ public class DalService : IDalService {
 
         return rowsAffected;
     }
-
-    public async Task<int> ExecuteAsync<T>(string sql, T parameters) {
-        if (sql is null) throw new ArgumentNullException("sql cannot be null");
-
-        int remainingRetries = RETRIES;
-        bool queryWasSuccessful = false;
-        Exception exception = null;
-        int rowsAffected = 0;
-        while (!queryWasSuccessful && remainingRetries > 0) {
-            int backoffSeconds = (RETRIES - remainingRetries--) * BACKOFF_SECONDS_INTERVAL;
-            Task.Delay(backoffSeconds * 1000).Wait();
-
-            try {
-                _logger.LogInformation($"Attempting to execute {sql} with parameters. Remaining retries: {remainingRetries}");
-
-                using (IDbConnection conn = _dbConnectionService.GetConnection()) {
-                    rowsAffected = await conn.ExecuteAsync(sql, parameters);
-                }
-
-                queryWasSuccessful = true;
-            } catch (Exception ex) {
-                _logger.LogError($"An exception occurred while attempting to execute a query. Exception: {ex.Message}");
-                exception = ex;
-            }
-        }
-
-        if (!queryWasSuccessful && exception is not null) {
-            throw exception;
-        }
-
-        return rowsAffected;
-    }
 }

MesaFabApproval.API/Services/ECNService.cs

@@ -4,7 +4,6 @@ namespace MesaFabApproval.API.Services;
 
 public interface IECNService {
     Task<bool> IsValidECNNumber(int number);
-    Task<IEnumerable<int>> GetAllECNNumbers();
 }
 
 public class ECNService : IECNService {
@@ -44,19 +43,4 @@ public class ECNService : IECNService {
             throw;
         }
     }
-
-    public async Task<IEnumerable<int>> GetAllECNNumbers() {
-        try {
-            _logger.LogInformation("Attempting to get all ECN#s");
-
-            string sql = "select ECNNumber from ECN where Deleted=0 and Cancelled=0";
-
-            IEnumerable<int> allEcnNumbers = await _dalService.QueryAsync<int>(sql);
-
-            return allEcnNumbers;
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to get all ECN#s, because {ex.Message}");
-            throw;
-        }
-    }
 }

MesaFabApproval.API/Services/MRBService.cs

@@ -852,8 +852,8 @@ public class MRBService : IMRBService {
                             convertToPart += partStr;
                     }
 
-                    dt.Rows.Add(action.Action, convertFromCustomer, convertFromPart, action.LotNumber, 
-                                action.Quantity.ToString(), convertToCustomer, convertToPart,
+                    dt.Rows.Add(action.Action, convertFromCustomer, convertFromPart, action.Quantity.ToString(),
+                                convertToCustomer, convertToPart,
                                 DateTimeUtilities.GetDateAsStringMinDefault(action.AssignedDate),
                                 DateTimeUtilities.GetDateAsStringMaxDefault(action.CompletedDate),
                                 action.CompletedByUser is null ? "" : action.CompletedByUser.GetFullName());

MesaFabApproval.API/Services/PCRBService.cs

@@ -37,10 +37,6 @@ public interface IPCRBService {
     Task NotifyApprovers(PCRBNotification notification);
     Task NotifyOriginator(PCRBNotification notification);
     Task NotifyResponsiblePerson(PCRBActionItemNotification notification);
-    Task CreateFollowUp(PCRBFollowUp followUp);
-    Task<IEnumerable<PCRBFollowUp>> GetFollowUpsByPlanNumber(int planNumber, bool bypassCache);
-    Task UpdateFollowUp(PCRBFollowUp followUp);
-    Task DeleteFollowUp(int id);
 }
 
 public class PCRBService : IPCRBService {
@@ -78,15 +74,16 @@ public class PCRBService : IPCRBService {
 
             if (pcrb is null) throw new ArgumentNullException("PCRB cannot be null");
 
-            pcrb.LastUpdateDate = DateTime.Now;
-
             StringBuilder queryBuilder = new();
             queryBuilder.Append("insert into CCChangeControl (OwnerID, Title, ChangeLevel, ReasonForChange, ");
-            queryBuilder.Append("ChangeDescription, IsITAR, CurrentStep, InsertTimeStamp, LastUpdateDate, Type) ");
-            queryBuilder.Append("values (@OwnerID, @Title, @ChangeLevel, @ReasonForChange, @ChangeDescription, ");
-            queryBuilder.Append("@IsITAR, @CurrentStep, @InsertTimeStamp, @LastUpdateDate, @Type)");
+            queryBuilder.Append("ChangeDescription, IsITAR, CurrentStep, InsertTimeStamp, LastUpdateDate) ");
+            queryBuilder.Append($"values ({pcrb.OwnerID}, '{pcrb.Title}', '{pcrb.ChangeLevel}', ");
+            queryBuilder.Append($"'{pcrb.ReasonForChange}', '{pcrb.ChangeDescription}', ");
+            queryBuilder.Append($"{Convert.ToInt32(pcrb.IsITAR)}, {pcrb.CurrentStep}, ");
+            queryBuilder.Append($"'{pcrb.InsertTimeStamp.ToString("yyyy-MM-dd HH:mm:ss")}', ");
+            queryBuilder.Append($"'{DateTime.Now.ToString("yyyy-MM-dd HH:mm:ss")}')");
 
-            int rowsCreated = await _dalService.ExecuteAsync(queryBuilder.ToString(), pcrb);
+            int rowsCreated = await _dalService.ExecuteAsync(queryBuilder.ToString());
 
             if (rowsCreated <= 0) throw new Exception("unable to insert new PCRB in the database");
         } catch (Exception ex) {
@@ -197,16 +194,18 @@ public class PCRBService : IPCRBService {
 
             if (pcrb is null) throw new ArgumentNullException("PCRB cannot be null");
 
-            pcrb.LastUpdateDate = DateTime.Now;
-
             StringBuilder queryBuilder = new();
-            queryBuilder.Append("update CCChangeControl set OwnerID=@OwnerID, Title=@Title, ChangeLevel=@ChangeLevel, ");
-            queryBuilder.Append("Type=@Type, CurrentStep=@CurrentStep, ReasonForChange=@ReasonForChange, ");
-            queryBuilder.Append("ChangeDescription=@ChangeDescription, IsITAR=@IsITAR, ClosedDate=@ClosedDate, ");
-            queryBuilder.Append("LastUpdateDate=@LastUpdateDate ");
-            queryBuilder.Append($"where PlanNumber=@PlanNumber");
+            queryBuilder.Append($"update CCChangeControl set OwnerID={pcrb.OwnerID}, ");
+            queryBuilder.Append($"Title='{pcrb.Title.Replace("'", "''")}', ChangeLevel='{pcrb.ChangeLevel}', ");
+            queryBuilder.Append($"CurrentStep={pcrb.CurrentStep}, ReasonForChange='{pcrb.ReasonForChange.Replace("'", "''")}', ");
+            queryBuilder.Append($"ChangeDescription='{pcrb.ChangeDescription.Replace("'", "''")}', ");
+            queryBuilder.Append($"IsITAR={Convert.ToInt32(pcrb.IsITAR)}, ");
+            queryBuilder.Append($"InsertTimeStamp='{pcrb.InsertTimeStamp.ToString("yyyy-MM-dd HH:mm:ss")}', ");
+            queryBuilder.Append($"ClosedDate='{pcrb.ClosedDate.ToString("yyyy-MM-dd HH:mm:ss")}', ");
+            queryBuilder.Append($"LastUpdateDate='{DateTime.Now.ToString("yyyy-MM-dd HH:mm:ss")}' ");
+            queryBuilder.Append($"where PlanNumber={pcrb.PlanNumber}");
 
-            int rowsAffected = await _dalService.ExecuteAsync(queryBuilder.ToString(), pcrb);
+            int rowsAffected = await _dalService.ExecuteAsync(queryBuilder.ToString());
 
             if (rowsAffected <= 0) throw new Exception("unable to perform update in database");
 
@@ -486,20 +485,6 @@ public class PCRBService : IPCRBService {
             int rowsAffected = await _dalService.ExecuteAsync(queryBuilder.ToString());
 
             if (rowsAffected <= 0) throw new Exception("update failed in database");
-
-            IEnumerable<PCRBAttendee>? attendees = _cache.Get<IEnumerable<PCRBAttendee>>($"pcrbAttendees{attendee.PlanNumber}");
-            if (attendees is not null) {
-                foreach (PCRBAttendee cachedAttendee in attendees) {
-                    if (cachedAttendee.ID == attendee.ID) {
-                        cachedAttendee.Location = attendee.Location;
-                        cachedAttendee.Attended = attendee.Attended;
-                        cachedAttendee.JobTitle = attendee.JobTitle;
-                        cachedAttendee.AttendeeID = attendee.AttendeeID;
-                        cachedAttendee.Step = attendee.Step;
-                    }
-                }
-                _cache.Set($"pcrbAttendees{attendee.PlanNumber}", attendees, DateTimeOffset.Now.AddMinutes(15));
-            }
         } catch (Exception ex) {
             _logger.LogError($"Unable to update attendee, because {ex.Message}");
             throw;
@@ -756,90 +741,6 @@ public class PCRBService : IPCRBService {
         }
     }
 
-    public async Task CreateFollowUp(PCRBFollowUp followUp) {
-        try {
-            _logger.LogInformation("Attempting to create PCRB follow up");
-
-            if (followUp is null) throw new ArgumentNullException("follow up cannot be null");
-
-            StringBuilder queryBuilder = new();
-            queryBuilder.Append("insert into CCPCRBFollowUp (PlanNumber, Step, FollowUpDate, CompletedDate) ");
-            queryBuilder.Append("values (@PlanNumber, @Step, @FollowUpDate, @CompletedDate)");
-
-            int rowsReturned = await _dalService.ExecuteAsync<PCRBFollowUp>(queryBuilder.ToString(), followUp);
-
-            if (rowsReturned <= 0) throw new Exception("unable to insert new follow up in the database");
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to create new follow up, because {ex.Message}");
-            throw;
-        }
-    }
-
-    public async Task<IEnumerable<PCRBFollowUp>> GetFollowUpsByPlanNumber(int planNumber, bool bypassCache) {
-        try {
-            _logger.LogInformation($"Attempting to fetch follow ups for PCRB {planNumber}");
-
-            if (planNumber <= 0) throw new ArgumentException($"{planNumber} is not a valid PCRB Plan#");
-
-            IEnumerable<PCRBFollowUp>? followUps = new List<PCRBFollowUp>();
-
-            if (!bypassCache)
-                followUps = _cache.Get<IEnumerable<PCRBFollowUp>>($"pcrbFollowUps{planNumber}");
-
-            if (followUps is null || followUps.Count() == 0) { 
-                string sql = "select * from CCPCRBFollowUp where PlanNumber=@PlanNumber";
-
-                followUps = await _dalService.QueryAsync<PCRBFollowUp>(sql, new { PlanNumber = planNumber });
-
-                if (followUps is not null)
-                    _cache.Set($"pcrbFollowUps{planNumber}", followUps, DateTimeOffset.Now.AddMinutes(15));
-            }
-
-            return followUps ?? new List<PCRBFollowUp>();
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to fetch follow ups for PCRB {planNumber}, because {ex.Message}");
-            throw;
-        }
-    }
-
-    public async Task UpdateFollowUp(PCRBFollowUp followUp) {
-        try {
-            _logger.LogInformation("Attempting to update a follow up");
-
-            if (followUp is null)
-                throw new ArgumentNullException("follow up cannot be null");
-
-            StringBuilder queryBuilder = new();
-            queryBuilder.Append("update CCPCRBFollowUp set Step=@Step, FollowUpDate=@FollowUpDate, IsComplete=@IsComplete, ");
-            queryBuilder.Append("IsDeleted=@IsDeleted, CompletedDate=@CompletedDate, Comments=@Comments ");
-            queryBuilder.Append("where ID=@ID");
-
-            int rowsAffected = await _dalService.ExecuteAsync<PCRBFollowUp>(queryBuilder.ToString(), followUp);
-
-            if (rowsAffected <= 0) throw new Exception("update failed in database");
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to update follow up, because {ex.Message}");
-            throw;
-        }
-    }
-
-    public async Task DeleteFollowUp(int id) {
-        try {
-            _logger.LogInformation($"Attempting to delete follow up {id}");
-
-            if (id <= 0) throw new ArgumentException($"{id} is not a valid follow up ID");
-
-            string sql = "delete from CCPCRBFollowUp where ID=@ID";
-
-            int rowsAffected = await _dalService.ExecuteAsync(sql, new { ID = id });
-
-            if (rowsAffected <= 0) throw new Exception("delete operation failed in database");
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to delete follow up {id}, because {ex.Message}");
-            throw;
-        }
-    }
-
     private async Task SaveAttachmentInDb(IFormFile file, PCRBAttachment attachment) {
         try {
             _logger.LogInformation($"Attempting to save attachment to database");

MesaFabApproval.API/Utilities/MonInUtils.cs

@@ -1,48 +0,0 @@
-using MesaFabApproval;
-using MesaFabApproval.API;
-using MesaFabApproval.API.Services;
-using MesaFabApproval.API.Utilities;
-using MesaFabApproval.Shared.Models;
-using MesaFabApproval.Shared.Services;
-
-namespace MesaFabApproval.API.Utilities;
-
-public interface IMonInUtils {
-    public void PostMetrics(string metricName,
-                            double latency,
-                            bool isArgumentError,
-                            bool isInternalError);
-}
-
-public class MonInUtils : IMonInUtils {
-    private readonly IMonInWorkerClient _monInClient;
-    private readonly ILogger<MonInUtils> _logger;
-
-    public MonInUtils(IMonInWorkerClient monInClient, ILogger<MonInUtils> logger) {
-        _monInClient = monInClient ??
-            throw new ArgumentNullException("IMonInWorkerClient not injected");
-        _logger = logger ??
-            throw new ArgumentNullException("ILogger not injected");
-    }
-
-    public void PostMetrics(string metricName,
-                            double latency,
-                            bool isArgumentError,
-                            bool isInternalError) {
-        try {
-            _logger.LogInformation("Attempting to post metrics to MonIn");
-
-            _monInClient.PostAverage(metricName + "Latency", latency);
-
-            if (isArgumentError) {
-                _monInClient.PostStatus(metricName, StatusValue.Ok);
-            } else if (isInternalError) {
-                _monInClient.PostStatus(metricName, StatusValue.Critical);
-            } else {
-                _monInClient.PostStatus(metricName, StatusValue.Ok);
-            }
-        } catch (Exception ex) {
-            _logger.LogError($"Unable to post metrics to MonIn, because {ex.Message}");
-        }
-    }
-}

MesaFabApproval.API/pipeline.yml

@@ -1,100 +0,0 @@
-trigger:
-  branches:
-    include:
-      - master
-  paths:
-    include:
-      - MesaFabApproval.API
-      - MesaFabApproval.Shared
-
-variables:
-  buildConfiguration: "Release"
-
-stages:
-  - stage: Test
-    displayName: "Test"
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem-Dev
-    jobs:
-      - job: BuildPublish
-        steps:
-          - script: |
-              echo build configuration: $(BuildConfiguration)
-            displayName: "Echo Check"
-
-          - task: DotNetCoreCLI@2
-            displayName: "Build"
-            inputs:
-              command: "build"
-              configuration: $(BuildConfiguration)
-              projects: MesaFabApproval.API
-
-          - task: DotNetCoreCLI@2
-            displayName: "Test"
-            inputs:
-              command: "test"
-              configuration: $(BuildConfiguration)
-              publishTestResults: true
-              projects: MesaFabApproval.API.Test
-
-          - task: DotNetCoreCLI@2
-            displayName: "Publish"
-            inputs:
-              command: "publish"
-              publishWebProjects: false
-              projects: MesaFabApproval.API
-              zipAfterPublish: true
-              arguments: "-c $(BuildConfiguration) -o $(Build.ArtifactStagingDirectory) -r win-x64 --self-contained"
-
-          - task: CopyFiles@2
-            displayName: "Copy Files"
-            inputs:
-              SourceFolder: "$(Build.ArtifactStagingDirectory)"
-              TargetFolder: 'D:\$(Build.Repository.Name)\API\$(Build.BuildId)\$(BuildConfiguration)'
-              CleanTargetFolder: true
-              retryCount: "3"
-
-  - stage: Prod
-    displayName: "Prod"
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem
-    jobs:
-      - job: BuildPublish
-        steps:
-          - script: |
-              echo build configuration: $(BuildConfiguration)
-            displayName: "Echo Check"
-
-          - task: DotNetCoreCLI@2
-            displayName: "Build"
-            inputs:
-              command: "build"
-              configuration: $(BuildConfiguration)
-              projects: MesaFabApproval.API
-
-          - task: DotNetCoreCLI@2
-            displayName: "Test"
-            inputs:
-              command: "test"
-              configuration: $(BuildConfiguration)
-              publishTestResults: true
-              projects: MesaFabApproval.API.Test
-
-          - task: DotNetCoreCLI@2
-            displayName: "Publish"
-            inputs:
-              command: "publish"
-              publishWebProjects: false
-              projects: MesaFabApproval.API
-              zipAfterPublish: true
-              arguments: "-c $(BuildConfiguration) -o $(Build.ArtifactStagingDirectory) -r win-x64 --self-contained"
-
-          - task: CopyFiles@2
-            displayName: "Copy Files"
-            inputs:
-              SourceFolder: "$(Build.ArtifactStagingDirectory)"
-              TargetFolder: 'D:\$(Build.Repository.Name)\API\$(Build.BuildId)\$(BuildConfiguration)'
-              CleanTargetFolder: true
-              retryCount: "3"

MesaFabApproval.Client.Test/MesaFabApproval.Client.Test.csproj

@@ -1,31 +0,0 @@
-<Project Sdk="Microsoft.NET.Sdk">
-
-  <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
-    <ImplicitUsings>enable</ImplicitUsings>
-    <Nullable>enable</Nullable>
-
-    <IsPackable>false</IsPackable>
-    <IsTestProject>true</IsTestProject>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <PackageReference Include="coverlet.collector" Version="6.0.0" />
-    <PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="8.0.1" />
-    <PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-    <PackageReference Include="Moq" Version="4.20.72" />
-    <PackageReference Include="MudBlazor" Version="8.3.0" />
-    <PackageReference Include="xunit" Version="2.5.3" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.3" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <ProjectReference Include="..\MesaFabApproval.Client\MesaFabApproval.Client.csproj" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <Using Include="Xunit" />
-  </ItemGroup>
-
-</Project>

MesaFabApproval.Client.Test/PCRBServiceTests.cs

@@ -1,347 +0,0 @@
-using System.Net;
-using System.Text.Json;
-
-using MesaFabApproval.Client.Services;
-using MesaFabApproval.Shared.Models;
-
-using Microsoft.Extensions.Caching.Memory;
-
-using Moq;
-using Moq.Protected;
-
-using MudBlazor;
-
-namespace MesaFabApproval.Client.Test;
-
-public class PCRBServiceTests {
-    private readonly Mock<IMemoryCache> _mockCache;
-    private readonly Mock<IHttpClientFactory> _mockHttpClientFactory;
-    private readonly Mock<ISnackbar> _mockSnackbar;
-    private readonly Mock<IUserService> _mockUserService;
-    private readonly PCRBService _pcrbService;
-
-    private static IEnumerable<PCRBFollowUp> FOLLOW_UPS = new List<PCRBFollowUp>() {
-        new PCRBFollowUp { ID = 1, PlanNumber = 1, Step = 1, FollowUpDate = DateTime.Now }
-    };
-
-    private static HttpResponseMessage SUCCESSFUL_RESPONSE = new HttpResponseMessage(HttpStatusCode.OK);
-    private static HttpResponseMessage UNSUCCESSFUL_RESPONSE = new HttpResponseMessage(HttpStatusCode.InternalServerError);
-
-    public static class MockMemoryCacheService {
-        public static Mock<IMemoryCache> GetMemoryCache(object expectedValue) {
-            Mock<IMemoryCache> mockMemoryCache = new Mock<IMemoryCache>();
-            mockMemoryCache
-                .Setup(x => x.TryGetValue(It.IsAny<object>(), out expectedValue))
-                .Returns(true);
-            mockMemoryCache
-                .Setup(x => x.CreateEntry(It.IsAny<object>()))
-                .Returns(Mock.Of<ICacheEntry>());
-            return mockMemoryCache;
-        }
-    }
-
-    public PCRBServiceTests() {
-        _mockCache = MockMemoryCacheService.GetMemoryCache(FOLLOW_UPS);
-        _mockHttpClientFactory = new Mock<IHttpClientFactory>();
-        _mockSnackbar = new Mock<ISnackbar>();
-        _mockUserService = new Mock<IUserService>();
-
-        _pcrbService = new PCRBService(
-            _mockCache.Object,
-            _mockHttpClientFactory.Object,
-            _mockSnackbar.Object,
-            _mockUserService.Object);
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithValidParams_ShouldCallHttpPost_AndRefreshCache() {
-        PCRBFollowUp followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 123,
-            Step = 1,
-            FollowUpDate = DateTime.Now,
-            Comments = "Test"
-        };
-
-        HttpResponseMessage getResponse = new HttpResponseMessage {
-            StatusCode = HttpStatusCode.OK,
-            Content = new StringContent(JsonSerializer.Serialize(new List<PCRBFollowUp> { followUp }))
-        };
-
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Post),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(SUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Get),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(getResponse)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await _pcrbService.CreateFollowUp(followUp);
-
-        mockHttpMessageHandler.Protected().Verify(
-            "SendAsync",
-            Times.Once(),
-            ItExpr.Is<HttpRequestMessage>(
-                req => 
-                    req.Method == HttpMethod.Post &&
-                    req.RequestUri != null &&
-                    req.RequestUri.AbsoluteUri.Equals("https://localhost:5000/pcrb/followUp")),
-            ItExpr.IsAny<CancellationToken>());
-
-        mockHttpMessageHandler.Protected().Verify(
-            "SendAsync",
-            Times.Once(),
-            ItExpr.Is<HttpRequestMessage>(
-                req => 
-                    req.Method == HttpMethod.Get &&
-                    req.RequestUri != null &&
-                    req.RequestUri.AbsoluteUri.Equals("https://localhost:5000/pcrb/followUps?planNumber=123&bypassCache=True")),
-            ItExpr.IsAny<CancellationToken>());
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithBadResponse_ShouldThrowException() {
-        PCRBFollowUp followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 123,
-            Step = 1,
-            FollowUpDate = DateTime.Now,
-            Comments = "Test"
-        };
-
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Post),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(UNSUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.CreateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task CreateFollowUp_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.CreateFollowUp(null));
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithBypassCache_ShouldCallHttpGetAndReturnFollowUps() {
-        PCRBFollowUp followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 123,
-            Step = 1,
-            FollowUpDate = DateTime.Now,
-            Comments = "Test"
-        };
-
-        HttpResponseMessage getResponse = new HttpResponseMessage {
-            StatusCode = HttpStatusCode.OK,
-            Content = new StringContent(JsonSerializer.Serialize(new List<PCRBFollowUp> { followUp }))
-        };
-
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Get),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(getResponse)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        IEnumerable<PCRBFollowUp> followUps = await _pcrbService.GetFollowUpsByPlanNumber(123, true);
-
-        mockHttpMessageHandler.Protected().Verify(
-            "SendAsync",
-            Times.Once(),
-            ItExpr.Is<HttpRequestMessage>(
-                req =>
-                    req.Method == HttpMethod.Get &&
-                    req.RequestUri != null &&
-                    req.RequestUri.AbsoluteUri.Equals("https://localhost:5000/pcrb/followUps?planNumber=123&bypassCache=True")),
-            ItExpr.IsAny<CancellationToken>());
-
-        Assert.Single(followUps);
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithoutBypassCache_ShouldReturnFollowUpsFromCache() {
-        IEnumerable<PCRBFollowUp> followUps = await _pcrbService.GetFollowUpsByPlanNumber(1, false);
-        Assert.Single(followUps);
-    }
-
-    [Fact]
-    public async Task GetFollowUpsByPlanNumber_WithBadResponse_ShouldThrowException() {
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Get),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(UNSUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.GetFollowUpsByPlanNumber(1, true));
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithValidParams_ShouldCallHttpPut() {
-        PCRBFollowUp followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 123,
-            Step = 1,
-            FollowUpDate = DateTime.Now,
-            Comments = "Test"
-        };
-
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Put),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(SUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await _pcrbService.UpdateFollowUp(followUp);
-
-        mockHttpMessageHandler.Protected().Verify(
-            "SendAsync",
-            Times.Once(),
-            ItExpr.Is<HttpRequestMessage>(
-                req =>
-                    req.Method == HttpMethod.Put &&
-                    req.RequestUri != null &&
-                    req.RequestUri.AbsoluteUri.Equals("https://localhost:5000/pcrb/followUp")),
-            ItExpr.IsAny<CancellationToken>());
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithNullParam_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentNullException>(() => _pcrbService.UpdateFollowUp(null));
-    }
-
-    [Fact]
-    public async Task UpdateFollowUp_WithBadResponse_ShouldThrowException() {
-        PCRBFollowUp followUp = new PCRBFollowUp {
-            ID = 1,
-            PlanNumber = 123,
-            Step = 1,
-            FollowUpDate = DateTime.Now,
-            Comments = "Test"
-        };
-
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Put),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(UNSUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.UpdateFollowUp(followUp));
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithValidParams_ShouldCallHttpDelete() {
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Delete),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(SUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await _pcrbService.DeleteFollowUp(1);
-
-        mockHttpMessageHandler.Protected().Verify(
-            "SendAsync",
-            Times.Once(),
-            ItExpr.Is<HttpRequestMessage>(
-                req =>
-                    req.Method == HttpMethod.Delete &&
-                    req.RequestUri != null &&
-                    req.RequestUri.AbsoluteUri.Equals("https://localhost:5000/pcrb/followUp?id=1")),
-            ItExpr.IsAny<CancellationToken>());
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithBadId_ShouldThrowException() {
-        await Assert.ThrowsAsync<ArgumentException>(() => _pcrbService.DeleteFollowUp(0));
-    }
-
-    [Fact]
-    public async Task DeleteFollowUp_WithBadResponse_ShouldThrowException() {
-        var mockHttpMessageHandler = new Mock<HttpMessageHandler>();
-
-        mockHttpMessageHandler.Protected()
-            .Setup<Task<HttpResponseMessage>>(
-                "SendAsync",
-                ItExpr.Is<HttpRequestMessage>(_ => _.Method == HttpMethod.Delete),
-                ItExpr.IsAny<CancellationToken>())
-            .ReturnsAsync(UNSUCCESSFUL_RESPONSE)
-            .Verifiable();
-
-        var httpClient = new HttpClient(mockHttpMessageHandler.Object) {
-            BaseAddress = new Uri("https://localhost:5000")
-        };
-
-        _mockHttpClientFactory.Setup(_ => _.CreateClient(It.IsAny<string>())).Returns(httpClient);
-
-        await Assert.ThrowsAsync<Exception>(() => _pcrbService.DeleteFollowUp(1));
-    }
-}

MesaFabApproval.Client/Pages/AuthenticatedRedirect.razor

@@ -1,2 +1,65 @@
 @page "/redirect"
 @attribute [AllowAnonymous]
+@inject MesaFabApprovalAuthStateProvider authStateProvider
+@inject IAuthenticationService authService
+@inject IUserService userService
+@inject ISnackbar snackbar
+@inject MesaFabApprovalAuthStateProvider authStateProvider
+@inject NavigationManager navigationManager
+
+@code {
+    private string? _jwt;
+    private string? _refreshToken;
+    private string? _redirectPath;
+
+    protected override async Task OnParametersSetAsync() {
+        try {
+            Uri uri = navigationManager.ToAbsoluteUri(navigationManager.Uri);
+
+            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("jwt", out var jwt)) {
+                _jwt = System.Net.WebUtility.UrlDecode(jwt);
+            }
+
+            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("refreshToken", out var refreshToken)) {
+                _refreshToken = System.Net.WebUtility.UrlDecode(refreshToken);
+            }
+
+            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("redirectPath", out var redirectPath)) {
+                _redirectPath = redirectPath.ToString();
+            }
+
+            if (!string.IsNullOrWhiteSpace(_jwt) && !string.IsNullOrWhiteSpace(_refreshToken)) {
+                await authService.SetTokens(_jwt, _refreshToken);
+
+                ClaimsPrincipal principal = authService.GetClaimsPrincipalFromJwt(_jwt);
+
+                string loginId = userService.GetLoginIdFromClaimsPrincipal(principal);
+
+                await authService.ClearCurrentUser();
+                await authService.ClearTokens();
+
+                await authService.SetLoginId(loginId);
+                await authService.SetTokens(_jwt, _refreshToken);
+                await authService.SetCurrentUser(null);
+
+                await authStateProvider.StateHasChanged(principal);
+            }
+
+            if (authStateProvider.CurrentUser is not null && !string.IsNullOrWhiteSpace(_redirectPath)) {
+                navigationManager.NavigateTo(_redirectPath);
+            } else {
+                await authStateProvider.Logout();
+
+                if (!string.IsNullOrWhiteSpace(_redirectPath)) {
+                    navigationManager.NavigateTo($"login/{_redirectPath}");
+                } else {
+                    navigationManager.NavigateTo("login");
+                }
+            }
+            
+        } catch (Exception ex) {
+            snackbar.Add($"Redirect failed, because {ex.Message}", Severity.Error);
+            navigationManager.NavigateTo("login");
+        }
+    }
+}

MesaFabApproval.Client/Pages/AuthenticatedRedirect.razor.cs

@@ -1,77 +0,0 @@
-using MesaFabApproval.Client.Services;
-
-using Microsoft.AspNetCore.Components;
-using Microsoft.AspNetCore.WebUtilities;
-
-using MudBlazor;
-
-using System.Security.Claims;
-
-namespace MesaFabApproval.Client.Pages;
-
-public partial class AuthenticatedRedirect {
-    [Inject] MesaFabApprovalAuthStateProvider authStateProvider { get; set; }
-    [Inject] IAuthenticationService authService { get; set; }
-    [Inject] IUserService userService { get; set; }
-    [Inject] ISnackbar snackbar { get; set; }
-    [Inject] NavigationManager navigationManager { get; set; }
-
-    private string? _jwt;
-    private string? _refreshToken;
-    private string? _redirectPath;
-
-    protected override async Task OnParametersSetAsync() {
-        try {
-            Uri uri = navigationManager.ToAbsoluteUri(navigationManager.Uri);
-
-            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("jwt", out var jwt)) {
-                _jwt = System.Net.WebUtility.UrlDecode(jwt);
-            }
-
-            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("refreshToken", out var refreshToken)) {
-                _refreshToken = System.Net.WebUtility.UrlDecode(refreshToken);
-            }
-
-            if (QueryHelpers.ParseQuery(uri.Query).TryGetValue("redirectPath", out var redirectPath)) {
-                _redirectPath = redirectPath.ToString();
-            }
-
-            if (!string.IsNullOrWhiteSpace(_jwt) && !string.IsNullOrWhiteSpace(_refreshToken)) {
-                await authService.SetTokens(_jwt, _refreshToken);
-
-                ClaimsPrincipal principal = authService.GetClaimsPrincipalFromJwt(_jwt);
-
-                string loginId = userService.GetLoginIdFromClaimsPrincipal(principal);
-
-                await authService.ClearCurrentUser();
-                await authService.ClearTokens();
-
-                await authService.SetLoginId(loginId);
-                await authService.SetTokens(_jwt, _refreshToken);
-                await authService.SetCurrentUser(null);
-
-                await authStateProvider.StateHasChanged(principal);
-            }
-
-            if (authStateProvider.CurrentUser is not null && !string.IsNullOrWhiteSpace(_redirectPath)) {
-                navigationManager.NavigateTo(_redirectPath);
-            } else {
-                await authStateProvider.Logout();
-
-                if (!string.IsNullOrWhiteSpace(_redirectPath)) {
-                    navigationManager.NavigateTo($"login/{_redirectPath}");
-                } else {
-                    navigationManager.NavigateTo("login");
-                }
-            }
-        } catch (Exception ex) {
-            await authStateProvider.Logout();
-
-            if (!string.IsNullOrWhiteSpace(_redirectPath)) {
-                navigationManager.NavigateTo($"login/{_redirectPath}");
-            } else {
-                navigationManager.NavigateTo("login");
-            }
-        }
-    }
-}

MesaFabApproval.Client/Pages/Components/MRBActionForm.razor

@@ -215,7 +215,9 @@
         bool actionIsValid = mrbAction.Action.Equals("Block") || mrbAction.Action.Equals("Convert") ||
             mrbAction.Action.Equals("Other") || mrbAction.Action.Equals("Recall") || mrbAction.Action.Equals("Scrap") ||
             mrbAction.Action.Equals("Unblock") || mrbAction.Action.Equals("Waiver");
-        actionIsValid = actionIsValid && !string.IsNullOrWhiteSpace(mrbAction.LotNumber);
+        actionIsValid = actionIsValid && !string.IsNullOrWhiteSpace(mrbAction.Customer) &&
+            !string.IsNullOrWhiteSpace(mrbAction.PartNumber) &&
+            !string.IsNullOrWhiteSpace(mrbAction.LotNumber);
         actionIsValid = actionIsValid && mrbAction.Quantity > 0;
 
         if (mrbAction.Action.Equals("Convert", StringComparison.InvariantCultureIgnoreCase)) {
@@ -223,9 +225,6 @@
                 !string.IsNullOrWhiteSpace(convertFromPart) &&
                 !string.IsNullOrWhiteSpace(convertToCustomer) &&
                 !string.IsNullOrWhiteSpace(convertToPart);
-        } else {
-            actionIsValid = actionIsValid && !string.IsNullOrWhiteSpace(mrbAction.Customer) &&
-            !string.IsNullOrWhiteSpace(mrbAction.PartNumber);
         }
 
         if (mrbAction.Action.Equals("Scrap", StringComparison.InvariantCultureIgnoreCase))

MesaFabApproval.Client/Pages/Components/PCR3DocumentForm.razor

@@ -31,16 +31,16 @@
                                   Immediate
                                   AutoGrow />
                 } else {
-                    <MudAutocomplete @bind-Value="@document.ECNNumber"
-                                     T="int"
-                                     SearchFunc="Search"
-                                     Required
-                                     Clearable
-                                     RequiredError="You must provide a valid ECN#"
-                                     Variant="Variant.Outlined"
-                                     Validation="@(new Func<int, Task<string>>(ECNNoIsValid))"
-                                     Label="ECN#"
-                                     Immediate />
+                    <MudTextField @bind-Value="@document.ECNNumber"
+                                  Required
+                                  RequiredError="You must provide a valid ECN#"
+                                  Clearable
+                                  Variant="Variant.Outlined"
+                                  InputType="@InputType.Number"
+                                  Validation="@(new Func<int, Task<string>>(ECNNoIsValid))"
+                                  Label="ECN#"
+                                  Immediate
+                                  AutoGrow />
                 }
                 <MudCheckBox Label="Complete"
                              Color="Color.Tertiary"
@@ -79,8 +79,6 @@
     [Parameter]
     public required PCR3Document document { get; set; }
 
-    private IEnumerable<int> allEcnNumbers = new List<int>();
-
     private string[] errors = { };
 
     private bool complete = false;
@@ -91,8 +89,6 @@
 
     protected override async Task OnParametersSetAsync() {
         complete = document.CompletedByID > 0;
-
-        allEcnNumbers = await ecnService.GetAllECNNumbers();
     }
 
     private async Task Save() {
@@ -147,8 +143,6 @@
             document.DocNumbers.ToLower().Equals("n/a") ||
             document.DocNumbers.ToLower().Equals("n a") ||
             document.DocNumbers.ToLower().Equals("not applicable")) {
-            if (string.IsNullOrWhiteSpace(document.Comment))
-                document.Comment = "Not required";
             return true;
         }
         return false;
@@ -161,13 +155,4 @@
         StateHasChanged();
         return result;
     }
-
-    private async Task<IEnumerable<int>> Search(string searchValue, CancellationToken token) {
-        if (string.IsNullOrWhiteSpace(searchValue))
-            return allEcnNumbers;
-
-        return allEcnNumbers
-                .Where(x => x.ToString().StartsWith(searchValue, StringComparison.InvariantCultureIgnoreCase))
-                .Order();
-    }
 }

MesaFabApproval.Client/Pages/Dashboard.razor

@@ -1,5 +1,16 @@
 @page "/"
 @page "/Dashboard"
+@inject IConfiguration Configuration
+@inject MesaFabApprovalAuthStateProvider stateProvider
+@inject IApprovalService approvalService
+@inject IMemoryCache cache
+@inject NavigationManager navigationManager
+@inject ISnackbar snackbar
+@inject IMRBService mrbService
+@inject IPCRBService pcrbService
+@inject IECNService ecnService
+@inject ICAService caService
+@inject IJSRuntime jsRuntime
 
 <PageTitle>Dashboard</PageTitle>
 
@@ -215,3 +226,138 @@
         </MudTabPanel>
     </MudTabs>
 </MudPaper>
+
+@code {
+    private IEnumerable<Approval> approvalList = new List<Approval>();
+    private IEnumerable<MRB> myMRBs = new List<MRB>();
+    private IEnumerable<PCRB> myPCRBs = new List<PCRB>();
+
+    private IEnumerable<int> ecnNumbers = new HashSet<int>();
+    private IEnumerable<int> caNumbers = new HashSet<int>();
+    private IEnumerable<int> mrbNumbers = new HashSet<int>();
+    private IEnumerable<int> pcrbNumbers = new HashSet<int>();
+
+    private bool myApprovalsProcessing = false;
+    private bool myMrbsProcessing = false;
+    private bool myPcrbsProcessing = false;
+
+    private string mrbSearchString = "";
+    private string pcrbSearchString = "";
+
+    protected async override Task OnParametersSetAsync() {
+        try {
+            if (stateProvider.CurrentUser is not null) {
+                myApprovalsProcessing = true;
+                approvalList = (await approvalService.GetApprovalsForUserId(stateProvider.CurrentUser.UserID, true))
+                    .Where(a => a.CompletedDate > DateTime.Now && a.ItemStatus == 0)
+                    .ToList()
+                    .OrderByDescending(x => x.AssignedDate);
+                myApprovalsProcessing = false;
+
+                myMrbsProcessing = true;
+                myMRBs = (await mrbService.GetAllMRBs(false)).Where(m => m.OriginatorID == stateProvider.CurrentUser.UserID)
+                    .ToList()
+                    .OrderByDescending(x => x.SubmittedDate);
+                myMrbsProcessing = false;
+
+                myPcrbsProcessing = true;
+                myPCRBs = (await pcrbService.GetAllPCRBs(false)).Where(p => p.OwnerID == stateProvider.CurrentUser.UserID)
+                    .ToList()
+                    .OrderByDescending(p => p.InsertTimeStamp);
+                myPcrbsProcessing = false;
+            }
+        } catch (Exception ex) {
+            myApprovalsProcessing = false;
+            myMrbsProcessing = false;
+            myPcrbsProcessing = false;
+            snackbar.Add($"Unable to load the dashboard, because {ex.Message}", Severity.Error);
+        }
+    }
+
+    private async Task FollowLink(int issueId) {
+        HashSet<Task> tasks = new();
+
+        bool isEcn = false;
+        bool isCa = false;
+        bool isMrb = false;
+        bool isPcrb = false;
+        if (ecnNumbers.Contains(issueId))
+            isEcn = true;
+        if (caNumbers.Contains(issueId))
+            isCa = true;
+        if (mrbNumbers.Contains(issueId))
+            isMrb = true;
+        if (pcrbNumbers.Contains(issueId))
+            isPcrb = true;
+
+        if (!isEcn && !isCa && !isMrb) {
+            Task<bool> isEcnTask = ecnService.ECNNumberIsValid(issueId);
+            tasks.Add(isEcnTask);
+
+            Task<bool> isCaTask = caService.CANumberIsValid(issueId);
+            tasks.Add(isCaTask);
+
+            Task<bool> isMrbTask = mrbService.NumberIsValid(issueId);
+            tasks.Add(isMrbTask);
+
+            Task<bool> isPcrbTask = pcrbService.IdIsValid(issueId);
+            tasks.Add(isPcrbTask);
+
+            await Task.WhenAll(tasks);
+
+            if (isEcnTask.Result) {
+                isEcn = true;
+            } else if (isCaTask.Result) {
+                isCa = true;
+            } else if (isMrbTask.Result) {
+                isMrb = true;
+            } else if (isPcrbTask.Result) {
+                isPcrb = true;
+            }
+        }
+
+        if (isEcn) await GoToExternal($"{Configuration["OldFabApprovalUrl"]}/ECN/Edit?IssueID={issueId}", "");
+        if (isCa) await GoToExternal($"{Configuration["OldFabApprovalUrl"]}/CorrectiveAction/Edit?IssueID={issueId}", "");
+        if (isMrb) GoTo($"mrb/{issueId}");
+        if (isPcrb) GoTo($"pcrb/{issueId}");
+    }
+
+    private void GoTo(string page) {
+        cache.Set("redirectUrl", page);
+        navigationManager.NavigateTo("/" + page);
+    }
+
+    private async Task GoToExternal(string url, string content) {
+        IJSObjectReference windowModule = await jsRuntime.InvokeAsync<IJSObjectReference>("import", "./js/OpenInNewWindow.js");
+        await windowModule.InvokeAsync<object>("OpenInNewWindow", url, content);
+    }
+
+    private bool FilterFuncForMRBTable(MRB mrb) => MRBFilterFunc(mrb, mrbSearchString);
+
+    private bool MRBFilterFunc(MRB mrb, string searchString) {
+        if (string.IsNullOrWhiteSpace(searchString))
+            return true;
+        if (mrb.Title.ToLower().Contains(searchString.Trim().ToLower()))
+            return true;
+        if (mrb.MRBNumber.ToString().Contains(searchString.Trim()))
+            return true;
+        return false;
+    }
+
+    private bool FilterFuncForPCRBTable(PCRB pcrb) => PCRBFilterFunc(pcrb, pcrbSearchString);
+
+    private bool PCRBFilterFunc(PCRB pcrb, string searchString) {
+        if (string.IsNullOrWhiteSpace(searchString))
+            return true;
+        if (pcrb.Title.ToLower().Contains(searchString.Trim().ToLower()))
+            return true;
+        if (pcrb.PlanNumber.ToString().Contains(searchString.Trim()))
+            return true;
+        return false;
+    }
+
+    private string GetCurrentPCRBStep(int step) {
+        if (step < 0 || step > (PCRB.Stages.Length - 1)) return string.Empty;
+        else return PCRB.Stages[step];
+    }
+}

MesaFabApproval.Client/Pages/Dashboard.razor.cs

@@ -1,157 +0,0 @@
-using MesaFabApproval.Client.Services;
-using MesaFabApproval.Shared.Models;
-
-using Microsoft.AspNetCore.Components;
-using Microsoft.Extensions.Caching.Memory;
-using Microsoft.JSInterop;
-
-using MudBlazor;
-
-namespace MesaFabApproval.Client.Pages;
-
-public partial class Dashboard {
-    [Inject] IConfiguration Configuration { get; set; }
-    [Inject] MesaFabApprovalAuthStateProvider stateProvider { get; set; }
-    [Inject] IApprovalService approvalService { get; set; }
-    [Inject] IMemoryCache cache { get; set; }
-    [Inject] NavigationManager navigationManager { get; set; }
-    [Inject] ISnackbar snackbar { get; set; }
-    [Inject] IMRBService mrbService { get; set; }
-    [Inject] IPCRBService pcrbService { get; set; }
-    [Inject] IECNService ecnService { get; set; }
-    [Inject] ICAService caService { get; set; }
-    [Inject] IJSRuntime jsRuntime { get; set; }
-
-    private IEnumerable<Approval> approvalList = new List<Approval>();
-    private IEnumerable<MRB> myMRBs = new List<MRB>();
-    private IEnumerable<PCRB> myPCRBs = new List<PCRB>();
-
-    private IEnumerable<int> ecnNumbers = new HashSet<int>();
-    private IEnumerable<int> caNumbers = new HashSet<int>();
-    private IEnumerable<int> mrbNumbers = new HashSet<int>();
-    private IEnumerable<int> pcrbNumbers = new HashSet<int>();
-
-    private bool myApprovalsProcessing = false;
-    private bool myMrbsProcessing = false;
-    private bool myPcrbsProcessing = false;
-
-    private string mrbSearchString = "";
-    private string pcrbSearchString = "";
-
-    protected async override Task OnParametersSetAsync() {
-        try {
-            if (stateProvider.CurrentUser is not null) {
-                myApprovalsProcessing = true;
-                approvalList = (await approvalService.GetApprovalsForUserId(stateProvider.CurrentUser.UserID, true))
-                    .Where(a => a.CompletedDate > DateTime.Now && a.ItemStatus == 0)
-                    .ToList()
-                    .OrderByDescending(x => x.AssignedDate);
-                myApprovalsProcessing = false;
-
-                myMrbsProcessing = true;
-                myMRBs = (await mrbService.GetAllMRBs(false)).Where(m => m.OriginatorID == stateProvider.CurrentUser.UserID)
-                    .ToList()
-                    .OrderByDescending(x => x.SubmittedDate);
-                myMrbsProcessing = false;
-
-                myPcrbsProcessing = true;
-                myPCRBs = (await pcrbService.GetAllPCRBs(false)).Where(p => p.OwnerID == stateProvider.CurrentUser.UserID)
-                    .ToList()
-                    .OrderByDescending(p => p.InsertTimeStamp);
-                myPcrbsProcessing = false;
-            }
-        } catch (Exception ex) {
-            myApprovalsProcessing = false;
-            myMrbsProcessing = false;
-            myPcrbsProcessing = false;
-            snackbar.Add(ex.Message, Severity.Error);
-        }
-    }
-
-    private async Task FollowLink(int issueId) {
-        HashSet<Task> tasks = new();
-
-        bool isEcn = false;
-        bool isCa = false;
-        bool isMrb = false;
-        bool isPcrb = false;
-        if (ecnNumbers.Contains(issueId))
-            isEcn = true;
-        if (caNumbers.Contains(issueId))
-            isCa = true;
-        if (mrbNumbers.Contains(issueId))
-            isMrb = true;
-        if (pcrbNumbers.Contains(issueId))
-            isPcrb = true;
-
-        if (!isEcn && !isCa && !isMrb) {
-            Task<bool> isEcnTask = ecnService.ECNNumberIsValid(issueId);
-            tasks.Add(isEcnTask);
-
-            Task<bool> isCaTask = caService.CANumberIsValid(issueId);
-            tasks.Add(isCaTask);
-
-            Task<bool> isMrbTask = mrbService.NumberIsValid(issueId);
-            tasks.Add(isMrbTask);
-
-            Task<bool> isPcrbTask = pcrbService.IdIsValid(issueId);
-            tasks.Add(isPcrbTask);
-
-            await Task.WhenAll(tasks);
-
-            if (isEcnTask.Result) {
-                isEcn = true;
-            } else if (isCaTask.Result) {
-                isCa = true;
-            } else if (isMrbTask.Result) {
-                isMrb = true;
-            } else if (isPcrbTask.Result) {
-                isPcrb = true;
-            }
-        }
-
-        if (isEcn) await GoToExternal($"{Configuration["OldFabApprovalUrl"]}/ECN/Edit?IssueID={issueId}", "");
-        if (isCa) await GoToExternal($"{Configuration["OldFabApprovalUrl"]}/CorrectiveAction/Edit?IssueID={issueId}", "");
-        if (isMrb) GoTo($"mrb/{issueId}");
-        if (isPcrb) GoTo($"pcrb/{issueId}");
-    }
-
-    private void GoTo(string page) {
-        cache.Set("redirectUrl", page);
-        navigationManager.NavigateTo("/" + page);
-    }
-
-    private async Task GoToExternal(string url, string content) {
-        IJSObjectReference windowModule = await jsRuntime.InvokeAsync<IJSObjectReference>("import", "./js/OpenInNewWindow.js");
-        await windowModule.InvokeAsync<object>("OpenInNewWindow", url, content);
-    }
-
-    private bool FilterFuncForMRBTable(MRB mrb) => MRBFilterFunc(mrb, mrbSearchString);
-
-    private bool MRBFilterFunc(MRB mrb, string searchString) {
-        if (string.IsNullOrWhiteSpace(searchString))
-            return true;
-        if (mrb.Title.ToLower().Contains(searchString.Trim().ToLower()))
-            return true;
-        if (mrb.MRBNumber.ToString().Contains(searchString.Trim()))
-            return true;
-        return false;
-    }
-
-    private bool FilterFuncForPCRBTable(PCRB pcrb) => PCRBFilterFunc(pcrb, pcrbSearchString);
-
-    private bool PCRBFilterFunc(PCRB pcrb, string searchString) {
-        if (string.IsNullOrWhiteSpace(searchString))
-            return true;
-        if (pcrb.Title.ToLower().Contains(searchString.Trim().ToLower()))
-            return true;
-        if (pcrb.PlanNumber.ToString().Contains(searchString.Trim()))
-            return true;
-        return false;
-    }
-
-    private string GetCurrentPCRBStep(int step) {
-        if (step < 0 || step > (PCRB.Stages.Length - 1)) return string.Empty;
-        else return PCRB.Stages[step];
-    }
-}

MesaFabApproval.Client/Services/ECNService.cs

@@ -5,7 +5,6 @@ namespace MesaFabApproval.Client.Services;
 public interface IECNService {
     Task<string> ECNNumberIsValidStr(int ecnNumber);
     Task<bool> ECNNumberIsValid(int number);
-    Task<IEnumerable<int>> GetAllECNNumbers();
 }
 
 public class ECNService : IECNService {
@@ -50,32 +49,4 @@ public class ECNService : IECNService {
             throw new Exception($"Unable to determine if {number} is a valid ECN#, because {ex.Message}");
         }
     }
-
-    public async Task<IEnumerable<int>> GetAllECNNumbers() {
-        try {
-            HttpClient httpClient = _httpClientFactory.CreateClient("API");
-
-            HttpRequestMessage requestMessage = new(HttpMethod.Get, $"ecn/allNumbers");
-
-            HttpResponseMessage responseMessage = await httpClient.SendAsync(requestMessage);
-
-            if (responseMessage.IsSuccessStatusCode) {
-                string responseContent = await responseMessage.Content.ReadAsStringAsync();
-
-                JsonSerializerOptions jsonSerializerOptions = new() {
-                    PropertyNameCaseInsensitive = true
-                };
-
-                IEnumerable<int> allEcnNumbers =
-                    JsonSerializer.Deserialize<IEnumerable<int>>(responseContent, jsonSerializerOptions) ??
-                    new List<int>();
-
-                return allEcnNumbers;
-            } else {
-                throw new Exception(responseMessage.ReasonPhrase);
-            }
-        } catch (Exception ex) {
-            throw new Exception($"Unable to get all ECN#s, because {ex.Message}");
-        }
-    }
 }

MesaFabApproval.Client/Services/PCRBService.cs

@@ -4,6 +4,7 @@ using System.Text.Json;
 
 using MesaFabApproval.Shared.Models;
 
+using Microsoft.AspNetCore.Components.Forms;
 using Microsoft.AspNetCore.StaticFiles;
 using Microsoft.Extensions.Caching.Memory;
 
@@ -39,10 +40,6 @@ public interface IPCRBService {
     Task NotifyApprovers(PCRBNotification notification);
     Task NotifyOriginator(PCRBNotification notification);
     Task NotifyResponsiblePerson(PCRBActionItemNotification notification);
-    Task CreateFollowUp(PCRBFollowUp followUp);
-    Task<IEnumerable<PCRBFollowUp>> GetFollowUpsByPlanNumber(int planNumber, bool bypassCache);
-    Task UpdateFollowUp(PCRBFollowUp followUp);
-    Task DeleteFollowUp(int id);
 }
 
 public class PCRBService : IPCRBService {
@@ -767,86 +764,4 @@ public class PCRBService : IPCRBService {
         if (!responseMessage.IsSuccessStatusCode)
             throw new Exception($"Unable to notify PCRB responsible person, because {responseMessage.ReasonPhrase}");
     }
-
-    public async Task CreateFollowUp(PCRBFollowUp followUp) {
-        if (followUp is null) throw new ArgumentNullException("follow up cannot be null");
-
-        HttpClient httpClient = _httpClientFactory.CreateClient("API");
-
-        HttpRequestMessage requestMessage = new(HttpMethod.Post, $"pcrb/followUp") {
-            Content = new StringContent(JsonSerializer.Serialize(followUp),
-                                               Encoding.UTF8,
-                                               "application/json")
-        };
-
-        HttpResponseMessage responseMessage = await httpClient.SendAsync(requestMessage);
-
-        if (!responseMessage.IsSuccessStatusCode)
-            throw new Exception(responseMessage.ReasonPhrase);
-
-        await GetFollowUpsByPlanNumber(followUp.PlanNumber, true);
-    }
-
-    public async Task<IEnumerable<PCRBFollowUp>> GetFollowUpsByPlanNumber(int planNumber, bool bypassCache) {
-        if (planNumber <= 0) throw new ArgumentException($"{planNumber} is not a valid PCRB Plan#");
-
-        IEnumerable<PCRBFollowUp>? followUps = null;
-        if (!bypassCache)
-            followUps = _cache.Get<IEnumerable<PCRBFollowUp>>($"pcrbFollowUps{planNumber}");
-
-        if (followUps is null) {
-            HttpClient httpClient = _httpClientFactory.CreateClient("API");
-
-            HttpRequestMessage requestMessage = new(HttpMethod.Get, $"pcrb/followUps?planNumber={planNumber}&bypassCache={bypassCache}");
-
-            HttpResponseMessage responseMessage = await httpClient.SendAsync(requestMessage);
-
-            if (responseMessage.IsSuccessStatusCode) {
-                string responseContent = await responseMessage.Content.ReadAsStringAsync();
-
-                JsonSerializerOptions jsonSerializerOptions = new() {
-                    PropertyNameCaseInsensitive = true
-                };
-
-                followUps = JsonSerializer.Deserialize<IEnumerable<PCRBFollowUp>>(responseContent, jsonSerializerOptions) ??
-                    new List<PCRBFollowUp>();
-
-                if (followUps.Count() > 0)
-                    _cache.Set($"pcrbFollowUps{planNumber}", followUps, DateTimeOffset.Now.AddMinutes(5));
-            } else {
-                throw new Exception(responseMessage.ReasonPhrase);
-            }
-        }
-
-        return followUps;
-    }
-    
-    public async Task UpdateFollowUp(PCRBFollowUp followUp) {
-        if (followUp is null) throw new ArgumentNullException("follow up cannot be null");
-
-        HttpClient httpClient = _httpClientFactory.CreateClient("API");
-
-        HttpRequestMessage requestMessage = new(HttpMethod.Put, $"pcrb/followUp") {
-            Content = new StringContent(JsonSerializer.Serialize(followUp),
-                                                   Encoding.UTF8,
-                                                   "application/json")
-        };
-
-        HttpResponseMessage responseMessage = await httpClient.SendAsync(requestMessage);
-
-        if (!responseMessage.IsSuccessStatusCode)
-            throw new Exception(responseMessage.ReasonPhrase);
-    }
-
-    public async Task DeleteFollowUp(int id) {
-        if (id <= 0) throw new ArgumentException($"{id} is not a valid PCRB follow up ID");
-
-        HttpClient httpClient = _httpClientFactory.CreateClient("API");
-
-        HttpRequestMessage requestMessage = new(HttpMethod.Delete, $"pcrb/followUp?id={id}");
-
-        HttpResponseMessage responseMessage = await httpClient.SendAsync(requestMessage);
-
-        if (!responseMessage.IsSuccessStatusCode) throw new Exception(responseMessage.ReasonPhrase);
-    }
 }

MesaFabApproval.Client/pipeline.yml

@@ -1,100 +0,0 @@
-trigger:
-  branches:
-    include:
-      - master
-  paths:
-    include:
-      - MesaFabApproval.Client
-      - MesaFabApproval.Shared
-
-variables:
-  buildConfiguration: "Release"
-
-stages:
-  - stage: Test
-    displayName: "Test"
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem-Dev
-    jobs:
-      - job: BuildPublish
-        steps:
-          - script: |
-              echo build configuration: $(BuildConfiguration)
-            displayName: "Echo Check"
-
-          - task: DotNetCoreCLI@2
-            displayName: "Build"
-            inputs:
-              command: "build"
-              configuration: $(BuildConfiguration)
-              projects: MesaFabApproval.Client
-
-          - task: DotNetCoreCLI@2
-            displayName: "Test"
-            inputs:
-              command: "test"
-              configuration: $(BuildConfiguration)
-              publishTestResults: true
-              projects: MesaFabApproval.Client.Test
-
-          - task: DotNetCoreCLI@2
-            displayName: "Publish"
-            inputs:
-              command: "publish"
-              publishWebProjects: false
-              projects: MesaFabApproval.Client
-              zipAfterPublish: true
-              arguments: "-c $(BuildConfiguration) -o $(Build.ArtifactStagingDirectory) -r win-x64 --self-contained"
-
-          - task: CopyFiles@2
-            displayName: "Copy Files"
-            inputs:
-              SourceFolder: "$(Build.ArtifactStagingDirectory)"
-              TargetFolder: 'D:\$(Build.Repository.Name)\Client\$(Build.BuildId)\$(BuildConfiguration)'
-              CleanTargetFolder: true
-              retryCount: "3"
-
-  - stage: Prod
-    displayName: "Prod"
-    pool:
-      name: MesaFabApproval
-      demands: Fab2ApprovalSystem
-    jobs:
-      - job: BuildPublish
-        steps:
-          - script: |
-              echo build configuration: $(BuildConfiguration)
-            displayName: "Echo Check"
-
-          - task: DotNetCoreCLI@2
-            displayName: "Build"
-            inputs:
-              command: "build"
-              configuration: $(BuildConfiguration)
-              projects: MesaFabApproval.Client
-
-          - task: DotNetCoreCLI@2
-            displayName: "Test"
-            inputs:
-              command: "test"
-              configuration: $(BuildConfiguration)
-              publishTestResults: true
-              projects: MesaFabApproval.Client.Test
-
-          - task: DotNetCoreCLI@2
-            displayName: "Publish"
-            inputs:
-              command: "publish"
-              publishWebProjects: false
-              projects: MesaFabApproval.Client
-              zipAfterPublish: true
-              arguments: "-c $(BuildConfiguration) -o $(Build.ArtifactStagingDirectory) -r win-x64 --self-contained"
-
-          - task: CopyFiles@2
-            displayName: "Copy Files"
-            inputs:
-              SourceFolder: "$(Build.ArtifactStagingDirectory)"
-              TargetFolder: 'D:\$(Build.Repository.Name)\Client\$(Build.BuildId)\$(BuildConfiguration)'
-              CleanTargetFolder: true
-              retryCount: "3"

MesaFabApproval.Shared/Models/PCRB.cs

@@ -23,5 +23,4 @@ public class PCRB {
     public DateTime InsertTimeStamp { get; set; } = DateTimeUtilities.MIN_DT;
     public DateTime LastUpdateDate { get; set; } = DateTimeUtilities.MIN_DT;
     public DateTime ClosedDate { get; set; } = DateTimeUtilities.MAX_DT;
-    public string Type { get; set; } = "";
 }

MesaFabApproval.Shared/Models/PCRBFollowUp.cs

@@ -1,14 +0,0 @@
-using MesaFabApproval.Shared.Utilities;
-
-namespace MesaFabApproval.Shared.Models;
-
-public class PCRBFollowUp {
-    public int ID { get; set; }
-    public required int PlanNumber { get; set; }
-    public required int Step { get; set; }
-    public required DateTime FollowUpDate { get; set; }
-    public bool IsComplete { get; set; } = false;
-    public bool IsDeleted { get; set; } = false;
-    public DateTime CompletedDate { get; set; } = DateTimeUtilities.MAX_DT;
-    public string Comments { get; set; } = string.Empty;
-}

azure-pipelines.yml

@@ -0,0 +1,114 @@
+trigger:
+  branches:
+    include:
+      - master
+  paths:
+    include:
+      - "Fab2ApprovalSystem/*"
+    exclude:
+      - "**/*.yaml"
+      - "**/*.yml"
+      - "SQL/*"
+      - "references/*"
+      - "packages/*"
+      - "Kendo/*"
+
+variables:
+  buildConfiguration: "Release"
+  targetFrameworkVersion: "v4.8"
+  coreVersion: "na"
+  assemblyTitle: "Fab2ApprovalSystem"
+  architecture: "x64"
+  msBuild: "C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/MSBuild/Current/Bin/MSBuild.exe"
+
+stages:
+  - stage: Development
+    displayName: Development
+    pool:
+      name: Mesa-FabApproval
+      demands: Fab2ApprovalSystem-Dev
+    variables:
+      ASPNETCORE_ENVIRONMENT: "Development"
+      configuration: "Debug"
+    jobs:
+      - job: Debug
+        steps:
+          - script: |
+              set gitCommit=$(Build.SourceVersion)
+              set gitCommitSeven=%gitCommit:~0,7%
+              echo %gitCommitSeven%
+              echo ##vso[task.setvariable variable=GitCommitSeven;]%gitCommitSeven%
+              echo $(GitCommitSeven)
+            displayName: GitCommitSeven
+
+          - script: |
+              echo BuildId: $(Build.BuildId)
+              echo Build reason: $(Build.Reason)
+              echo Repo Id: $(Build.Repository.Id)
+              echo Repo name: $(Build.Repository.Name)
+              echo Source version: $(Build.SourceVersion)
+              echo Core version: $(CoreVersion)
+              echo Build configuration: $(BuildConfiguration)
+              echo Configuration: $(Configuration)
+              echo Target Framework version: $(TargetFrameworkVersion)
+              echo Assembly title: $(AssemblyTitle)
+              echo MSBuild: $(msBuild)
+            displayName: "Echo Check"
+
+          - script: |
+              mklink /J ".vscode\.UserSecrets" "%AppData%\Microsoft\UserSecrets\f2da5035-aba9-4676-9f8d-d6689f84663d"
+              mklink /J "DMO" "..\Fab2ApprovalSystem\DMO"
+              mklink /J "Jobs" "..\Fab2ApprovalSystem\Jobs"
+              mklink /J "JobSchedules" "..\Fab2ApprovalSystem\JobSchedules"
+              mklink /J "Misc" "..\Fab2ApprovalSystem\Misc"
+              mklink /J "Models" "..\Fab2ApprovalSystem\Models"
+              mklink /J "PdfGenerator" "..\Fab2ApprovalSystem\PdfGenerator"
+              mklink /J "Utilities" "..\Fab2ApprovalSystem\Utilities"
+              mklink /J "ViewModels" "..\Fab2ApprovalSystem\ViewModels"
+            workingDirectory: Fab2ApprovalMKLink
+            displayName: "MKLink - Symbolic Link of Type Junction"
+
+          - script: |
+              dotnet user-secrets init
+              dotnet user-secrets set BuildNumber $(Build.BuildId)
+              dotnet user-secrets set GitCommitSeven $(GitCommitSeven)
+              dotnet user-secrets list
+            workingDirectory: Fab2ApprovalMKLink
+            displayName: "MKLink - Safe storage of app secrets"
+
+          - script: dotnet build --configuration $(buildConfiguration)
+            workingDirectory: Fab2ApprovalMKLink
+            displayName: "MKLink - Build"
+
+          - script: dotnet build --configuration $(buildConfiguration)
+            workingDirectory: Fab2ApprovalTests
+            displayName: "Tests - Build"
+
+          - script: dotnet test --configuration $(buildConfiguration)
+            workingDirectory: Fab2ApprovalTests
+            displayName: "Tests - Test"
+
+          - script: dotnet clean --configuration $(buildConfiguration)
+            workingDirectory: Fab2ApprovalTests
+            displayName: "Tests - Clean"
+
+          - script: dotnet clean --configuration $(buildConfiguration)
+            workingDirectory: Fab2ApprovalMKLink
+            displayName: "MKLink - Clean"
+
+          - script: echo $(Build.SourceVersion)-$(Build.BuildId)>bin_x_x_\$(Configuration)\$(CoreVersion)\win-x64\$(Build.Repository.Name).txt
+            workingDirectory: Fab2ApprovalMKLink
+            displayName: "Force Fail"
+            enabled: "false"
+
+          - script: '"$(msBuild)" /target:Restore /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(BuildConfiguration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:RestoreSources=D:/nupkg $(AssemblyTitle).csproj'
+            workingDirectory: Fab2ApprovalSystem
+            displayName: "Framework - Restore"
+
+          - script: '"$(msBuild)" /target:Build /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(BuildConfiguration);TargetFrameworkVersion=$(TargetFrameworkVersion) $(AssemblyTitle).csproj'
+            workingDirectory: Fab2ApprovalSystem
+            displayName: "Framework - Build"
+
+          - script: '"$(msBuild)" /DetailedSummary /ConsoleLoggerParameters:PerformanceSummary;ErrorsOnly; /p:Configuration=$(Configuration);TargetFrameworkVersion=$(TargetFrameworkVersion) /p:DebugSymbols=false /p:DeleteExistingFiles=true /p:DeployOnBuild=true /p:EnableUpdateAble=true /p:ExcludeApp_Data=true /p:LastUsedBuildConfiguration=$(BuildConfiguration) /p:LastUsedPlatform="Any CPU" /p:LaunchSiteAfterPublish=true /p:OutputPath="D:\$(TargetFrameworkVersion)\$(Build.Repository.Name)\$(Build.BuildId)\$(Configuration)" /p:PreCompileBeforePublish=true /p:PublishProvider=FileSystem /p:PublishUrl="D:/PublishUrl" /p:SiteUrlToLaunchAfterPublish="" /p:WDPMergeOption=DoNotMerge /p:WebPublishMethod=FileSystem $(AssemblyTitle).csproj'
+            workingDirectory: Fab2ApprovalSystem
+            displayName: "Framework - Pack"