RE1-T91 RE1-T107 Fixing timers, voice number verification.

Author: ucswift

.claude/settings.local.json

@@ -1,16 +1,18 @@
 {
   "permissions": {
     "allow": [
-      "mcp__dual-graph__graph_scan",
       "mcp__dual-graph__graph_continue",
-      "Bash(find G:ResgridResgridCoreResgrid.ModelBilling -type f -name *.cs)",
-      "Bash(python3:*)",
+      "mcp__dual-graph__graph_read",
+      "Bash(dotnet build:*)",
+      "mcp__dual-graph__graph_register_edit",
+      "mcp__dual-graph__graph_scan",
+      "Bash(find /g/Resgrid/Resgrid -type d \\\\\\(-name *mobile* -o -name *Mobile* -o -name *app* -o -name *App* -o -name *apps* -o -name *Apps* \\\\\\))",
       "Bash(xargs grep:*)",
-      "Bash(grep -E \"\\\\.cs$|Program\")",
-      "Bash(grep -l \"DepartmentMembers\\\\|DepartmentMember\" \"G:\\\\Resgrid\\\\Resgrid/Repositories/Resgrid.Repositories.DataRepository/\"*.cs)",
-      "Bash(find /g/Resgrid/Resgrid/Repositories/Resgrid.Repositories.DataRepository/Servers -name *.cs)",
-      "Bash(find G:/Resgrid/Resgrid -newer G:/Resgrid/Resgrid/CLAUDE.md -name *.cs -not -path */obj/* -not -path */.git/*)",
-      "Bash(dotnet build:*)"
+      "Bash(grep -r \"mapbox\\\\|react-native\\\\|rnmapbox\" /g/Resgrid/Resgrid --include=package.json --include=*.ts --include=*.tsx)",
+      "Bash(find G:ResgridResgridWebResgrid.WebAreasUserViews -type f -name *.cshtml)",
+      "Bash(grep -r \"google.maps\\\\|mapboxgl\\\\|leaflet\\\\|openstreetmap\" G:/Resgrid/Resgrid/Web/Resgrid.Web/Areas/User/Apps/src --include=*.ts)",
+      "Bash(grep -r \"leaflet\\\\|L\\\\.tileLayer\\\\|OpenStreetMap\" /g/Resgrid/Resgrid/Web/Resgrid.Web.Services --include=*.cs)",
+      "Bash(find /g/Resgrid/Resgrid -type f -name *.swift -o -name *.kt -o -name *.java)"
     ]
   },
   "hooks": {

Core/Resgrid.Config/NumberProviderConfig.cs

@@ -15,6 +15,7 @@ public static class NumberProviderConfig
 		public static string TwilioResgridNumber = "";
 		public static string TwilioApiUrl = SystemBehaviorConfig.ResgridApiBaseUrl + "/api/Twilio/IncomingMessage";
 		public static string TwilioVoiceCallApiUrl = SystemBehaviorConfig.ResgridApiBaseUrl + "/api/Twilio/VoiceCall?userId={0}&callId={1}";
+		public static string TwilioVoiceVerificationApiUrl = SystemBehaviorConfig.ResgridApiBaseUrl + "/api/Twilio/VoiceVerification?userId={0}&contactType={1}";
 		public static string TwilioVoiceApiUrl = SystemBehaviorConfig.ResgridApiBaseUrl + "/api/Twilio/InboundVoice";
 
 		// Diafaan (https://www.diafaan.com)

Core/Resgrid.Localization/Areas/User/Home/EditProfile.en.resx

@@ -402,6 +402,9 @@
   <data name="VerificationCodeSent" xml:space="preserve">
     <value>A verification code has been sent.</value>
   </data>
+  <data name="VerificationCodeVoiceCallSent" xml:space="preserve">
+    <value>A verification call is being placed. Please answer and note the code spoken to you.</value>
+  </data>
   <data name="VerificationSuccessful" xml:space="preserve">
     <value>Verification successful.</value>
   </data>

Core/Resgrid.Model/PersonnelCallCheckInStatus.cs

@@ -0,0 +1,49 @@
+using System;
+
+namespace Resgrid.Model
+{
+	/// <summary>
+	/// Represents the check-in timer status for a single dispatched person on a call
+	/// that has personnel check-in timers enabled.
+	/// </summary>
+	public class PersonnelCallCheckInStatus
+	{
+		/// <summary>The ASP.NET Identity user identifier.</summary>
+		public string UserId { get; set; }
+
+		/// <summary>
+		/// The user's display name (first + last name from their profile).
+		/// May be null if the profile could not be resolved.
+		/// </summary>
+		public string FullName { get; set; }
+
+		/// <summary>UTC timestamp of the user's most-recent personnel check-in on this call, or null if never checked in.</summary>
+		public DateTime? LastCheckIn { get; set; }
+
+		/// <summary>True when the user must check in immediately (timer has expired).</summary>
+		public bool NeedsCheckIn { get; set; }
+
+		/// <summary>
+		/// Minutes remaining until the next check-in is required.
+		/// Positive = time still available, negative = how many minutes overdue.
+		/// Zero or negative implies <see cref="NeedsCheckIn"/> is true.
+		/// </summary>
+		public double MinutesRemaining { get; set; }
+
+		/// <summary>
+		/// Colour-coded status string: "Green" (within timer), "Warning" (within warning
+		/// threshold), or "Critical" (timer expired).
+		/// </summary>
+		public string Status { get; set; }
+
+		/// <summary>
+		/// The resolved duration (in minutes) of the personnel check-in timer for this call.
+		/// </summary>
+		public int DurationMinutes { get; set; }
+
+		/// <summary>
+		/// The warning threshold (in minutes) of the personnel check-in timer for this call.
+		/// </summary>
+		public int WarningThresholdMinutes { get; set; }
+	}
+}

Core/Resgrid.Model/Providers/IOutboundVoiceProvider.cs

@@ -5,5 +5,10 @@ namespace Resgrid.Model.Providers
 	public interface IOutboundVoiceProvider
 	{
 		Task<bool> CommunicateCallAsync(string phoneNumber, UserProfile profile, Call call);
+
+		/// <summary>
+		/// Places a Twilio voice call that speaks the verification code digits to the user.
+		/// </summary>
+		Task<bool> SendVoiceVerificationCallAsync(string phoneNumber, string userId, int contactType);
 	}
 }

Core/Resgrid.Model/Repositories/ICallsRepository.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
 
@@ -81,5 +81,15 @@ public interface ICallsRepository: IRepository<Call>
 		Task<IEnumerable<Call>> GetAllCallsByContactIdAsync(string contactId, int departmentId);
 
 		Task<int> GetCallsCountByDepartmentDateRangeAsync(int departmentId, DateTime startDate, DateTime endDate);
+
+		/// <summary>
+		/// Gets all active calls for a department that have check-in timers enabled
+		/// and that the specified user has been dispatched on.
+		/// Optimised as a single JOIN query to avoid N+1 lookups.
+		/// </summary>
+		/// <param name="userId">The identity user identifier to filter dispatches by.</param>
+		/// <param name="departmentId">The department identifier (used to scope the result).</param>
+		/// <returns>Active calls with check-in timers that the user is dispatched on.</returns>
+		Task<IEnumerable<Call>> GetActiveCallsWithCheckInTimersForUserAsync(string userId, int departmentId);
 	}
 }

