minor fix

Signed-off-by: Lillie Dae <lillie.dae@answerdigital.com>

src/Api/Hl7ApplicationConfigEntity.cs

@@ -31,21 +31,21 @@ public class Hl7ApplicationConfigEntity : MongoDBEntityBase
         /// Gets or sets the sending identifier.
         /// </summary>
         [JsonProperty("sending_identifier")]
-        public KeyValuePair<string, string> SendingId { get; set; }
+        public StringKeyValuePair SendingId { get; set; } = new();
 
         /// <summary>
         /// Gets or sets the data link.
         /// Value is either PatientId or StudyInstanceUid
         /// </summary>
         [JsonProperty("data_link")]
-        public KeyValuePair<string, DataLinkType> DataLink { get; set; }
+        public DataKeyValuePair DataLink { get; set; } = new();
 
         /// <summary>
         /// Gets or sets the data mapping.
         /// Value is a DICOM Tag
         /// </summary>
         [JsonProperty("data_mapping")]
-        public Dictionary<string, string> DataMapping { get; set; } = new();
+        public List<StringKeyValuePair> DataMapping { get; set; } = new();
 
         public IEnumerable<string> Validate()
         {
@@ -95,6 +95,27 @@ public override string ToString()
         }
     }
 
+    //string key, string value
+    public class StringKeyValuePair : IKeyValuePair<string, string>
+    {
+        [Key]
+        public string Key { get; set; } = string.Empty;
+        public string Value { get; set; } = string.Empty;
+    }
+
+    public class DataKeyValuePair : IKeyValuePair<string, DataLinkType>
+    {
+        [Key]
+        public string Key { get; set; } = string.Empty;
+        public DataLinkType Value { get; set; }
+    }
+
+    public interface IKeyValuePair<TKey, TValue>
+    {
+        public TKey Key { get; set; }
+        public TValue Value { get; set; }
+    }
+
     public enum DataLinkType
     {
         PatientId,

src/Database/EntityFramework/Configuration/Hl7ApplicationConfigConfiguration.cs

@@ -25,11 +25,6 @@ internal class Hl7ApplicationConfigConfiguration : IEntityTypeConfiguration<Hl7A
         public void Configure(EntityTypeBuilder<Hl7ApplicationConfigEntity> builder)
         {
             builder.HasKey(j => j.Id);
-            builder.Property(j => j.DataLink).IsRequired();
-            builder.Property(j => j.DataMapping).IsRequired();
-            builder.Property(j => j.SendingId).IsRequired();
-
-            builder.Ignore(p => p.Id);
         }
     }
 }

src/Database/EntityFramework/Repositories/Hl7ApplicationConfigRepository.cs

@@ -54,7 +54,7 @@ public Hl7ApplicationConfigRepository(ILogger<Hl7ApplicationConfigRepository> lo
         }
 
         public Task<List<Hl7ApplicationConfigEntity>> GetAllAsync(CancellationToken cancellationToken = default) =>
-            _retryPolicy.ExecuteAsync(() => { return _dataset.ToListAsync(cancellationToken); });
+            _retryPolicy.ExecuteAsync(() => _dataset.ToListAsync(cancellationToken));
 
         public Task<Hl7ApplicationConfigEntity?> GetByIdAsync(string id) =>
             _retryPolicy.ExecuteAsync(() => _dataset.FirstOrDefaultAsync(x => x.Id.Equals(id)));