review changes

Altinn · Sep 7, 2023 · ef60a41 · ef60a41
1 parent bc4594d
commit ef60a41
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/Altinn.App.Core/Models/Layout/Components/GridComponent.cs b/src/Altinn.App.Core/Models/Layout/Components/GridComponent.cs
@@ -18,7 +18,9 @@ public GridComponent(string id, string type, IReadOnlyDictionary<string, string>
 }
 
 
-/// <inheritdoc/>
+/// <summary>
+/// Class for parsing a Grid component's rows and cells and extracting the child component IDs
+/// </summary>
 public class GridConfig
 {
 

diff --git a/src/Altinn.App.Core/Models/Layout/Components/PageComponent.cs b/src/Altinn.App.Core/Models/Layout/Components/PageComponent.cs
@@ -25,10 +25,10 @@ public PageComponent(string id, List<BaseComponent> children, Dictionary<string,
     public Dictionary<string, BaseComponent> ComponentLookup { get; }
 
     /// <summary>
-    /// AddChild is not needed for PageComponent
+    /// AddChild is not needed for PageComponent, and the base implementation would not work as intended.
     /// </summary>
     public override void AddChild(BaseComponent child)
     {
-        throw new NotImplementedException();
+
     }
 }