chore: make helperText Field property optional (#218)

This is a partial revert of 4ce47f0. Fields' `helperText` property doesn't need to be required. This is *arguably* a breaking change, but because it's making a required field optional, I don't think it is.
digidem · Aug 25, 2024 · a220065 · a220065
1 parent 4e9b724
commit a220065
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/proto/field/v1.proto b/proto/field/v1.proto
@@ -38,5 +38,5 @@ message Field_1 {
   repeated Option options = 10;
   bool universal = 11;
   optional string placeholder = 12;
-  string helperText = 13;
+  optional string helperText = 13;
 }
diff --git a/schema/field/v1.json b/schema/field/v1.json
@@ -102,13 +102,6 @@
       "type": "string"
     }
   },
-  "required": [
-    "tagKey",
-    "type",
-    "label",
-    "schemaName",
-    "universal",
-    "helperText"
-  ],
+  "required": ["tagKey", "type", "label", "schemaName", "universal"],
   "additionalProperties": false
 }
diff --git a/test/fixtures/good-docs-minimal.js b/test/fixtures/good-docs-minimal.js
@@ -64,7 +64,6 @@ export const goodDocsMinimal = [
       type: 'text',
       deleted: false,
       universal: false,
-      helperText: '',
     },
     expected: {},
   },