fix(schema): allow rows to be empty in form schema

bpmn-io · Nov 1, 2023 · 71ffc5c · 71ffc5c
1 parent 866b1ab
commit 71ffc5c
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/form-js/CHANGELOG.md b/packages/form-js/CHANGELOG.md
@@ -6,6 +6,10 @@ All notable changes to [form-js](https://github.com/bpmn-io/form-js) are documen
 
 ___Note:__ Yet to be released changes appear here._
 
+### JSON Schema
+
+* `FEAT`: initial release, moved from https://github.com/pinussilvestrus/form-json-schema
+
 ## 1.4.0
 
 ### General 
@@ -37,10 +41,6 @@ ___Note:__ Yet to be released changes appear here._
 
 `FEAT`: provide `additionalModules` to both viewer and editor ([#776](https://github.com/bpmn-io/form-js/pull/776))
 
-### JSON Schema
-
-* `FEAT`: initial release, moved from https://github.com/pinussilvestrus/form-json-schema
-
 ## 1.3.3
 
 ## General

diff --git a/packages/form-json-schema/package.json b/packages/form-json-schema/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@bpmn-io/form-json-schema",
-  "version": "1.3.3",
+  "version": "1.4.0",
   "description": "JSON Schema for form-js",
   "files": [
     "resources"

diff --git a/packages/form-json-schema/src/defs/layout.json b/packages/form-json-schema/src/defs/layout.json
@@ -6,7 +6,7 @@
   "properties": {
     "row": {
       "$id": "/#component/layout/row",
-      "type": "string",
+      "type": [ "string", "null" ],
       "description": "Row in which a form field is placed."
     },
     "columns": {

diff --git a/packages/form-json-schema/test/fixtures/layout-empty-row.js b/packages/form-json-schema/test/fixtures/layout-empty-row.js
@@ -0,0 +1,15 @@
+export const form = {
+  type: 'default',
+  components: [
+    {
+      type: 'textfield',
+      key: 'firstName',
+      layout: {
+        row: null,
+        columns: 12
+      }
+    }
+  ]
+};
+
+export const errors = null;
diff --git a/.../form-json-schema/test/spec/schemaSpec.js → ...form-json-schema/test/spec/schema.spec.js b/.../form-json-schema/test/spec/schemaSpec.js → ...form-json-schema/test/spec/schema.spec.js
diff --git a/...m-json-schema/test/spec/validationSpec.js → ...-json-schema/test/spec/validation.spec.js b/...m-json-schema/test/spec/validationSpec.js → ...-json-schema/test/spec/validation.spec.js
@@ -102,6 +102,9 @@ describe('validation', function() {
   testForm('validate-validationType');
 
 
+  testForm('layout-empty-row');
+
+
   describe('rules - required properties', function() {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -102,6 +102,9 @@ describe('validation', function() {
		testForm('validate-validationType');


		testForm('layout-empty-row');


		describe('rules - required properties', function() {


Expand Down