chore: remove nullable enable and add comments

src/Core/Auth/Models/Data/EmergencyAccessNotify.cs

@@ -1,14 +1,10 @@
-// FIXME: Update this file to be null safe and then delete the line below
-#nullable disable
-
-
-using Bit.Core.Auth.Entities;
+using Bit.Core.Auth.Entities;
 
 namespace Bit.Core.Auth.Models.Data;
 
 public class EmergencyAccessNotify : EmergencyAccess
 {
-    public string GrantorEmail { get; set; }
-    public string GranteeName { get; set; }
-    public string GranteeEmail { get; set; }
+    public string? GrantorEmail { get; set; }
+    public string? GranteeName { get; set; }
+    public string? GranteeEmail { get; set; }
 }

src/Core/Auth/Repositories/IEmergencyAccessRepository.cs

@@ -2,8 +2,6 @@
 using Bit.Core.Auth.Models.Data;
 using Bit.Core.KeyManagement.UserKey;
 
-#nullable enable
-
 namespace Bit.Core.Repositories;
 
 public interface IEmergencyAccessRepository : IRepository<EmergencyAccess, Guid>
@@ -11,7 +9,17 @@ public interface IEmergencyAccessRepository : IRepository<EmergencyAccess, Guid>
     Task<int> GetCountByGrantorIdEmailAsync(Guid grantorId, string email, bool onlyRegisteredUsers);
     Task<ICollection<EmergencyAccessDetails>> GetManyDetailsByGrantorIdAsync(Guid grantorId);
     Task<ICollection<EmergencyAccessDetails>> GetManyDetailsByGranteeIdAsync(Guid granteeId);
+    /// <summary>
+    /// Fetches emergency access details by EmergencyAccess id and grantor id
+    /// </summary>
+    /// <param name="id">Emergency Access Id</param>
+    /// <param name="grantorId">Grantor Id</param>
+    /// <returns>EmergencyAccessDetails or null</returns>
     Task<EmergencyAccessDetails?> GetDetailsByIdGrantorIdAsync(Guid id, Guid grantorId);
+    /// <summary>
+    /// Database call to fetch emergency accesses that need notification emails sent through a Job
+    /// </summary>
+    /// <returns>collection of EmergencyAccessNotify objects that require notification</returns>
     Task<ICollection<EmergencyAccessNotify>> GetManyToNotifyAsync();
     Task<ICollection<EmergencyAccessDetails>> GetExpiredRecoveriesAsync();
 

src/Infrastructure.EntityFramework/Auth/Repositories/EmergencyAccessRepository.cs

@@ -10,8 +10,6 @@ using Microsoft.Data.SqlClient;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.DependencyInjection;
 
-#nullable enable
-
 namespace Bit.Infrastructure.EntityFramework.Auth.Repositories;
 
 public class EmergencyAccessRepository : Repository<Core.Auth.Entities.EmergencyAccess, EmergencyAccess, Guid>, IEmergencyAccessRepository