Core/Resgrid.Model/Services/ICallsService.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;
@@ -441,5 +441,14 @@ Task<bool> ClearGroupForDispatchesAsync(int departmentGroupId,
 		Task<List<CallVideoFeed>> GetCallVideoFeedsByCallIdAsync(int callId);
 
 		Task<bool> DeleteCallVideoFeedAsync(CallVideoFeed feed, string deletedByUserId, CancellationToken cancellationToken = default(CancellationToken));
+
+		/// <summary>
+		/// Gets all active calls for a department that have personnel check-in timers
+		/// enabled and that the specified user has been dispatched on.
+		/// </summary>
+		/// <param name="userId">The ASP.NET Identity user identifier.</param>
+		/// <param name="departmentId">The department to scope the search to.</param>
+		/// <returns>Active calls with check-in timers dispatched to the user.</returns>
+		Task<List<Call>> GetActiveCallsWithCheckInTimersForUserAsync(string userId, int departmentId);
 	}
 }

Core/Resgrid.Model/Services/ICheckInTimerService.cs

@@ -26,5 +26,24 @@ public interface ICheckInTimerService
 
 		// Timer Status Computation
 		Task<List<CheckInTimerStatus>> GetActiveTimerStatusesForCallAsync(Call call);
+
+		// ── New: per-user and per-call personnel check-in status ────────────────
+
+		/// <summary>
+		/// Returns a check-in summary for every active call (with check-in timers
+		/// enabled) that <paramref name="userId"/> has been dispatched on.
+		/// Used by API Endpoint 1.
+		/// </summary>
+		/// <param name="userId">The ASP.NET Identity user identifier to query for.</param>
+		/// <param name="departmentId">Dept scope (from JWT claims) – prevents cross-dept data access.</param>
+		Task<List<UserCallCheckInSummary>> GetUserActiveCallCheckInSummariesAsync(string userId, int departmentId);
+
+		/// <summary>
+		/// For a call that has a personnel check-in timer active, returns the current
+		/// check-in status for every person dispatched on that call.
+		/// Used by API Endpoint 2.
+		/// </summary>
+		/// <param name="call">The call to evaluate. Must have <see cref="Call.CheckInTimersEnabled"/> = true.</param>
+		Task<List<PersonnelCallCheckInStatus>> GetCallPersonnelCheckInStatusesAsync(Call call);
 	}
 }

