Layout Groups #8

- Remove remaining backwards compatibility.

Author: BirjuVachhani

lib/src/data/data_manager.dart

@@ -1099,9 +1099,6 @@ class DataManager {
     // Process Variables
     if (model.updates.variables.containsKey(layoutID)) {
       log('\tLayout [$layoutID] has variables.');
-      // backward compatibility. Old layout structure. Layout contains only
-      // one canvas. So we need to fetch variables using layoutID.
-      // TODO: remove after migration
       try {
         final SDKLayoutVariables? variables =
             await getOrFetchVariables(layoutID);
@@ -1122,9 +1119,6 @@ class DataManager {
     // Process Conditions
     if (model.updates.conditions.containsKey(layoutID)) {
       log('\tLayout [$layoutID] has conditions.');
-      // backward compatibility. Old layout structure. Layout contains only
-      // one canvas. So we need to fetch conditions using layoutID.
-      // TODO: remove after migration
       try {
         final SDKLayoutConditions? conditions =
             await getOrFetchConditions(layoutID);

lib/src/model/sdk_publish_model.dart

@@ -184,7 +184,6 @@ class SDKPublishLayout extends PrivacyBase {
   /// A list of the nodes that makes up this layout.
   /// CanvasID -> <NodeID -> Node>
   @CanvasesMapConverter()
-  @JsonKey(readValue: nodesOrCanvasesReader)
   final Map<String, Map<String, BaseNode>> canvases;
 
   final List<Breakpoint> breakpoints;
@@ -266,17 +265,6 @@ class SDKPublishLayout extends PrivacyBase {
         lastUpdated,
         breakpoints,
       ];
-
-  // For backwards compatibility
-  static Map<String, dynamic> nodesOrCanvasesReader(
-      Map<dynamic, dynamic> json, String key) {
-    if (json.containsKey('nodes') && json.containsKey('canvasId')) {
-      // backwards compatibility
-      return {json['canvasId']: json['nodes']};
-    }
-    assert(json.containsKey('canvases'));
-    return json[key];
-  }
 }
 
 /// Represents a single variation of a common font.

lib/src/model/sdk_publish_model.g.dart

@@ -203,7 +203,6 @@ class _CodelesslyLayoutBuilderState extends State<CodelesslyLayoutBuilder> {
     }
     final publishModel = context.read<Codelessly>().dataManager.publishModel!;
 
-    // Uses layout id for backwards compatibility.
     final Map<String, VariableData> variablesMap =
         publishModel.variables[widget.layout.id]?.variables[widget.canvasId] ??
             {};
@@ -221,7 +220,6 @@ class _CodelesslyLayoutBuilderState extends State<CodelesslyLayoutBuilder> {
       codelesslyContext.variables[variable.id] = notifier;
     }
 
-    // Uses layout id for backwards compatibility.
     final conditions = publishModel
             .conditions[widget.layout.id]?.conditions[widget.canvasId] ??
         {};