Core/Resgrid.Model/Services/IContactVerificationService.cs

@@ -25,9 +25,10 @@ public interface IContactVerificationService
 		Task<bool> SendMobileVerificationCodeAsync(string userId, int departmentId, string departmentNumber, CancellationToken cancellationToken = default);
 
 		/// <summary>
-		/// Generates a verification code and sends it via SMS to the user's home number.
+		/// Generates a verification code and delivers it via a Twilio voice call to the
+		/// user's home number, speaking the digits and repeating multiple times.
 		/// Returns <c>false</c> if the user has no home number, if rate limits are exceeded,
-		/// or if the send fails.
+		/// or if the call fails.
 		/// </summary>
 		Task<bool> SendHomeVerificationCodeAsync(string userId, int departmentId, string departmentNumber, CancellationToken cancellationToken = default);
 

Core/Resgrid.Model/UserCallCheckInSummary.cs

@@ -0,0 +1,57 @@
+using System;
+
+namespace Resgrid.Model
+{
+	/// <summary>
+	/// Represents the check-in timer status for a single user on a single active call
+	/// that has personnel check-in timers enabled.
+	/// </summary>
+	public class UserCallCheckInSummary
+	{
+		/// <summary>The call identifier.</summary>
+		public int CallId { get; set; }
+
+		/// <summary>The call name / nature of call.</summary>
+		public string CallName { get; set; }
+
+		/// <summary>The human-readable call number (e.g. "2024-0042").</summary>
+		public string CallNumber { get; set; }
+
+		/// <summary>UTC timestamp when the call was logged.</summary>
+		public DateTime CallStartedOn { get; set; }
+
+		/// <summary>True when a personnel-type check-in timer is active for this call.</summary>
+		public bool HasPersonnelTimer { get; set; }
+
+		/// <summary>
+		/// How long (in minutes) the user has before they must check in.
+		/// Only meaningful when <see cref="HasPersonnelTimer"/> is true.
+		/// </summary>
+		public int DurationMinutes { get; set; }
+
+		/// <summary>
+		/// Number of minutes before the deadline at which a "Warning" status is issued.
+		/// Only meaningful when <see cref="HasPersonnelTimer"/> is true.
+		/// </summary>
+		public int WarningThresholdMinutes { get; set; }
+
+		/// <summary>UTC timestamp of the user's most-recent check-in on this call, or null if never checked in.</summary>
+		public DateTime? LastCheckIn { get; set; }
+
+		/// <summary>True when the user must check in immediately (timer has expired).</summary>
+		public bool NeedsCheckIn { get; set; }
+
+		/// <summary>
+		/// Minutes remaining until the next check-in is required.
+		/// Positive = time still available, negative = how many minutes overdue.
+		/// Zero or negative implies <see cref="NeedsCheckIn"/> is true.
+		/// </summary>
+		public double MinutesRemaining { get; set; }
+
+		/// <summary>
+		/// Colour-coded status string: "Green", "Warning", "Critical", or "NoTimer"
+		/// when no personnel timer is configured for the call.
+		/// </summary>
+		public string Status { get; set; }
+	}
+}